-
Notifications
You must be signed in to change notification settings - Fork 69
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
Remove NoDefaultBackupLocation e2e entry + registry probe timeout bump (fix azure flakes) #717
Remove NoDefaultBackupLocation e2e entry + registry probe timeout bump (fix azure flakes) #717
Conversation
kaovilai
commented
May 31, 2022
•
edited
Loading
edited
- revert to b64e96e
- verify test passes 3 times
- ignore github actions failure, separate PR.
- fix azure flake by bumping registry probe timeout 75c8b51
- cherrypick other reverted commits after b64e96e on top except nodefaultbackuplocation e2e
- In follow-up PRs
- add working nodefaultbackuplocation e2e test case this time.
- minimal refactoring for a valid test case.
- refactoring tests to a better state.
- add working nodefaultbackuplocation e2e test case this time.
azure registry probe flake #1 |
/retest |
/test 4.7-operator-e2e-azure |
/test 4.9-operator-e2e-azure infra flake |
ci azure infra create failure. trying again tmr |
/retest |
cherrypicking time :D |
use logrus ParseLevel directly
* fix OADP 526 * add ignore err comments
* Adding initial datamover design * Typo in reviewers * Add summary & user stories * Fixing nits * Adding intial feedback * Adding feedback * nit: changing the selector field name * Adding review comments * Adding plugin clarification * updating image to reflect the latest design * Reflect recent velero csi plugin changes
* OADP-535 allow for nullable resource allocations * Add missing nullables * Include nullable on additionalProperties
infra flake |
/LGTM |
@hhpatel14 please have a double check |
@kaovilai: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM