New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Inconsistent find-path behavior for cni binaries #46476
Comments
Fixes [kubernetes#46476] Signed-off-by: Abhinav Dahiya <abhinav.dahiya@coreos.com>
…der_cni Automatic merge from submit-queue (batch tested with PRs 46782, 46719, 46339, 46609, 46494) Fix inconsistency in finding cni binaries Fixes [#46476] Signed-off-by: Abhinav Dahiya <abhinav.dahiya@coreos.com> **What this PR does / why we need it**: This fixes the inconsistency in finding the appropriate cni binaries. Currently `lo` cniNetwork follows vendorCniDir > binDir whereas default for all others is binDir > vendorCniDir. This PR makes vendorCniDir > binDir as default behavior. **Why we need it**: Hypercube right now ships cni binaries in /opt/cni/bin. And to use latest version of calico you need to override kubelet's /opt/cni/bin from host which means all other cni plugins (flannel, loopback etc...) have to be mounted from host too. Keeping vendordir at higher order allows easy installation of newer versions of plugins.
Fixes [kubernetes#46476] Signed-off-by: Abhinav Dahiya <abhinav.dahiya@coreos.com>
Issues go stale after 90d of inactivity. Prevent issues from auto-closing with an If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or |
Rotten issues close after 30d of inactivity. Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Is this a request for help?: Yes
What keywords did you search in Kubernetes issues before filing this one? (If you have found any duplicates, you should instead reply there.): cni binaries, cni vendorDirPrefix
Is this a BUG REPORT or FEATURE REQUEST? : feature
What happened:
Inconsistent find-path priority order for cni binaries in CNI:
cni plugin loopback binary path find priority order
The priority order is vendor dir > bin dir
for the all other binaries the path find priority order
The priority order is bin dir > vendor dir
What you expected to happen:
It would be better if this behavior is consistent. Most probably, vendor dir > bin dir makes more sense.
/cc: @aaronlevy
The text was updated successfully, but these errors were encountered: