-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
INIT Verb for CNI 1.1 #975
Comments
Just a note: INIT (and DEINIT if we need) implementation requires lock mechanism in libcni or CNI plugin because libcni may be invoked twice or more at same time. To prevent race condition, we need some lock before INIT is invoked, somewhere, hence we may require
|
What if we call INIT when the network configuration is loaded by the container runtime? Prior the configuration being loaded, the runtime is not able to schedule containers |
At this point, I think we should drop INIT from v1.1. There are enough other things in this release. Thoughts? |
Let’s push it and get some feedback |
This is still awaiting some kind of motivating story; we have nothing against it, but it needs a bit more possible use-cases. |
We have come to the conclusion that INIT should be implemented in the CNI. To do this we need to do the following.
The text was updated successfully, but these errors were encountered: