Skip to content

Commit

Permalink
Merge pull request #593 from salasberryfin/e2e-test-embedded-capi-dis…
Browse files Browse the repository at this point in the history
…abled-failing

test: fix v1 controller test on embedded-capi-disabled suite
  • Loading branch information
salasberryfin authored Jul 10, 2024
2 parents 0795c35 + 0e1a722 commit 6ce1c25
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ import (
)

var _ = Describe("[AWS] [EKS] Create and delete CAPI cluster functionality should work with namespace auto-import (embedded capi disable from start)", Label(e2e.FullTestLabel), func() {

BeforeEach(func() {
SetClient(setupClusterResult.BootstrapClusterProxy.GetClient())
SetContext(ctx)
Expand Down
5 changes: 3 additions & 2 deletions test/e2e/suites/embedded-capi-disabled/suite_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -197,8 +197,9 @@ var _ = BeforeSuite(func() {
Tag: "v0.0.1",
WaitDeploymentsReadyInterval: e2eConfig.GetIntervals(setupClusterResult.BootstrapClusterProxy.GetName(), "wait-controllers"),
AdditionalValues: map[string]string{
"cluster-api-operator.cert-manager.enabled": "false",
"rancherTurtles.features.embedded-capi.disabled": "false",
"cluster-api-operator.cert-manager.enabled": "false",
"rancherTurtles.features.embedded-capi.disabled": "false",
"rancherTurtles.features.managementv3-cluster.enabled": "false",
},
}
if flagVals.UseEKS {
Expand Down

0 comments on commit 6ce1c25

Please sign in to comment.