-
Notifications
You must be signed in to change notification settings - Fork 56
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
ospdo adoption ovn adoption #691
base: main
Are you sure you want to change the base?
ospdo adoption ovn adoption #691
Conversation
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/829b96bfa6714628a1a57ed9087eb705 ❌ adoption-standalone-to-crc-ceph FAILURE in 1h 46m 08s |
880ebfa
to
9c37dbf
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/3b1ab48e5e004401b7a88b6cd1b3a23e ❌ adoption-standalone-to-crc-ceph RETRY_LIMIT in 14m 59s |
9c37dbf
to
5ed66b8
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/3e586397826e475c91fc4b8a59fa3943 ❌ adoption-standalone-to-crc-ceph FAILURE in 1h 46m 53s |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/50ff7a1d710b4a3ab7e72696b9a09b9b ❌ adoption-standalone-to-crc-ceph FAILURE in 53m 44s |
recheck |
9fd8aea
to
b8e13d7
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/084a1a5f4c174fea9834e4e2069b6000 ✔️ adoption-standalone-to-crc-ceph SUCCESS in 3h 27m 00s |
recheck |
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, also tagging @holser and @archana203 because the RHV based job is more sensitive to the cleanup logic being correct, but i think what we have here should work fine there too.
b8e13d7
to
3cebd7f
Compare
@@ -33,3 +33,16 @@ ovn_nic_mapping_patch: | | |||
ovnController: | |||
nicMappings: | |||
baremetal: baremetal | |||
dpa_dir: "../.." |
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.
Why these variables here?
Add Director Operator ovn_adoption scripts for adoption
Related CI patch and background:
https://gitlab.cee.redhat.com/ci-framework/ci-framework-jobs/-/merge_requests/966
PRs submitted together :
related PR list
Jiras:
https://issues.redhat.com/browse/OSPRH-1891
https://issues.redhat.com/browse/OSPRH-2984