Closed justinsb closed 1 year ago
/test pull-declarative-test
We're getting closer, but sometimes kubectl seems to skip one or more discovery requests...
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: justinsb, yuwenma
The full list of commands accepted by this bot can be found here.
The pull request process is described here
We sleep briefly (100ms) after updating the status.
This is a hack to ensure that we see the watch event, because otherwise we can start a re-reconcile based on our derived objects, and then reconcile again based on our own status update.
This is racy behaviour and causes non-determinism which breaks our golden tests.
We do this only in this test controller, but maybe we should have similar logic to "debounce" in all controllers.
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Additional documentation: