[MediaWiki-commits] [Gerrit] MobileApps: Change RESTBase URI - change (operations/puppet)

2016-02-03 Thread Ema (Code Review) 
Ema has submitted this change and it was merged.

Change subject: MobileApps: Change RESTBase URI
..


MobileApps: Change RESTBase URI

The Mobile Content Service has changed the way they include the RESTBase
URI to a more flexible scheme that allows them to use both RESTBase's
internal URI, as well as the generic https://{domain}/api/rest_v1
external URI. This patch conforms to the new standard.

Note: to be deployed in conjunction with
I6d703b627fc22a0bfe7c27b32efa54ef0e10b5cf .

Change-Id: I4523d97598ef10d2075bd5ec9f60ba8bf5acfef4
Depends-On: I6d703b627fc22a0bfe7c27b32efa54ef0e10b5cf
Bug: T125252
---
M modules/mobileapps/manifests/init.pp
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Ema: Verified; Looks good to me, approved



diff --git a/modules/mobileapps/manifests/init.pp 
b/modules/mobileapps/manifests/init.pp
index 1bf95db..f651702 100644
--- a/modules/mobileapps/manifests/init.pp
+++ b/modules/mobileapps/manifests/init.pp
@@ -14,7 +14,7 @@
 #   The host/IP where to reach RESTBase
 #
 class mobileapps(
-$restbase_uri = 'http://restbase.svc.eqiad.wmnet:7231',
+$restbase_uri = 'http://restbase.svc.eqiad.wmnet:7231/%DOMAIN%/v1',
 ) {
 service::node { 'mobileapps':
 port=> ,

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4523d97598ef10d2075bd5ec9f60ba8bf5acfef4
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Mobrovac 
Gerrit-Reviewer: BearND 
Gerrit-Reviewer: Ema 
Gerrit-Reviewer: Giuseppe Lavagetto 
Gerrit-Reviewer: Mholloway 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

[MediaWiki-commits] [Gerrit] MobileApps: Change RESTBase URI - change (operations/puppet)

2016-01-29 Thread Mobrovac (Code Review) 
Mobrovac has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/267392

Change subject: MobileApps: Change RESTBase URI
..

MobileApps: Change RESTBase URI

The Mobile Content Service has changed the way they include the RESTBase
URI to a more flexible scheme that allows them to use both RESTBase's
internal URI, as well as the generic https://{domain}/api/rest_v1
external URI. This patch conforms to the new standard.

Note: to be deployed in conjunction with
I6d703b627fc22a0bfe7c27b32efa54ef0e10b5cf .

Change-Id: I4523d97598ef10d2075bd5ec9f60ba8bf5acfef4
Depends-On: I6d703b627fc22a0bfe7c27b32efa54ef0e10b5cf
Bug: T125252
---
M modules/mobileapps/manifests/init.pp
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/92/267392/1

diff --git a/modules/mobileapps/manifests/init.pp 
b/modules/mobileapps/manifests/init.pp
index 1bf95db..f651702 100644
--- a/modules/mobileapps/manifests/init.pp
+++ b/modules/mobileapps/manifests/init.pp
@@ -14,7 +14,7 @@
 #   The host/IP where to reach RESTBase
 #
 class mobileapps(
-$restbase_uri = 'http://restbase.svc.eqiad.wmnet:7231',
+$restbase_uri = 'http://restbase.svc.eqiad.wmnet:7231/%DOMAIN%/v1',
 ) {
 service::node { 'mobileapps':
 port=> ,

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4523d97598ef10d2075bd5ec9f60ba8bf5acfef4
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Mobrovac 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits