Add phase-2 emulated SR-IOV presubmit#5090
Conversation
|
Skipping CI for Draft Pull Request. |
|
[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 |
There was a problem hiding this comment.
Code Review
This pull request adds a new presubmit job, pull-kubevirt-e2e-k8s-1.36-sig-network-emulated-sriov, to the Prow configuration. The reviewer suggested reordering the keys in the job definition (specifically moving run_before_merge) to maintain alphabetical sorting consistency with the rest of the file.
9a49ef5 to
ba09dc3
Compare
|
thanks Daniel ! |
|
Rehearsal jobs created for this PR: Further information on rehearsalsA rehearsal can be triggered for all jobs by commenting either A rehearsal for a specific job can be triggered by commenting Commenting A pull request can be rehearsed if either the user is authorized to rehearse or the pull Authorized users are the group of users that are members of the KubeVirt GitHub [1]: see OWNERS file definition for reference. |
|
/hold until kubevirt/kubevirt#16975 is merged do we want to make kind-sriov optional once this one is merged directly ? |
|
/rehearse failed due to NUMA bug (on it) |
|
Rehearsal jobs created for this PR: Further information on rehearsalsA rehearsal can be triggered for all jobs by commenting either A rehearsal for a specific job can be triggered by commenting Commenting A pull request can be rehearsed if either the user is authorized to rehearse or the pull Authorized users are the group of users that are members of the KubeVirt GitHub [1]: see OWNERS file definition for reference. |
|
Rehearsal jobs created for this PR: Further information on rehearsalsA rehearsal can be triggered for all jobs by commenting either A rehearsal for a specific job can be triggered by commenting Commenting A pull request can be rehearsed if either the user is authorized to rehearse or the pull Authorized users are the group of users that are members of the KubeVirt GitHub [1]: see OWNERS file definition for reference. |
| preset-podman-shared-images: "true" | ||
| preset-shared-images: "true" | ||
| max_concurrency: 11 | ||
| name: pull-kubevirt-e2e-k8s-1.36-emulated-igb |
There was a problem hiding this comment.
Please change the name to refelct this is about testing SR-IOV with emulated HW, at least this is sig-network lane
pull-kubevirt-e2e-k8s-1.36-sig-network-sriov-emulated
There was a problem hiding this comment.
explained here why it is wrong
kubevirt/kubevirt#16975 (comment)
lets please continue on one of them only
|
/rehearse wip: trying with alpine that has emulated SR-IOV |
|
Rehearsal jobs created for this PR: Further information on rehearsalsA rehearsal can be triggered for all jobs by commenting either A rehearsal for a specific job can be triggered by commenting Commenting A pull request can be rehearsed if either the user is authorized to rehearse or the pull Authorized users are the group of users that are members of the KubeVirt GitHub [1]: see OWNERS file definition for reference. |
|
/rehearse |
|
Rehearsal jobs created for this PR: Further information on rehearsalsA rehearsal can be triggered for all jobs by commenting either A rehearsal for a specific job can be triggered by commenting Commenting A pull request can be rehearsed if either the user is authorized to rehearse or the pull Authorized users are the group of users that are members of the KubeVirt GitHub [1]: see OWNERS file definition for reference. |
|
Rehearsal jobs created for this PR: Further information on rehearsalsA rehearsal can be triggered for all jobs by commenting either A rehearsal for a specific job can be triggered by commenting Commenting A pull request can be rehearsed if either the user is authorized to rehearse or the pull Authorized users are the group of users that are members of the KubeVirt GitHub [1]: see OWNERS file definition for reference. |
|
/rehearse |
|
Rehearsal jobs created for this PR: Further information on rehearsalsA rehearsal can be triggered for all jobs by commenting either A rehearsal for a specific job can be triggered by commenting Commenting A pull request can be rehearsed if either the user is authorized to rehearse or the pull Authorized users are the group of users that are members of the KubeVirt GitHub [1]: see OWNERS file definition for reference. |
|
/rehearse |
|
Rehearsal jobs created for this PR: Further information on rehearsalsA rehearsal can be triggered for all jobs by commenting either A rehearsal for a specific job can be triggered by commenting Commenting A pull request can be rehearsed if either the user is authorized to rehearse or the pull Authorized users are the group of users that are members of the KubeVirt GitHub [1]: see OWNERS file definition for reference. |
|
/rehearse all passed with alpine with igb and more required fixes but it would be better to split the effort, as it require few additional steps / bugs |
|
Rehearsal jobs created for this PR: Further information on rehearsalsA rehearsal can be triggered for all jobs by commenting either A rehearsal for a specific job can be triggered by commenting Commenting A pull request can be rehearsed if either the user is authorized to rehearse or the pull Authorized users are the group of users that are members of the KubeVirt GitHub [1]: see OWNERS file definition for reference. |
bb23c46 to
28587ff
Compare
Add a new k8s 1.36 sig-network emulated SR-IOV presubmit as a phase-2 signal that runs before merge instead of always running. Assisted-by: Cursor Signed-off-by: Or Shoval <oshoval@redhat.com>
Signed-off-by: Or Shoval <oshoval@redhat.com>
|
renamed the name and rebased |
What this PR does / why we need it:
Add a new k8s 1.36 sig-network emulated SR-IOV presubmit as a
phase-2 signal that runs before merge instead of always running.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
Depends on kubevirt/kubevirt#16975
once we merge this lane, we can make kind-sriov optional.
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.
Release note: