Closed justinsb closed 1 year ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: justinsb
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/hold while I figure out how the read was avoided - probably the watch, but TBD!
/retest
/hold cancel
Changes look pretty good now, just more resources.
Will probably still be flaky until the next PR goes in (starting watches before apply)
/retest
/retest
Took 4 tries previously, this is number 2
/retest
/retest
/retest
@justinsb: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
pull-declarative-test | a7e9f8cc38bcf51789135b45c1925326b629c0be | link | true | /test pull-declarative-test |
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.
This was done as part of the hooks PR, which got the tests (hopefully) into a deterministic state, so fewer flakes :crossed_fingers:
/close
@justinsb: Closed this PR.
We want to pick up the latest fixes to mockkubeapiserver.
We regenerate the test data also, we now have more API types.
Also we add more normalization around discovery requests, we try to reorder adjacent requests into alphabetical order, now that we are doing more of them this is more important for golden tests to be the same.