Skip to content

Cherry-pick : Fixing a couple of network plugin life-cycle mgmt issues#29858

Merged
tiborvass merged 4 commits intomoby:1.13.xfrom
mavenugo:1.13-nplugins
Jan 4, 2017
Merged

Cherry-pick : Fixing a couple of network plugin life-cycle mgmt issues#29858
tiborvass merged 4 commits intomoby:1.13.xfrom
mavenugo:1.13-nplugins

Conversation

@mavenugo
Copy link
Contributor

@mavenugo mavenugo commented Jan 3, 2017

Cherry-picking #29556

@vieux
Copy link
Contributor

vieux commented Jan 3, 2017

@mavenugo doesn't compile

@mavenugo
Copy link
Contributor Author

mavenugo commented Jan 3, 2017

@vieux yes. we also need to cherry-pick #29665. I will take care of it.

The current GetAll handles both V2 and legacy plugins. Also due to the
nature of V1 plugins, it also loads them. This causes problems when
loading is not required. Hence adding an independent API that will
return only the plugins that are loaded using v2 mangaed plugins.

Signed-off-by: Madhu Venugopal <madhu@docker.com>
Signed-off-by: Madhu Venugopal <madhu@docker.com>
Signed-off-by: Madhu Venugopal <madhu@docker.com>
Signed-off-by: Madhu Venugopal <madhu@docker.com>
@vieux
Copy link
Contributor

vieux commented Jan 4, 2017

@mavenugo windows error related ?

@vieux
Copy link
Contributor

vieux commented Jan 4, 2017

ping @tiborvass

LGTM

@tiborvass
Copy link
Contributor

LGTM

@thaJeztah
Copy link
Member

@mavenugo looks like this was not a 1-on-1 cherry pick of #29556, but also contains changes that are in #29891, correct?

@mavenugo
Copy link
Contributor Author

mavenugo commented Jan 9, 2017

@thaJeztah the cherry-pick for 1.13.0 branch in docker must come from libnetwork v0.9 branch and yes, by the time this vendoring happened we got 1 additional patch from #29891 which fixes #28836.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants