Alexandros Kosiaris has submitted this change and it was merged.
Change subject: planet: Use the per site url-downloader instance
......................................................................
planet: Use the per site url-downloader instance
Although we don't have yet planet in codfw, this future proofs it
Change-Id: I15855e116957808633fff17b53df06dfdc838af5
---
M modules/role/manifests/planet/venus.pp
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Alexandros Kosiaris: Verified; Looks good to me, approved
diff --git a/modules/role/manifests/planet/venus.pp
b/modules/role/manifests/planet/venus.pp
index 3da821e..f251410 100644
--- a/modules/role/manifests/planet/venus.pp
+++ b/modules/role/manifests/planet/venus.pp
@@ -212,8 +212,8 @@
# protocol-relative link to a meta or index page for all planets
$planet_meta_link = 'meta.wikimedia.org/wiki/Planet_Wikimedia'
- # since we are on Ganeti now we need to proxy to fetch external URLs
- $planet_http_proxy = 'http://url-downloader.wikimedia.org:8080'
+ # since we are on a private IP now we need to proxy to fetch external URLs
+ $planet_http_proxy = "http://url-downloader.${::site}.wikimedia.org:8080";
# the 'planet' class from modules/planet/init.pp does the setup
class {'::planet':
--
To view, visit https://gerrit.wikimedia.org/r/287077
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I15855e116957808633fff17b53df06dfdc838af5
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
