Skip to content

[17.11] Don't filter nodes if logdriver==none#2426

Closed
nishanttotla wants to merge 1 commit into
moby:bump_v17.11from
nishanttotla:dont-filter-nodes-if-logdriver-none-17.11
Closed

[17.11] Don't filter nodes if logdriver==none#2426
nishanttotla wants to merge 1 commit into
moby:bump_v17.11from
nishanttotla:dont-filter-nodes-if-logdriver-none-17.11

Conversation

@nishanttotla
Copy link
Copy Markdown
Contributor

Cherry-pick #2396.

git cherry-pick -s -x 9608c63711181bff3a33c71f763aedde2d3596ec

Cherry-pick was clean.

Signed-off-by: Brian Goff cpuguy83@gmail.com
(cherry picked from commit 9608c63)
Signed-off-by: Nishant Totla nishanttotla@gmail.com

The "none" driver is a special keyword to disable logging, so don't
filter nodes if the "none" driver is not listed on the nodes, since it's
not really a driver and doesn't exist.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
(cherry picked from commit 9608c63)
Signed-off-by: Nishant Totla <nishanttotla@gmail.com>
@nishanttotla
Copy link
Copy Markdown
Contributor Author

nishanttotla commented Nov 3, 2017

@nishanttotla
Copy link
Copy Markdown
Contributor Author

OK CI failure seems related, I'll take a look.

@nishanttotla
Copy link
Copy Markdown
Contributor Author

Closing PR because 17.11 branch is now cut off at 312be59, which includes this commit.

@nishanttotla nishanttotla deleted the dont-filter-nodes-if-logdriver-none-17.11 branch November 3, 2017 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants