Skip to content

Commit

Permalink
chore: Remove AL2 AMISelectorTerms pin from e2e tests now that AL20…
Browse files Browse the repository at this point in the history
…23 accelerated AMIs are available (aws#7253)
  • Loading branch information
bryantbiggs authored and engedaam committed Oct 25, 2024
1 parent fd2b607 commit 940d81f
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions test/suites/integration/extended_resources_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,6 @@ var _ = Describe("Extended Resources", func() {
})
It("should provision nodes for a deployment that requests nvidia.com/gpu", func() {
ExpectNvidiaDevicePluginCreated()
// TODO: jmdeal@ remove AL2 pin once AL2023 accelerated AMIs are available
nodeClass.Spec.AMISelectorTerms = []v1.AMISelectorTerm{{Alias: "al2@latest"}}
numPods := 1
dep := test.Deployment(test.DeploymentOptions{
Replicas: int32(numPods),
Expand Down Expand Up @@ -224,8 +222,6 @@ var _ = Describe("Extended Resources", func() {
}
// Only select private subnets since instances with multiple network instances at launch won't get a public IP.
nodeClass.Spec.SubnetSelectorTerms[0].Tags["Name"] = "*Private*"
// TODO: jmdeal@ remove AL2 pin once AL2023 accelerated AMIs are available
nodeClass.Spec.AMISelectorTerms = []v1.AMISelectorTerm{{Alias: "al2@latest"}}

numPods := 1
dep := test.Deployment(test.DeploymentOptions{
Expand Down

0 comments on commit 940d81f

Please sign in to comment.