Skip to content

Fix sporadic test failure in service broker deletion during creation#5045

Merged
philippthun merged 1 commit intocloudfoundry:mainfrom
sap-contributions:fix-sporadic-test-failures-service-broker-deletion
Apr 20, 2026
Merged

Fix sporadic test failure in service broker deletion during creation#5045
philippthun merged 1 commit intocloudfoundry:mainfrom
sap-contributions:fix-sporadic-test-failures-service-broker-deletion

Conversation

@philippthun
Copy link
Copy Markdown
Member

The test was flaky because both create and delete jobs could be enqueued with the same timestamp, making pickup order non-deterministic. When the delete job ran first, the create job failed with ServiceBrokerGone.

Fix by setting a higher priority number for the delete job so the create job always runs first.

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

The test was flaky because both create and delete jobs could be enqueued
with the same timestamp, making pickup order non-deterministic. When the
delete job ran first, the create job failed with ServiceBrokerGone.

Fix by setting a higher priority number for the delete job so the create
job always runs first.
Copy link
Copy Markdown
Contributor

@johha johha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@philippthun philippthun merged commit fe7834d into cloudfoundry:main Apr 20, 2026
11 checks passed
ari-wg-gitbot added a commit to cloudfoundry/capi-release that referenced this pull request Apr 20, 2026
Changes in cloud_controller_ng:

- Fix sporadic test failure in service broker deletion during creation
    PR: cloudfoundry/cloud_controller_ng#5045
    Author: Philipp Thun <philipp.thun@sap.com>

- Fix race condition in route destination updates
    PR: cloudfoundry/cloud_controller_ng#4945
    Author: Philipp Thun <philipp.thun@sap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants