New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix for TestStorageVersionMigrationWithCRD integration test failure #123983
Fix for TestStorageVersionMigrationWithCRD integration test failure #123983
Conversation
/sig api-machinery |
/cc @enj |
Signed-off-by: Anish Ramasekar <anish.ramasekar@gmail.com>
0d5a112
to
4f1c91b
Compare
/lgtm |
LGTM label has been added. Git tree hash: d32de55c131bc322674bdfccb83ec5987f6a8cac
|
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aramase, enj The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/milestone v1.30 (fixing flaking test) |
What type of PR is this?
/kind flake
What this PR does / why we need it:
Looking at the logs, the actual failure is here
kubernetes/test/integration/storageversionmigrator/util.go
Lines 798 to 801 in a2a7090
that's from
kubernetes/test/integration/storageversionmigrator/util.go
Lines 1002 to 1010 in a2a7090
which is failing after the 30s context deadline. Increasing the timeout to wait for migration to complete.
Which issue(s) this PR fixes:
Fixes #123921
Does this PR introduce a user-facing change?