-
Notifications
You must be signed in to change notification settings - Fork 18
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
Pull method created data import cron #118
Pull method created data import cron #118
Conversation
Report bugs in Issues The following are automatically added:
Available user actions:
Supported /retest check runs
Supported labels
|
D/S test |
D/S test |
D/S test |
D/S test |
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.
D/S test tox -e verify-bugs-are-open
passed: cnv-tests-tox-executor/5485
D/S test |
D/S test |
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.
D/S test tox -e verify-bugs-are-open
passed: cnv-tests-tox-executor/5544
/verified |
/check-can-merge |
Successfully removed PR tag: quay.io/openshift-cnv/openshift-virtualization-tests:pr-118. |
New container for quay.io/openshift-cnv/openshift-virtualization-tests:latest published |
Short description:
Change the method we used to create a new data import cron
More details:
Changed the way the data import cron is created to deploy the resource from a complete new object
What this PR does / why we need it:
Latest DataImportCron object added required resource on pull_method, which is blocking the creation of the current resource.
In addition since it is not a required resource, created a PR for the wrapper we use - RedHatQE/openshift-python-wrapper#2280
Which issue(s) this PR fixes:
Failing test class
TestDataImportCronDefaultStorageClass