Ema has submitted this change and it was merged. Change subject: Fix multiple definition of /var/lib/puppet ......................................................................
Fix multiple definition of /var/lib/puppet We introduced a definition of File[/var/lib/puppet] in https://gerrit.wikimedia.org/r/#/c/268684/. However, File[/var/lib/puppet] was already defined in puppetmaster::ssl. This patch introduces a new class called base::puppet::common which is then included by base::puppet as well as puppetmaster::ssl to fix the issue. Change-Id: Ia6f17457bccf33bb7b155c619bfc08f990899f71 --- M modules/base/manifests/puppet.pp A modules/base/manifests/puppet/common.pp M modules/puppetmaster/manifests/ssl.pp 3 files changed, 13 insertions(+), 9 deletions(-) Approvals: Ema: Verified; Looks good to me, approved ArielGlenn: Looks good to me, but someone else must approve jenkins-bot: Verified diff --git a/modules/base/manifests/puppet.pp b/modules/base/manifests/puppet.pp index 0458c4e..3639da5 100644 --- a/modules/base/manifests/puppet.pp +++ b/modules/base/manifests/puppet.pp @@ -90,14 +90,7 @@ source => 'puppet:///modules/base/logrotate/puppet', } - # Mode 0751 to make sure non-root users can access - # /var/lib/puppet/state/agent_disabled.lock to check if puppet is enabled - file { '/var/lib/puppet': - ensure => directory, - owner => 'puppet', - group => 'puppet', - mode => '0751', - } + include ::base::puppet::common file { '/usr/local/bin/puppet-enabled': mode => '0555', diff --git a/modules/base/manifests/puppet/common.pp b/modules/base/manifests/puppet/common.pp new file mode 100644 index 0000000..c8ea912 --- /dev/null +++ b/modules/base/manifests/puppet/common.pp @@ -0,0 +1,10 @@ +class base::puppet::common { + # Mode 0751 to make sure non-root users can access + # /var/lib/puppet/state/agent_disabled.lock to check if puppet is enabled + file { '/var/lib/puppet': + ensure => directory, + owner => 'puppet', + group => 'puppet', + mode => '0751', + } +} diff --git a/modules/puppetmaster/manifests/ssl.pp b/modules/puppetmaster/manifests/ssl.pp index ea9086f..a8bfff0 100644 --- a/modules/puppetmaster/manifests/ssl.pp +++ b/modules/puppetmaster/manifests/ssl.pp @@ -14,6 +14,8 @@ $before = undef } + include ::base::puppet::common + # Move the puppetmaster's SSL files to a separate directory from the client file { [ '/var/lib/puppet/server', @@ -25,7 +27,6 @@ mode => '0771', before => $before; [ - '/var/lib/puppet', "${ssldir}/ca", "${ssldir}/certificate_requests", "${ssldir}/certs", -- To view, visit https://gerrit.wikimedia.org/r/270263 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia6f17457bccf33bb7b155c619bfc08f990899f71 Gerrit-PatchSet: 3 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Ema <e...@wikimedia.org> Gerrit-Reviewer: ArielGlenn <ar...@wikimedia.org> Gerrit-Reviewer: Ema <e...@wikimedia.org> Gerrit-Reviewer: Giuseppe Lavagetto <glavage...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits