Skip to content

Conversation

@fcrisciani
Copy link

The feature was not getting properly triggered, move it as
first operation in the configure

Signed-off-by: Flavio Crisciani flavio.crisciani@docker.com

The feature was not getting properly triggered, move it as
first operation in the configure

Signed-off-by: Flavio Crisciani <flavio.crisciani@docker.com>
@mavenugo
Copy link
Contributor

mavenugo commented Jun 7, 2017

Makes sense. if vxlanIDM is nil, we it will initialize it and return prior to setting the OSTweaks.
Moving it to top of the function is correct.

LGTM

@mavenugo mavenugo merged commit eb57059 into moby:master Jun 7, 2017
@fcrisciani fcrisciani deleted the sysctl_gc_thresh branch June 7, 2017 04:33
@fcrisciani
Copy link
Author

Follow up for #1789

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.

2 participants