Add new baremetal job to run minor update #3470
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
rabi
force-pushed
the
minor_update
branch
3 times, most recently
from
6473a57 to
cb98d78
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/5ba10f3919104fd2a79357d338bebd6a ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 51m 28s |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/e18adbea8a1c477fb148770e87b080cb ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 41m 10s |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/5c7b87fe3a494f5bb5e3ee1ef9310655 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 48m 21s |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/bf766197804241c6afd86794d9aabf2e ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 50m 46s |
rabi
force-pushed
the
minor_update
branch
3 times, most recently
from
f5ed54b to
71074d3
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/f911fd1b4465476f9d7795e3f0e158fc ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 24m 05s |
rabi
force-pushed
the
minor_update
branch
2 times, most recently
from
90457b4 to
a94e475
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/4878bc7a780d4f8d8da193530ca5d41a ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 49m 43s |
rabi
changed the title
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/3630cd8b93324a2a974c3679cbfc22fe Warning: |
|
recheck |
|
/test images |
rabi
force-pushed
the
minor_update
branch
3 times, most recently
from
0d10a74 to
dd3e719
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/2f52a2895c4f4dfa8fea365cae46aa0a ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 12m 12s |
rabi
force-pushed
the
minor_update
branch
3 times, most recently
from
692d36c to
1949cec
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/1d8c55fa17ee410d81a54e40cf8bbb1a ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 14m 30s |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/4d409cb08e6448e2ae3ab14255c5a4ca ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 20m 23s |
Adds a new baremetal job to test minor update with upstream github content. This will currently test from fr3->main(with PR). Signed-off-by: rabi <ramishra@redhat.com>
|
I tried with #3497 to run the tasks in zuul host and encountered number of issues. If you guys know a better way to do it let me know. It's pretty complex and still does not work yet. I think once we convert other deploy jobs to not use ansible-playbook command, this update job can be changed easily. |
|
Alright. Let's see what Jason would say. If that feature is important and require to be merged ASAP - let's go. |
Thanks Daniel. Are we good to merge this? I agree that we should simplify and remove the controller which adds to complexity in the future. |
ciecierski
left a comment
ciecierski
left a comment
There was a problem hiding this comment.
Looks good to me. Thank you for your work.
Cleaning operators and deploying them back is not the same as using built index with two operators versions, but that is fine to me. In future it would be great to have content provider job that builds correct index and we use it. This would simplify ansible playbooks a lot.
|
@danpawlik PO doesn't need to chime in on every PR, let's merge since it was reviewed and it is for improving edpm-ansible test coverage, owned by EDPM team |
5 participants
Adds a new baremetal job to run minor update.
jira: https://issues.redhat.com/browse/OSPRH-23171
Depends-On: openstack-k8s-operators/install_yamls#1109
Depends-On: openstack-k8s-operators/edpm-ansible#1054