kubernetes: Many CNI Plugins don't properly support DEL
Started as a tangent of #43014
Many CNI plugins don’t properly handle a DEL action on missing resources. The spec says, Plugins should generally complete a DEL action without error even if some resources are missing, however many improperly return an error.
I see the steps moving forward to be:
- Modify CNI plugins bundled with Kubernetes to properly support del (flannel already done; bridge, ipvlan, macvlan, and ptp in containernetworking/cni#412)
- Create release of containernetworking/cni
- Build Kubernetes version of CNI using Makefile
- Modify references to CNI version throughout project
Does this make sense?
cc/ @caseydavenport @luxas @calebamiles @yujuhong @janwillies @yujuhong @dchen1107 @djosborne @jbeda @bowei @thockin @feiskyer @aaronlevy @sameo @GheRivero @tomdee @bboreham @cmluciano @dcbw @freehan
About this issue
- Original URL
- State: closed
- Created 7 years ago
- Comments: 17 (13 by maintainers)
Commits related to this issue
- Merge pull request #43536 from ethernetdan/cni_plugins_v0.5.1 Automatic merge from submit-queue bump Kubernetes packaged CNI to v0.5.1 Update version of packaged CNI to v0.5.1 in order as part of a... — committed to kubernetes/kubernetes by deleted user 7 years ago
- Merge pull request #43546 from calebamiles/wip-bump-cni-ref Automatic merge from submit-queue Bump CNI consumers to v0.5.1 **What this PR does / why we need it**: - vendored CNI plugins properly h... — committed to kubernetes/kubernetes by deleted user 7 years ago
- WIP: Bump to Kubernetes v1.6.1 (#492) * Bump to Kubernetes v1.6.1 This change was just the result of running the following commands: ``` $ contrib/bump-version v1.6.1_coreos.0 Updating contri... — committed to kubernetes-retired/kube-aws by mumoshu 7 years ago
- WIP: Bump to Kubernetes v1.6.1 (#492) * Bump to Kubernetes v1.6.1 This change was just the result of running the following commands: ``` $ contrib/bump-version v1.6.1_coreos.0 Updating contri... — committed to HotelsDotCom/kube-aws by mumoshu 7 years ago
@cmluciano It would be great to get flannel tested too. You probably need a master build of the flannel CNI plugin and possibly some flannel patches to be merged. Feel free to DM on slack if you need more info.
I just want to say thanks to everyone for jumping on this.
On Tue, Mar 21, 2017 at 6:11 PM, Tom Denham notifications@github.com wrote: