Skip to content

Commit

Permalink
[CRUD Suite] Delete nodegroups in subnet before the subnet itself (#7250
Browse files Browse the repository at this point in the history
)

* Improve CRUD suite runtime~

* refactor to use ginkgo decorators

* fix IAM service account

* change cluster name

* remove duplicate cluster workloads test

* refactor describe stacks test

* [CRUD integration] Add wait flag to delete cluster command

* delete additional subnet

* fix typo

Co-authored-by: Chetan Patwal <cPu1@users.noreply.github.com>

* small refactor to trigger workflows

* [CRUD Suite] delete nodegroups in subnet before the subnet itself

---------

Co-authored-by: Chetan Patwal <cPu1@users.noreply.github.com>
  • Loading branch information
TiberiuGC and cPu1 authored Nov 6, 2023
1 parent c402580 commit 09620d8
Showing 1 changed file with 16 additions and 2 deletions.
18 changes: 16 additions & 2 deletions integration/tests/crud/creategetdelete_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -947,6 +947,14 @@ var _ = Describe("(Integration) Create, Get, Scale & Delete", func() {
).
WithoutArg("--region", params.Region).
WithStdin(clusterutils.Reader(clusterConfig))).To(RunSuccessfully())

Expect(params.EksctlDeleteCmd.WithArgs(
"nodegroup",
"--verbose", "4",
"--cluster", params.ClusterName,
"--wait",
newSubnetConfigFileMng,
)).To(RunSuccessfully())
})

It("should be able to create a nodegroup in a new subnet via CLI", func() {
Expand All @@ -959,6 +967,14 @@ var _ = Describe("(Integration) Create, Get, Scale & Delete", func() {
"--subnet-ids", extraSubnetID,
newSubnetCLIMng,
)).To(RunSuccessfully())

Expect(params.EksctlDeleteCmd.WithArgs(
"nodegroup",
"--verbose", "4",
"--cluster", params.ClusterName,
"--wait",
newSubnetCLIMng,
)).To(RunSuccessfully())
})
})
})
Expand Down Expand Up @@ -1075,8 +1091,6 @@ var _ = Describe("(Integration) Create, Get, Scale & Delete", func() {
ContainElement(scaleMultipleMng),
ContainElement(GPUMng),
ContainElement(drainMng),
ContainElement(newSubnetCLIMng),
ContainElement(newSubnetConfigFileMng),
)))
})
})
Expand Down

0 comments on commit 09620d8

Please sign in to comment.