Yuvipanda has uploaded a new change for review. https://gerrit.wikimedia.org/r/322224
Change subject: labstore: Add account_services to secondary too ...................................................................... labstore: Add account_services to secondary too Since labstore1004 is technically secondary rn Change-Id: Ia13e8dbc2977e57032e2e678c97fa8c35b58860e --- M modules/labstore/manifests/account_services.pp M modules/role/manifests/labs/nfs/secondary.pp 2 files changed, 4 insertions(+), 16 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/operations/puppet refs/changes/24/322224/1 diff --git a/modules/labstore/manifests/account_services.pp b/modules/labstore/manifests/account_services.pp index ba21125..082dfcc 100644 --- a/modules/labstore/manifests/account_services.pp +++ b/modules/labstore/manifests/account_services.pp @@ -27,10 +27,6 @@ ensure => present, } - # Set to true only for the labstore that is currently - # actively serving files - $is_active = (hiera('active_labstore_host') == $::hostname) - $ldapconfig = hiera_hash('labsldapconfig', {}) include passwords::mysql::labsdb @@ -77,21 +73,12 @@ mode => '0550', } - # Terrible hack - if $is_active { - $service_ensure = 'present' - } else { - $service_ensure = 'absent' - } - base::service_unit { 'create-dbusers': - ensure => $service_ensure, + ensure => present, systemd => true, } - if $is_active { - nrpe::monitor_systemd_unit_state { 'create-dbusers': - description => 'Ensure mysql credential creation for tools users is running', - } + nrpe::monitor_systemd_unit_state { 'create-dbusers': + description => 'Ensure mysql credential creation for tools users is running', } } diff --git a/modules/role/manifests/labs/nfs/secondary.pp b/modules/role/manifests/labs/nfs/secondary.pp index 6bc2a7b..9aad6c5 100644 --- a/modules/role/manifests/labs/nfs/secondary.pp +++ b/modules/role/manifests/labs/nfs/secondary.pp @@ -7,6 +7,7 @@ include labstore::fileserver::exports include labstore::fileserver::secondary include labstore::backup_keys + include labstore::account_services # Enable RPS to balance IRQs over CPUs interface::rps { $monitor: } -- To view, visit https://gerrit.wikimedia.org/r/322224 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ia13e8dbc2977e57032e2e678c97fa8c35b58860e Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Yuvipanda <yuvipa...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits