Skip to content

Commit

Permalink
Merge pull request #8042 from cPu1/fix-addons-test
Browse files Browse the repository at this point in the history
Fix addons integration test
  • Loading branch information
cPu1 authored Nov 25, 2024
2 parents aa58c6a + a15085b commit da74269
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions integration/tests/addons/addons_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -619,13 +619,6 @@ var _ = Describe("(Integration) [EKS Addons test]", func() {
assertAddonHasPodIDs(api.AWSEBSCSIDriverAddon, 1)
assertStackExists(makePodIDStackName(api.AWSEBSCSIDriverAddon, ebsCSIControllerSA))
assertStackNotExists(makeIRSAStackName(api.AWSEBSCSIDriverAddon))

By("falling back to IRSA when `autoApplyPodIdentityAssociations: true` but addon doesn't support podIDs")
clusterConfig.Addons = []*api.Addon{{Name: api.AWSEFSCSIDriverAddon}}
Expect(makeCreateAddonCMD()).To(RunSuccessfully())
assertAddonHasPodIDs(api.AWSEFSCSIDriverAddon, 0)
assertStackNotExists(makePodIDStackName(api.AWSEFSCSIDriverAddon, efsCSIControllerSA))
assertStackExists(makeIRSAStackName(api.AWSEFSCSIDriverAddon))
})

It("should remove IAM permissions when deleting addons", func() {
Expand Down

0 comments on commit da74269

Please sign in to comment.