Make the procs by platform

boxcutter · Sep 26, 2024 · 11b49b5 · 11b49b5
1 parent 2ba0fbd
commit 11b49b5
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 17 deletions.
diff --git a/cookbooks/boxcutter_site_settings/recipes/users.rb b/cookbooks/boxcutter_site_settings/recipes/users.rb
@@ -41,19 +41,22 @@
   FB::Users.initialize_group(node, group)
 end
 
-node.default['fb_users']['users']['nobody'] = {
-  'shell' => '/usr/sbin/nologin',
-  'gid' => 'nogroup',
-  'home' => '/nonexistent',
-  'action' => :add,
-  'only_if' => proc { node.ubuntu? }
-}
-
-node.default['fb_users']['users']['nobody'] = {
-  'comment' => 'Kernel Overflow User',
-  'shell' => '/usr/sbin/nologin',
-  'gid' => '65534',
-  'home' => '/',
-  'action' => :add,
-  'only_if' => proc { node.centos? }
-}
+case node['platform']
+when 'ubuntu'
+  node.default['fb_users']['users']['nobody'] = {
+    'shell' => '/usr/sbin/nologin',
+    'gid' => 'nogroup',
+    'home' => '/nonexistent',
+    'action' => :add,
+    'only_if' => proc { node.ubuntu? },
+  }
+when 'centos'
+  node.default['fb_users']['users']['nobody'] = {
+    'comment' => 'Kernel Overflow User',
+    'shell' => '/usr/sbin/nologin',
+    'gid' => '65534',
+    'home' => '/',
+    'action' => :add,
+    'only_if' => proc { node.centos? },
+  }
+end
diff --git a/cookbooks/boxcutter_users/libraries/default.rb b/cookbooks/boxcutter_users/libraries/default.rb
@@ -154,6 +154,6 @@ class Users
       'nogroup' => {
         'gid' => 65534,
       },
-    }
+    }.freeze
   end
 end