Pass status along ipam
This PR bumps the plugins to use CNI 1.2.0 and allows delegating the STATUS verb to IPAM plugins.
This way, other CNI plugins implementing STATUS can delegate the STATUS calls to IPAM CNIs.
@LionelJouin very good point. Do you think this affects more than just tests? I'm not sure if there's anything that embeds types.NetConf that also marshal to disk. But, we probably need to revert https://github.com/containernetworking/cni/pull/1039
Could be. I created a ticket for it: https://github.com/containernetworking/cni/issues/1096 SBR (flaking) and tuning plugins are also failing, but I haven't checked the reason yet.
Is this ready to merge, or will you be implementing STATUS passthrough in this PR?
Is this ready to merge, or will you be implementing STATUS passthrough in this PR?
@squeed did you mean GC ? This PR does just that (i.e. status passthrough).
I'd rather defer GC to another PR.
I don't see anything providing an implementation of Status, so this PR lays the groundwork but doesn't, AFAICT, pass STATUS to a delegated plugin.
I don't see anything providing an implementation of Status, so this PR lays the groundwork but doesn't, AFAICT, pass STATUS to a delegated plugin.
I see !
Yeah, let's defer that as well :)
This was done in #1082