Bump dependencies #184
Bump dependencies #184
Conversation
Signed-off-by: Paulo Gomes <[email protected]>
Signed-off-by: Paulo Gomes <[email protected]>
Signed-off-by: Paulo Gomes <[email protected]>
There was a problem hiding this comment.
Note that weave CNI deprecation should be following this plan:
Weave is deprecated in 1.27, 1.28 and 1.29. Weave will be removed in 1.30 as stated here: rancher/rancher#42730
package/Dockerfile
Outdated
| RUN xx-verify --static /tmp/bandwidth \ | ||
| && xx-verify --static /tmp/bridge \ | ||
| && xx-verify --static /tmp/dhcp \ | ||
| && xx-verify --static /tmp/firewall \ | ||
| && xx-verify --static /tmp/flannel \ | ||
| && xx-verify --static /tmp/host-device \ | ||
| && xx-verify --static /tmp/host-local \ | ||
| && xx-verify --static /tmp/ipvlan \ | ||
| && xx-verify --static /tmp/loopback \ | ||
| && xx-verify --static /tmp/macvlan \ | ||
| && xx-verify --static /tmp/portmap \ | ||
| && xx-verify --static /tmp/ptp \ | ||
| && xx-verify --static /tmp/sbr \ | ||
| && xx-verify --static /tmp/static \ | ||
| && xx-verify --static /tmp/tuning \ | ||
| && xx-verify --static /tmp/vlan \ | ||
| && xx-verify --static /tmp/vrf |
There was a problem hiding this comment.
Aren't these binaries used for the other CNI plugins (e.g. flannel needs the bridge CNI plugin)? They might come from another place but just in case I ask
There was a problem hiding this comment.
In fact, I created this issue when I was looking at RKE1 but I never had time: rancher/rke#3327
There was a problem hiding this comment.
Aren't these binaries used for the other CNI plugins (e.g. flannel needs the bridge CNI plugin)?
I could not find anywhere else looking for those files at /tmp/ apart from the Weave CNI tbh. But happy to follow that up as part of the issue you created on RKE.
Signed-off-by: Paulo Gomes <[email protected]>
Signed-off-by: Paulo Gomes <[email protected]>
Signed-off-by: Paulo Gomes <[email protected]>
|
@manuelbuil thank you for the review and linking up the other issues. I reverted the changes around removal of Weave CNI for the time being. Instead I have just bumped their versions instead. PTAL |
No description provided.