Archlinux doesn't have a limits.d directory by default #84

Closed
evaryont opened this Issue Jul 12, 2015 · 1 comment

Comments

Projects
None yet
2 participants
@evaryont

/etc/security/limits.d doesn't exist by default. It works and is included when it does exist, but needs to be made before https://github.com/hardening-io/chef-os-hardening/blob/master/recipes/limits.rb#L21 is ran. A simple wrapper recipe works:

directory '/etc/security/limits.d' do
  owner 'root'
  group 'root'
  mode '0755'
end
include_recipe 'os-hardening::default'

chris-rock added a commit that referenced this issue Jul 13, 2015

rndmh3ro pushed a commit to dev-sec/ansible-os-hardening that referenced this issue Jul 13, 2015

@rndmh3ro rndmh3ro referenced this issue in dev-sec/ansible-os-hardening Jul 13, 2015

Merged

Create limits.d-directory if it does not exist. #33

@Rockstar04

This comment has been minimized.

Show comment
Hide comment
@Rockstar04

Rockstar04 Jul 14, 2015

Member

Looks like a fix for this was merged in, you can point Berkshelf to master if you would like to use these changes until we tag a new release.

Member

Rockstar04 commented Jul 14, 2015

Looks like a fix for this was merged in, you can point Berkshelf to master if you would like to use these changes until we tag a new release.

@Rockstar04 Rockstar04 closed this Jul 14, 2015

Rockstar04 added a commit that referenced this issue Jul 14, 2015

Merge pull request #85 from hardening-io/limit.d-directory
fixes #84

Tests passing, and already merged in Anisible playbook. This looks good to me!

If we want we can add test later

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

Merge pull request #84 from hardening-io/debian8test
add Debian 8  to local test-kitchen
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment