Andrew Bogott has submitted this change and it was merged.

Change subject: Puppet masters in labs needs puppet::self::geoip
......................................................................


Puppet masters in labs needs puppet::self::geoip

Puppet dies with:

err: /Stage[main]/Geoip::Data::Puppet/File[/usr/share/GeoIP]: Could not
evaluate: Could not retrieve information from environment production
source(s) puppet:///volatile/GeoIP at
/etc/puppet/modules/geoip/manifests/data/puppet.pp:22

Applying puppet::self::geoip fix the problem.

Follow up f13f911e1f / https://gerrit.wikimedia.org/r/#/c/119555/

Change-Id: I7561f649440871c06ac1c92144a3f1ba5c9557a1
---
M modules/puppet/manifests/self/master.pp
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Andrew Bogott: Looks good to me, approved
  Hashar: Verified; Looks good to me, but someone else must approve
  jenkins-bot: Verified



diff --git a/modules/puppet/manifests/self/master.pp 
b/modules/puppet/manifests/self/master.pp
index 9fcc753..dc1fecf 100644
--- a/modules/puppet/manifests/self/master.pp
+++ b/modules/puppet/manifests/self/master.pp
@@ -20,6 +20,8 @@
         }
     }
 
+    include puppet::self::geoip
+
     # If localhost, only bind to loopback.
     $bindaddress = $server ? {
         'localhost' => '127.0.0.1',

-- 
To view, visit https://gerrit.wikimedia.org/r/121677
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7561f649440871c06ac1c92144a3f1ba5c9557a1
Gerrit-PatchSet: 5
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar <[email protected]>
Gerrit-Reviewer: Andrew Bogott <[email protected]>
Gerrit-Reviewer: BryanDavis <[email protected]>
Gerrit-Reviewer: Hashar <[email protected]>
Gerrit-Reviewer: Ottomata <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to