Repository navigation
Conversation
|
Skipping CI for Draft Pull Request. |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
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 Tempest CI jobs now use CentOS 10. The Cyborg job deploys a PCI simulator, configures Cyborg, and creates a device profile before Tempest tests run. ChangesCyborg Tempest Enablement
Priority: ⬇️ Low Estimated code review effort: 3 (Moderate) | ~25 minutes Change: Other Suggested reviewers: Merge Risk: 🟡 Moderate · up to The Cyborg CI setup disables certificate checks for an OpenStack request. If the pod uses its existing credentials, an impersonated endpoint could expose them; restore certificate validation before merging. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
783c843 to
2c221fc
Compare
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
recheck |
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
recheck |
|
Build succeeded (check pipeline). ✔️ openstack-meta-content-provider SUCCESS in 1h 56m 28s |
2c221fc to
3adac2a
Compare
|
Build succeeded (check pipeline). ✔️ openstack-meta-content-provider SUCCESS in 43m 24s |
|
recheck |
|
This change depends on a change with an invalid configuration. |
|
recheck |
|
Build failed (check pipeline). Post ❌ openstack-meta-content-provider FAILURE in 4m 20s |
3adac2a to
e36ac39
Compare
|
This change depends on a change that failed to merge. Change openstack-k8s-operators/ci-framework#4163 is needed. |
e36ac39 to
101e733
Compare
|
Build succeeded (check pipeline). ✔️ openstack-meta-content-provider SUCCESS in 40m 25s |
|
recheck |
|
Build succeeded (check pipeline). ✔️ openstack-meta-content-provider SUCCESS in 42m 24s |
101e733 to
e844ede
Compare
|
Build succeeded (check pipeline). ✔️ openstack-meta-content-provider SUCCESS in 42m 15s |
|
recheck |
|
Build succeeded (check pipeline). ✔️ openstack-meta-content-provider SUCCESS in 2h 14m 22s |
e844ede to
b8b4de1
Compare
|
Build succeeded (check pipeline). ✔️ openstack-meta-content-provider SUCCESS in 1h 17m 30s |
b8b4de1 to
5c6356c
Compare
|
Build succeeded (check pipeline). ✔️ openstack-meta-content-provider SUCCESS in 1h 41m 26s |
|
recheck |
7ab49c8 to
6029dbb
Compare
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
recheck |
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider FAILURE in 5m 33s |
|
recheck |
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider FAILURE in 5m 36s |
|
recheck |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 42m 53s |
|
recheck |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 17m 56s |
|
recheck |
1 similar comment
|
recheck |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 36m 29s |
|
recheck |
6029dbb to
04f47ea
Compare
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 24m 35s |
|
recheck |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 32m 17s |
04f47ea to
f811811
Compare
|
/test functional |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 5h 28m 22s |
|
recheck |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 19m 26s |
f811811 to
34300f4
Compare
Set up comprehensive testing environment for Cyborg PCI drivers: - Update Cyborg tempest job to use CentOS 10 with PCI simulator modules - Configure Cyborg agent with pci_driver enabled via ConfigMap - Implement pre-deployment playbook to build fake PCI SRIOV modules - Add post-deployment resource playbook for device profile creation - Include tempest overrides for Cyborg PCI device profile testing Depends-On: openstack-k8s-operators/s2i-openstack-containers#196 Signed-off-by: Chandan <raukadah@gmail.com>
34300f4 to
c46cef0
Compare
|
Build succeeded (check pipeline). ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 29m 46s |
@SeanMooney thank you for the review. Depends-On: openstack-k8s-operators/s2i-openstack-containers#196 is merged now. We can remove the hold to get it merged, thank you! |
Set up comprehensive testing environment for Cyborg PCI drivers:
Depends-On: openstack-k8s-operators/s2i-openstack-containers#196