remove users that exists but don't have access to the node's environment

rubygems · Mar 14, 2015 · 1cda061 · 1cda061
1 parent 5d8ffa6
commit 1cda061
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/cookbooks/rubygems-people/recipes/default.rb b/cookbooks/rubygems-people/recipes/default.rb
@@ -13,9 +13,8 @@
 users.each do |user_name|
   user = data_bag_item('users', user_name)
 
-  next unless user['environments'].include?(node.chef_environment)
-
-  if user['action'] && user['action'] == 'remove'
+  if (user['action'] && user['action'] == 'remove') || \
+   (!user['environments'].include?(node.chef_environment))
     user_account user['username'] do
       action :remove
       only_if "/usr/bin/id -u #{user['username']}"