Skip to content

Commit

Permalink
rebase and some fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
johannes94 committed Dec 16, 2024
1 parent 3dc51b6 commit d7b23d7
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 5 deletions.
14 changes: 10 additions & 4 deletions e2e/multicluster/multicluster_migration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,14 @@ const (
dpCloudProvider = "standalone"
dpRegion = "standalone"
// the cluster ids configured for given clusters in fleet-manager's cluster config
cluster1ID = "cluster-1-id"
cluster2ID = "cluster-2-id"

// This assumes the ID used for cluster configuration
// REPO/dev/config/dataplane-cluster-configuration.yaml
cluster1ID = "1234567890abcdef1234567890abcdef" // pragma: allowlist secret

// This assumes the ID set for cluster 2 in deploy script
// $REPO/scripts/ci/multicluster_tests/deploy.sh
cluster2ID = "1234567890abcdef1234567890abcdeg" // pragma: allowlist secret
)

var (
Expand Down Expand Up @@ -86,7 +92,7 @@ var _ = Describe("Central Migration Test", Ordered, func() {
}
})

Describe("CentralRequest pre migration", Ordered, func() {
Describe("CentralRequest pre migration", func() {
It("should be assigned to cluster1", func() {
assertClusterAssignment(cluster1ID, centralRequest.Id, fleetmanagerAdminClient)
})
Expand Down Expand Up @@ -137,7 +143,7 @@ var _ = Describe("Central Migration Test", Ordered, func() {
Expect(err).ToNot(HaveOccurred())
})

Describe("CentralRequest post migration", Ordered, func() {
Describe("CentralRequest post migration", func() {
It("should be assigned to cluster2", func() {
assertClusterAssignment(cluster1ID, centralRequest.Id, fleetmanagerAdminClient)
})
Expand Down
1 change: 0 additions & 1 deletion e2e/multicluster/multicluster_suite_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -76,5 +76,4 @@ var _ = BeforeSuite(func() {

route53Client = route53.New(sess)
}

})

0 comments on commit d7b23d7

Please sign in to comment.