Packages #12

Merged
merged 4 commits into from May 13, 2014

Conversation

Projects
None yet
2 participants
@chris-rock
Member

chris-rock commented May 13, 2014

  • remove yum update
  • improve package documentation
@arlimus

This comment has been minimized.

Show comment
Hide comment
@arlimus

arlimus May 13, 2014

Member

Thank you for the update!

Could you please make the list of removed packages optional, i.e. add an option that defaults to false which removes a set of unnecessary packages from servers. Add the list of packages for removal to attributes to make them configurable.

The default set to false is necessary in order to avoid clashes for some users who may still use these packages. Let's offer it as an option.

Also we need some documentation on each package that is removed as to why it is removed (just referencing a source).

Member

arlimus commented May 13, 2014

Thank you for the update!

Could you please make the list of removed packages optional, i.e. add an option that defaults to false which removes a set of unnecessary packages from servers. Add the list of packages for removal to attributes to make them configurable.

The default set to false is necessary in order to avoid clashes for some users who may still use these packages. Let's offer it as an option.

Also we need some documentation on each package that is removed as to why it is removed (just referencing a source).

@arlimus

This comment has been minimized.

Show comment
Hide comment
@arlimus

arlimus May 13, 2014

Member

Great! Thanks for the adjustments as well 👍

Member

arlimus commented May 13, 2014

Great! Thanks for the adjustments as well 👍

arlimus added a commit that referenced this pull request May 13, 2014

Merge pull request #12 from TelekomLabs/packages
* feature: remove some dangerous packages by default (option added to make it configurable)
* udpate readme with information of package removal and internals

@arlimus arlimus merged commit 306e72c into master May 13, 2014

@arlimus arlimus deleted the packages branch May 13, 2014

@chris-rock chris-rock restored the packages branch Sep 16, 2014

@chris-rock chris-rock deleted the packages branch Sep 16, 2014

@rndmh3ro rndmh3ro referenced this pull request in dev-sec/ansible-os-hardening Jun 24, 2015

Merged

Update readme, todo, changelog, vars #30

@chris-rock chris-rock referenced this pull request in dev-sec/linux-baseline Jun 24, 2015

Open

Test verification for package removal #32

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

Merge pull request #12 from ehaselwanter/master
use the role from the integration test suite, not distinct recipes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment