subnet_test.go•9.17 kB
/*
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/
package nodeclass_test
import (
"github.com/aws/aws-sdk-go-v2/aws"
"github.com/aws/aws-sdk-go-v2/service/ec2"
ec2types "github.com/aws/aws-sdk-go-v2/service/ec2/types"
"github.com/samber/lo"
v1 "github.com/aws/karpenter-provider-aws/pkg/apis/v1"
"github.com/aws/karpenter-provider-aws/pkg/test"
. "github.com/onsi/ginkgo/v2"
. "github.com/onsi/gomega"
. "sigs.k8s.io/karpenter/pkg/test/expectations"
)
var _ = Describe("NodeClass Subnet Status Controller", func() {
BeforeEach(func() {
nodeClass = test.EC2NodeClass(v1.EC2NodeClass{
Spec: v1.EC2NodeClassSpec{
SubnetSelectorTerms: []v1.SubnetSelectorTerm{
{
Tags: map[string]string{"*": "*"},
},
},
SecurityGroupSelectorTerms: []v1.SecurityGroupSelectorTerm{
{
Tags: map[string]string{"*": "*"},
},
},
AMIFamily: lo.ToPtr(v1.AMIFamilyCustom),
AMISelectorTerms: []v1.AMISelectorTerm{
{
Tags: map[string]string{"*": "*"},
},
},
},
})
})
It("Should update EC2NodeClass status for Subnets", func() {
ExpectApplied(ctx, env.Client, nodeClass)
ExpectObjectReconciled(ctx, env.Client, controller, nodeClass)
nodeClass = ExpectExists(ctx, env.Client, nodeClass)
Expect(nodeClass.Status.Subnets).To(Equal([]v1.Subnet{
{
ID: "subnet-test1",
Zone: "test-zone-1a",
ZoneID: "tstz1-1a",
},
{
ID: "subnet-test2",
Zone: "test-zone-1b",
ZoneID: "tstz1-1b",
},
{
ID: "subnet-test3",
Zone: "test-zone-1c",
ZoneID: "tstz1-1c",
},
{
ID: "subnet-test4",
Zone: "test-zone-1a-local",
ZoneID: "tstz1-1alocal",
},
}))
Expect(nodeClass.StatusConditions().IsTrue(v1.ConditionTypeSubnetsReady)).To(BeTrue())
})
It("Should have the correct ordering for the Subnets", func() {
awsEnv.EC2API.DescribeSubnetsBehavior.Output.Set(&ec2.DescribeSubnetsOutput{Subnets: []ec2types.Subnet{
{SubnetId: aws.String("subnet-test1"), AvailabilityZone: aws.String("test-zone-1a"), AvailabilityZoneId: aws.String("tstz1-1a"), AvailableIpAddressCount: aws.Int32(20)},
{SubnetId: aws.String("subnet-test2"), AvailabilityZone: aws.String("test-zone-1b"), AvailabilityZoneId: aws.String("tstz1-1b"), AvailableIpAddressCount: aws.Int32(100)},
{SubnetId: aws.String("subnet-test3"), AvailabilityZone: aws.String("test-zone-1c"), AvailabilityZoneId: aws.String("tstz1-1c"), AvailableIpAddressCount: aws.Int32(50)},
}})
ExpectApplied(ctx, env.Client, nodeClass)
ExpectObjectReconciled(ctx, env.Client, controller, nodeClass)
nodeClass = ExpectExists(ctx, env.Client, nodeClass)
Expect(nodeClass.Status.Subnets).To(Equal([]v1.Subnet{
{
ID: "subnet-test2",
Zone: "test-zone-1b",
ZoneID: "tstz1-1b",
},
{
ID: "subnet-test3",
Zone: "test-zone-1c",
ZoneID: "tstz1-1c",
},
{
ID: "subnet-test1",
Zone: "test-zone-1a",
ZoneID: "tstz1-1a",
},
}))
Expect(nodeClass.StatusConditions().IsTrue(v1.ConditionTypeSubnetsReady)).To(BeTrue())
})
It("Should resolve a valid selectors for Subnet by tags", func() {
nodeClass.Spec.SubnetSelectorTerms = []v1.SubnetSelectorTerm{
{
Tags: map[string]string{`Name`: `test-subnet-1`},
},
{
Tags: map[string]string{`Name`: `test-subnet-2`},
},
}
ExpectApplied(ctx, env.Client, nodeClass)
ExpectObjectReconciled(ctx, env.Client, controller, nodeClass)
nodeClass = ExpectExists(ctx, env.Client, nodeClass)
Expect(nodeClass.Status.Subnets).To(Equal([]v1.Subnet{
{
ID: "subnet-test1",
Zone: "test-zone-1a",
ZoneID: "tstz1-1a",
},
{
ID: "subnet-test2",
Zone: "test-zone-1b",
ZoneID: "tstz1-1b",
},
}))
Expect(nodeClass.StatusConditions().IsTrue(v1.ConditionTypeSubnetsReady)).To(BeTrue())
})
It("Should resolve a valid selectors for Subnet by ids", func() {
nodeClass.Spec.SubnetSelectorTerms = []v1.SubnetSelectorTerm{
{
ID: "subnet-test1",
},
}
ExpectApplied(ctx, env.Client, nodeClass)
ExpectObjectReconciled(ctx, env.Client, controller, nodeClass)
nodeClass = ExpectExists(ctx, env.Client, nodeClass)
Expect(nodeClass.Status.Subnets).To(Equal([]v1.Subnet{
{
ID: "subnet-test1",
Zone: "test-zone-1a",
ZoneID: "tstz1-1a",
},
}))
Expect(nodeClass.StatusConditions().IsTrue(v1.ConditionTypeSubnetsReady)).To(BeTrue())
})
It("Should update Subnet status when the Subnet selector gets updated by tags", func() {
ExpectApplied(ctx, env.Client, nodeClass)
ExpectObjectReconciled(ctx, env.Client, controller, nodeClass)
nodeClass = ExpectExists(ctx, env.Client, nodeClass)
Expect(nodeClass.Status.Subnets).To(Equal([]v1.Subnet{
{
ID: "subnet-test1",
Zone: "test-zone-1a",
ZoneID: "tstz1-1a",
},
{
ID: "subnet-test2",
Zone: "test-zone-1b",
ZoneID: "tstz1-1b",
},
{
ID: "subnet-test3",
Zone: "test-zone-1c",
ZoneID: "tstz1-1c",
},
{
ID: "subnet-test4",
Zone: "test-zone-1a-local",
ZoneID: "tstz1-1alocal",
},
}))
nodeClass.Spec.SubnetSelectorTerms = []v1.SubnetSelectorTerm{
{
Tags: map[string]string{
"Name": "test-subnet-1",
},
},
{
Tags: map[string]string{
"Name": "test-subnet-2",
},
},
}
ExpectApplied(ctx, env.Client, nodeClass)
ExpectObjectReconciled(ctx, env.Client, controller, nodeClass)
nodeClass = ExpectExists(ctx, env.Client, nodeClass)
Expect(nodeClass.Status.Subnets).To(Equal([]v1.Subnet{
{
ID: "subnet-test1",
Zone: "test-zone-1a",
ZoneID: "tstz1-1a",
},
{
ID: "subnet-test2",
Zone: "test-zone-1b",
ZoneID: "tstz1-1b",
},
}))
Expect(nodeClass.StatusConditions().IsTrue(v1.ConditionTypeSubnetsReady)).To(BeTrue())
})
It("Should update Subnet status when the Subnet selector gets updated by ids", func() {
ExpectApplied(ctx, env.Client, nodeClass)
ExpectObjectReconciled(ctx, env.Client, controller, nodeClass)
nodeClass = ExpectExists(ctx, env.Client, nodeClass)
Expect(nodeClass.Status.Subnets).To(Equal([]v1.Subnet{
{
ID: "subnet-test1",
Zone: "test-zone-1a",
ZoneID: "tstz1-1a",
},
{
ID: "subnet-test2",
Zone: "test-zone-1b",
ZoneID: "tstz1-1b",
},
{
ID: "subnet-test3",
Zone: "test-zone-1c",
ZoneID: "tstz1-1c",
},
{
ID: "subnet-test4",
Zone: "test-zone-1a-local",
ZoneID: "tstz1-1alocal",
},
}))
nodeClass.Spec.SubnetSelectorTerms = []v1.SubnetSelectorTerm{
{
ID: "subnet-test1",
},
}
ExpectApplied(ctx, env.Client, nodeClass)
ExpectObjectReconciled(ctx, env.Client, controller, nodeClass)
nodeClass = ExpectExists(ctx, env.Client, nodeClass)
Expect(nodeClass.Status.Subnets).To(Equal([]v1.Subnet{
{
ID: "subnet-test1",
Zone: "test-zone-1a",
ZoneID: "tstz1-1a",
},
}))
Expect(nodeClass.StatusConditions().IsTrue(v1.ConditionTypeSubnetsReady)).To(BeTrue())
})
It("Should not resolve a invalid selectors for Subnet", func() {
nodeClass.Spec.SubnetSelectorTerms = []v1.SubnetSelectorTerm{
{
Tags: map[string]string{`foo`: `invalid`},
},
}
ExpectApplied(ctx, env.Client, nodeClass)
ExpectObjectReconciled(ctx, env.Client, controller, nodeClass)
nodeClass = ExpectExists(ctx, env.Client, nodeClass)
Expect(nodeClass.Status.Subnets).To(BeNil())
Expect(nodeClass.StatusConditions().Get(v1.ConditionTypeSubnetsReady).IsFalse()).To(BeTrue())
})
It("Should not resolve a invalid selectors for an updated subnet selector", func() {
ExpectApplied(ctx, env.Client, nodeClass)
ExpectObjectReconciled(ctx, env.Client, controller, nodeClass)
nodeClass = ExpectExists(ctx, env.Client, nodeClass)
Expect(nodeClass.Status.Subnets).To(Equal([]v1.Subnet{
{
ID: "subnet-test1",
Zone: "test-zone-1a",
ZoneID: "tstz1-1a",
},
{
ID: "subnet-test2",
Zone: "test-zone-1b",
ZoneID: "tstz1-1b",
},
{
ID: "subnet-test3",
Zone: "test-zone-1c",
ZoneID: "tstz1-1c",
},
{
ID: "subnet-test4",
Zone: "test-zone-1a-local",
ZoneID: "tstz1-1alocal",
},
}))
nodeClass.Spec.SubnetSelectorTerms = []v1.SubnetSelectorTerm{
{
Tags: map[string]string{`foo`: `invalid`},
},
}
ExpectApplied(ctx, env.Client, nodeClass)
ExpectObjectReconciled(ctx, env.Client, controller, nodeClass)
nodeClass = ExpectExists(ctx, env.Client, nodeClass)
Expect(nodeClass.Status.Subnets).To(BeNil())
Expect(nodeClass.StatusConditions().Get(v1.ConditionTypeSubnetsReady).IsFalse()).To(BeTrue())
})
})