Repository navigation
Pin nova tempest-multinode nodeset on the job definition - #1233
openshift-merge-bot[bot] merged 1 commit into
Conversation
Set the 3xl nodeset on nova-operator-tempest-multinode so all consumers inherit it (cyborg via parent, the s2i-component-jobs template, and the experimental pipeline). Drop the now-redundant per-pipeline overrides. Signed-off-by: Amartya Sinha <amsinha@redhat.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. 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 multinode job now defines the two-node CentOS 9/CRC nodeset. The Nova and Cyborg GitHub checks inherit that nodeset. The Ceph check retains its existing nodeset override. ChangesTempest Nodeset Configuration
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix Suggested reviewers: Merge Risk: ⚪ Minimal · up to Nova and Cyborg checks use the shared two-node CentOS 9/CRC nodeset, while Ceph retains its override; no actionable merge risk is evident. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🧪 Generate unit tests (beta)
🛠️ Fix failing CI checks 💡
Comment |
|
This change depends on a change that failed to merge. Change #1232 is needed. |
|
recheck |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: amartyasinha, gibizer 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 |
|
Build succeeded (check pipeline). ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 00m 48s |
baa8fca
into
openstack-k8s-operators:main
Set the 3xl nodeset on nova-operator-tempest-multinode so all consumers
inherit it (cyborg via parent, the s2i-component-jobs template, and the
experimental pipeline). Drop the now-redundant per-pipeline overrides.
s2i template job was not having nodeset pin, thus CI on s2i repo for nova tempest was flaky.
Depends-On: #1232
Depends-On: openstack-k8s-operators/s2i-openstack-containers#239