[release-4.21] OCPBUGS-82064: Bump CNI version to 1.1.0#284
Conversation
Changes-Include: - Add STATUS handling for delegate requests and single‑plugin - Invoke STATUS for conf/conflist delegates via libcni - Preserve CNI error codes/messages through daemon API and shim - Add tests for STATUS error propagation Signed-off-by: Tim Rozet <trozet@nvidia.com> (cherry picked from commit ea38900)
Signed-off-by: Tim Rozet <trozet@nvidia.com> (cherry picked from commit e091897)
So that we can be compatible with 1.1.0. Signed-off-by: Tim Rozet <trozet@nvidia.com> (cherry picked from commit 921191d)
support only existed for confList before. Signed-off-by: Tim Rozet <trozet@nvidia.com> (cherry picked from commit 56d18ef)
Fix thin auto-config generation to select 00-multus.conflist for all CNI versions >= 1.0.0 (not only exact 1.0.0) Signed-off-by: Tim Rozet <trozet@nvidia.com> (cherry picked from commit c943f9f)
|
@arkadeepsen: This pull request explicitly references no jira issue. 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 openshift-eng/jira-lifecycle-plugin repository. |
|
Important Review skippedAuto reviews are limited based on label configuration. 🚫 Review skipped — only excluded labels are configured. (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
|
@arkadeepsen: This PR was included in a payload test run from openshift/ovn-kubernetes#3087
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/b24e2180-336e-11f1-8eef-893f056ce8e5-0 |
|
@arkadeepsen: This pull request references Jira Issue OCPBUGS-82064, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. 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 openshift-eng/jira-lifecycle-plugin repository. |
|
https://github.com/k8snetworkplumbingwg/multus-cni/pull/1490/commits need these commits as well @arkadeepsen |
STATUS and GC are plugin-level commands with no pod context per the CNI 1.1.0 spec. The thick plugin daemon incorrectly required CNI_CONTAINERID, CNI_NETNS, and K8S_POD_NAME/K8S_POD_NAMESPACE for these commands, causing failures when invoked by kubelet. Signed-off-by: Jason Kary <jkary@redhat.com> (cherry picked from commit dfb8eaa)
- Check os.OpenFile error in STATUS/GC test - Document that k8sArgs may be nil for STATUS/GC in HandleCNIRequest Made-with: Cursor (cherry picked from commit 2a1b966)
|
@arkadeepsen: This PR was included in a payload test run from openshift/ovn-kubernetes#3087
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/1aba7c50-34ff-11f1-8881-f5621743b1c0-0 |
|
/approve |
|
/label backport-risk-assessed |
|
/jira refresh |
|
@arkadeepsen: This pull request references Jira Issue OCPBUGS-82064, which is valid. 7 validation(s) were run on this bug
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 openshift-eng/jira-lifecycle-plugin repository. |
|
@arkadeepsen: This PR was included in a payload test run from openshift/ovn-kubernetes#3087
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/c1cb8020-371b-11f1-8314-beaf8a27bf87-0 |
|
/retest-required |
|
@arkadeepsen: all tests passed! 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. |
|
@arkadeepsen: This PR has been marked as verified by 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: arkadeepsen, bpickard22 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 |
4f696e7
into
openshift:release-4.21
|
@arkadeepsen: Jira Issue OCPBUGS-82064: Some pull requests linked via external trackers have merged: The following pull request, linked via external tracker, has not merged:
All associated pull requests must be merged or unlinked from the Jira bug in order for it to move to the next state. Once unlinked, request a bug refresh with Jira Issue OCPBUGS-82064 has not been moved to the MODIFIED state. This PR is marked as verified. If the remaining PRs listed above are marked as verified before merging, the issue will automatically be moved to VERIFIED after all of the changes from the PRs are available in an accepted nightly payload. 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 openshift-eng/jira-lifecycle-plugin repository. |
|
Fix included in release 4.21.0-0.nightly-2026-04-14-123546 |
Manual cherry-pick of ea38900 e091897 921191d 56d18ef c943f9f dfb8eaa 2a1b966. No conflicts.