Skip to content

Pass plugingetter as part of swarm node config.#30145

Merged
tiborvass merged 2 commits intomoby:masterfrom
anusha-ragunathan:ps-swarm
Jan 20, 2017
Merged

Pass plugingetter as part of swarm node config.#30145
tiborvass merged 2 commits intomoby:masterfrom
anusha-ragunathan:ps-swarm

Conversation

@anusha-ragunathan
Copy link
Contributor

This is necessary for swarmkit to support cluster wide plugins, such as
globally scoped network plugins.

Signed-off-by: Anusha Ragunathan [email protected]

cc @mavenugo @aaronlehmann

@vdemeester
Copy link
Member

@anusha-ragunathan CI failures 👼

23:46:21 These files are not properly gofmt'd:
23:46:21  - daemon/cluster/noderunner.go

@mavenugo
Copy link
Contributor

Minor comment : can you pls move vendor/github.com/docker/swarmkit/node/node.go to the vendor commit ?

This is necessary for swarmkit to support cluster wide plugins, such as
globally scoped network plugins.

Signed-off-by: Anusha Ragunathan <[email protected]>
@anusha-ragunathan
Copy link
Contributor Author

@mavenugo : done.

Signed-off-by: Anusha Ragunathan <[email protected]>
@mavenugo
Copy link
Contributor

thanks @anusha-ragunathan .

LGTM

@tiborvass
Copy link
Contributor

code LGTM

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.

6 participants