Tim Landscheidt has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/328451 )
Change subject: Tools: Fully qualify hostnames ...................................................................... Tools: Fully qualify hostnames This change fully qualifies some hostnames in the "flat" namespace beneath .eqiad.wmflabs. Hostnames should only refer to the project-specific instance, i. e. beneath .tools.eqiad.wmflabs. This is most obvious in the case of role::toollabs::services: Here the parameter $active_host was tested for equality with $::fqdn. However, due to $active_host being "tools-services-01.eqiad.wmflabs" and $::fqdn being "tools-services-01.tools.eqiad.wmflabs", this would have failed if $active_host had not been overwritten at https://wikitech.wikimedia.org/wiki/Hiera:Tools. An exception to this rule are modules/toollabs/files/host_aliases and modules/toollabs/templates/mail-relay.exim4.conf.erb that need to list legacy hostnames. Bug: T153608 Change-Id: I6888bb449bd59bf4c522e3b14d6a38dc2540decd --- M modules/role/manifests/toollabs/services.pp M modules/toollabs/files/exec-manage M modules/toollabs/manifests/init.pp 3 files changed, 3 insertions(+), 3 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/operations/puppet refs/changes/51/328451/1 diff --git a/modules/role/manifests/toollabs/services.pp b/modules/role/manifests/toollabs/services.pp index 7b76a1d..f2f44db 100644 --- a/modules/role/manifests/toollabs/services.pp +++ b/modules/role/manifests/toollabs/services.pp @@ -1,6 +1,6 @@ # filtertags: labs-project-tools class role::toollabs::services( - $active_host = 'tools-services-01.eqiad.wmflabs', + $active_host = 'tools-services-01.tools.eqiad.wmflabs', ) { system::role { 'role::toollabs::services': description => 'Tool Labs manifest based services', diff --git a/modules/toollabs/files/exec-manage b/modules/toollabs/files/exec-manage index bb82f32..8c221ce 100644 --- a/modules/toollabs/files/exec-manage +++ b/modules/toollabs/files/exec-manage @@ -11,7 +11,7 @@ echo "exec-manage [status|depool|repool|test] exec_host_name" echo "exec-manage [list]" echo "" - echo "Example: exec-manage status tools-exec1001.eqiad.wmflabs " + echo "Example: exec-manage status tools-exec1001.tools.eqiad.wmflabs" } if [[ ! "$1" || "$1" == '-h' ]]; then diff --git a/modules/toollabs/manifests/init.pp b/modules/toollabs/manifests/init.pp index a590802..cbb9fc4 100644 --- a/modules/toollabs/manifests/init.pp +++ b/modules/toollabs/manifests/init.pp @@ -4,7 +4,7 @@ $external_hostname = undef, $external_ip = undef, $is_mail_relay = false, - $active_mail_relay = 'tools-mail.eqiad.wmflabs', + $active_mail_relay = 'tools-mail.tools.eqiad.wmflabs', $mail_domain = 'tools.wmflabs.org', ) { -- To view, visit https://gerrit.wikimedia.org/r/328451 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I6888bb449bd59bf4c522e3b14d6a38dc2540decd Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Tim Landscheidt <t...@tim-landscheidt.de> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits