From 4a5d10bf0ec8432f712a5df17ad1954750cdef92 Mon Sep 17 00:00:00 2001 From: Steven Schlegel Date: Fri, 13 Oct 2017 11:10:50 +0200 Subject: [PATCH 1/4] bugfix: manage homeDir through puppet + define homeDir as /home/nexus --- manifests/init.pp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/manifests/init.pp b/manifests/init.pp index fd90c5b..37faec0 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -90,11 +90,13 @@ ensure => present } + $my_nexus_home = "/home/${nexus_user}" user { $nexus_user: ensure => present, comment => 'Nexus User', gid => $nexus_group, - home => $nexus_root, + home => "$my_nexus_home", + managehome => true, shell => '/bin/sh', # required to start application via script. system => true, require => Group[$nexus_group] From 0c11f9a70660ab981160a7fb9bc0e5d30808b12e Mon Sep 17 00:00:00 2001 From: Steven Schlegel Date: Fri, 13 Oct 2017 11:26:13 +0200 Subject: [PATCH 2/4] Fix linting issues --- manifests/init.pp | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/manifests/init.pp b/manifests/init.pp index 37faec0..96d414e 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -92,14 +92,14 @@ $my_nexus_home = "/home/${nexus_user}" user { $nexus_user: - ensure => present, - comment => 'Nexus User', - gid => $nexus_group, - home => "$my_nexus_home", + ensure => present, + comment => 'Nexus User', + gid => $nexus_group, + home => $my_nexus_home, managehome => true, - shell => '/bin/sh', # required to start application via script. - system => true, - require => Group[$nexus_group] + shell => '/bin/sh', # required to start application via script. + system => true, + require => Group[$nexus_group] } } From 2527d830899bd9266fe17854f8c5642918158fd1 Mon Sep 17 00:00:00 2001 From: Steven Schlegel Date: Fri, 13 Oct 2017 12:02:13 +0200 Subject: [PATCH 3/4] bugfix: syntax fix for operand --- manifests/service.pp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/manifests/service.pp b/manifests/service.pp index fd07c2b..0d65d71 100644 --- a/manifests/service.pp +++ b/manifests/service.pp @@ -49,8 +49,8 @@ owner => 'root', group => 'root', content => template('nexus/nexus.systemd.erb'), - } -> - service { 'nexus': + } + -> service { 'nexus': ensure => running, name => 'nexus', enable => true, From 29cb1e6424ec27412639d064f6736661ae12485c Mon Sep 17 00:00:00 2001 From: Steven Schlegel Date: Fri, 13 Oct 2017 12:33:04 +0200 Subject: [PATCH 4/4] bugfix: ruby spec (nexus home) --- spec/classes/init_spec.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/spec/classes/init_spec.rb b/spec/classes/init_spec.rb index a2ac385..db7e35a 100644 --- a/spec/classes/init_spec.rb +++ b/spec/classes/init_spec.rb @@ -34,7 +34,7 @@ 'ensure' => 'present', 'comment' => 'Nexus User', 'gid' => 'nexus', - 'home' => '/srv', + 'home' => '/home/nexus', 'shell' => '/bin/sh', 'system' => true, 'require' => 'Group[nexus]',