Skip to content
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

Move sysctl vars to defaults #67

Merged
merged 2 commits into from
Feb 24, 2016
Merged

Move sysctl vars to defaults #67

merged 2 commits into from
Feb 24, 2016

Conversation

rndmh3ro
Copy link
Member

see #50. This adds the ability to overwrite the sysctl-dict.

@conorsch
Copy link
Contributor

You'll also need to remove the include_vars: sysctl.yml task from tasks/sysctl.yml, since the vars file doesn't exist anymore.

@rndmh3ro
Copy link
Member Author

rndmh3ro commented Feb 1, 2016

Thanks @conorsch, for pointing this out!

@conorsch
Copy link
Contributor

conorsch commented Feb 1, 2016

Works great for me under Ansible v2 with the change—thanks, @rndmh3ro. +1 for merge

@conorsch
Copy link
Contributor

@rndmh3ro Anything blocking merge here?

@chris-rock chris-rock force-pushed the mv_sysctl_to_defaults_2 branch from c8961cf to 47f3dc0 Compare February 24, 2016 06:41
@chris-rock chris-rock force-pushed the mv_sysctl_to_defaults_2 branch from 47f3dc0 to e9eac79 Compare February 24, 2016 07:08
@chris-rock
Copy link
Member

That makes live much easier @rndmh3ro

chris-rock added a commit that referenced this pull request Feb 24, 2016
@chris-rock chris-rock merged commit 5cc477b into master Feb 24, 2016
@chris-rock chris-rock deleted the mv_sysctl_to_defaults_2 branch February 24, 2016 07:13
@conorsch
Copy link
Contributor

Magnificent, thanks for merging!

@conorsch conorsch mentioned this pull request Feb 24, 2016
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.

3 participants