Skip to content

Commit

Permalink
Cleanup failed services (#911)
Browse files Browse the repository at this point in the history
* chore: Fixes broker 'failed to allocate' test errors

We can only have 5 instances of our test broker running. Tests that use the test broker can see 'failed to allocate' errors. This is because in the case of a failed creation of a service in our tests will fail to clean up the test broker, causing other test runs to fail.

* chore: corrected typo in commit 3ebc0f1

Cleanup referred to wrong service name.
  • Loading branch information
ifindlay-cci authored Jan 6, 2025
1 parent 3ebc0f1 commit d1f5e99
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion acceptance-tests/mssql_fog_existing_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ var _ = Describe("MSSQL Failover Group Existing", Label("mssql-db-failover-group
By("connecting to the existing failover group")
servicePlanExisting := "existing"
serviceNameExisting := random.Name(random.WithPrefix(serviceOffering, servicePlanExisting))
defer services.Delete(serviceName)
defer services.Delete(serviceNameExisting)
dbFogInstance := services.CreateInstance(
serviceOffering,
servicePlanExisting,
Expand Down

0 comments on commit d1f5e99

Please sign in to comment.