Merge pull request #831 from /issues/548

manage homedirectory for prometheus user when manage_user is true
voxpupuli · Dec 30, 2024 · 76b9e76 · 76b9e76
2 parents a5ccb71 + 4a05371
commit 76b9e76
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/manifests/install.pp b/manifests/install.pp
@@ -79,6 +79,8 @@
         system => true,
         groups => $prometheus::server::extra_groups,
         shell  => $prometheus::server::usershell,
+        managehome => true,
+        home       => $prometheus::server::localstorage,
     })
 
     if $prometheus::server::manage_group {

diff --git a/spec/classes/prometheus_spec.rb b/spec/classes/prometheus_spec.rb
@@ -28,6 +28,18 @@
           it { is_expected.to contain_class('prometheus::server') }
           it { is_expected.to contain_class('prometheus::service_reload') }
 
+          if facts[:os]['name'] != 'Archlinux'
+            it {
+              is_expected.to contain_user('prometheus').with(
+                'ensure' => 'present',
+                'system' => true,
+                'groups' => [],
+                'managehome' => true,
+                'home' => '/var/lib/prometheus'
+              )
+            }
+          end
+
           if facts[:os]['name'] == 'Archlinux'
             it { expect(subject).not_to contain_file('/var/lib/prometheus') }
           else