Repository navigation
Switch component (tempest) jobs to use CS10 - #1250
openshift-merge-bot[bot] merged 2 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe Nova, Cyborg, and Ceph Tempest configurations change their CentOS scenario or nodeset settings. The Ceph job replaces an extras setting and removes a Cinder backend override. The S2I content-provider image list no longer includes ChangesTempest job configuration
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~8 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The PR moves the Tempest jobs to CS10 and adjusts Ceph S2I configuration. The required framework resources are available, and no concrete user-facing or CI failure is established; no actionable merge blocker remains. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
/retest-required |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 4h 37m 52s |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 6h 47m 15s |
When cifmw_extras, one of the included defined in the base ceph job is lost (@scenarios/centos-9/hci_ceph_backends.yml). But s2i_extras allows child jobs to add additional extra variables. Depends-On: openstack-k8s-operators/s2i-openstack-containers#200 Signed-off-by: Luigi Toscano <ltoscano@redhat.com>
|
Build succeeded (check pipeline). ✔️ openstack-k8s-operators-content-provider SUCCESS in 4h 12m 13s |
|
/retest-required |
Since we've CS10 nodesets available openstack-k8s-operators/ci-framework#4163, let's use them Signed-off-by: Amartya Sinha <amsinha@redhat.com>
|
Build succeeded (check pipeline). ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 29m 26s |
gibizer
left a comment
There was a problem hiding this comment.
green and stable. thanks
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: amartyasinha, ashu-011, gibizer, mrkisaolamb The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
eae1ba1
into
openstack-k8s-operators:main
Since we've CS10 nodesets available openstack-k8s-operators/ci-framework#4163, let's use them
openstack-k8s-operators/s2i-openstack-containers#196 sets s2i-test-base-ceph job's parent to be cs10, thus our ceph job which is dependent on that can use cs10 nodeset without any issue.