Kueue downstream ci for s390x - #83236
Conversation
|
Hi @Meghagaur. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Meghagaur 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 |
|
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 configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (4)
🚧 Files skipped from review as they are similar to previous changes (4)
WalkthroughThe PR adds a release-1.4 Kueue operator configuration for s390x libvirt testing. It defines the cluster workflow, release inputs, daily end-to-end test, ownership, and generated metadata. ChangesKueue s390x libvirt CI
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant Prow
participant KueueReleaseConfig
participant LibvirtS390xWorkflow
participant Boskos
participant IBMZLibvirtHypervisors
participant DownstreamE2ETest
Prow->>KueueReleaseConfig: Start daily release test
KueueReleaseConfig->>LibvirtS390xWorkflow: Start s390x workflow
LibvirtS390xWorkflow->>Boskos: Acquire managed resources
LibvirtS390xWorkflow->>IBMZLibvirtHypervisors: Provision ephemeral OpenShift cluster
LibvirtS390xWorkflow->>DownstreamE2ETest: Run operator end-to-end test
LibvirtS390xWorkflow->>IBMZLibvirtHypervisors: Gather logs and clean up
Important Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional. ❌ Failed checks (1 error, 1 inconclusive)
✅ Passed checks (13 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
/ok-to-test |
|
@sivaramsingana: Cannot trigger testing until a trusted user reviews the PR and leaves an DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/ok-to-test |
|
/ok-to-test |
2c27259 to
1b35c38
Compare
|
/pj-rehearse periodic-ci-openshift-kueue-operator-release-1.4-libvirt-s390x-test-e2e-downstream-4-22 |
|
@Meghagaur: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/ok-to-test |
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
|
/pj-rehearse periodic-ci-openshift-kueue-operator-release-1.4-libvirt-s390x-test-e2e-downstream-4-22 |
|
@Meghagaur: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
e97f2ec to
4de17cf
Compare
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
|
/retest |
|
/pj-rehearse periodic-ci-openshift-kueue-operator-release-1.4-libvirt-s390x-test-e2e-downstream-4-22 |
|
@Meghagaur: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
@Meghagaur: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
/pj-rehearse periodic-ci-openshift-kueue-operator-release-1.4-libvirt-s390x-test-e2e-downstream-4-22 |
|
@Meghagaur: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
Summary by CodeRabbit
This PR adds downstream CI support for the Kueue operator release-1.4 on s390x. It configures 4.22 release inputs, build dependencies, resource requests, and a daily downstream end-to-end test.
The test uses the
libvirt-s390x-vpn-ozprofile and thekueue-operator-libvirt-s390xworkflow. The workflow provisions ephemeral s390x OpenShift 4.21 clusters on remote IBM Z libvirt hypervisors, then performs test execution, log collection, and cleanup.