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

remove ntp #20

Merged
merged 1 commit into from May 19, 2014

Conversation

Projects
None yet
2 participants
@arlimus
Member

arlimus commented May 19, 2014

while i personally approve the idea that NTP must be on every machine, i would prefer to have it done separately and with hardening options. it's currently a separate module at opscode and i think we should align with this strategy to not meld it too deeply into os-hardening. opinions?

Solves #19

Signed-off-by: Dominik Richter dominik.richter@gmail.com

remove ntp
while i personally approve the idea that NTP must be on every machine, i would prefer to have it done separately and with hardening options. it's currently a separate module at opscode and i think we should align with this strategy to not meld it too deeply into os-hardening. opinions?

Signed-off-by: Dominik Richter <dominik.richter@gmail.com>
@chris-rock

This comment has been minimized.

Show comment
Hide comment
@chris-rock

chris-rock May 19, 2014

Member

I agree, that we should minimize the dependencies. Let's integrate this in our chef and puppet example kitchen to keep the best practice.

Sent from my iPhone

On 19.05.2014, at 17:02, Dominik Richter notifications@github.com wrote:

while i personally approve the idea that NTP must be on every machine, i would prefer to have it done separately and with hardening options. it's currently a separate module at opscode and i think we should align with this strategy to not meld it too deeply into os-hardening. opinions?

Solves #19

Signed-off-by: Dominik Richter dominik.richter@gmail.com

You can merge this Pull Request by running

git pull https://github.com/TelekomLabs/chef-os-hardening remove_ntp
Or view, comment on, or merge it at:

#20

Commit Summary

remove ntp
File Changes

M README.md (1)
M TUTORIAL.md (1)
M metadata.rb (1)
M recipes/packages.rb (3)
Patch Links:

https://github.com/TelekomLabs/chef-os-hardening/pull/20.patch
https://github.com/TelekomLabs/chef-os-hardening/pull/20.diff

Reply to this email directly or view it on GitHub.

Member

chris-rock commented May 19, 2014

I agree, that we should minimize the dependencies. Let's integrate this in our chef and puppet example kitchen to keep the best practice.

Sent from my iPhone

On 19.05.2014, at 17:02, Dominik Richter notifications@github.com wrote:

while i personally approve the idea that NTP must be on every machine, i would prefer to have it done separately and with hardening options. it's currently a separate module at opscode and i think we should align with this strategy to not meld it too deeply into os-hardening. opinions?

Solves #19

Signed-off-by: Dominik Richter dominik.richter@gmail.com

You can merge this Pull Request by running

git pull https://github.com/TelekomLabs/chef-os-hardening remove_ntp
Or view, comment on, or merge it at:

#20

Commit Summary

remove ntp
File Changes

M README.md (1)
M TUTORIAL.md (1)
M metadata.rb (1)
M recipes/packages.rb (3)
Patch Links:

https://github.com/TelekomLabs/chef-os-hardening/pull/20.patch
https://github.com/TelekomLabs/chef-os-hardening/pull/20.diff

Reply to this email directly or view it on GitHub.

chris-rock added a commit that referenced this pull request May 19, 2014

@chris-rock chris-rock merged commit c262e01 into master May 19, 2014

@chris-rock chris-rock deleted the remove_ntp branch May 19, 2014

@chris-rock

This comment has been minimized.

Show comment
Hide comment
@chris-rock

chris-rock May 19, 2014

Member

@arlimus thank you very much

Member

chris-rock commented May 19, 2014

@arlimus thank you very much

rollbrettler pushed a commit to rollbrettler/chef-os-hardening that referenced this pull request Sep 16, 2016

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