Mobrovac has uploaded a new change for review.

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

Change subject: Citoid: Fix the service URL properly
......................................................................

Citoid: Fix the service URL properly

I8183d8d8b4875bf3b50e7485f9b63781ff14db31 tried to fix the extension's
URL, but failed to do so. Instead of using MW variables, use the config
from hiera which is known to work.

Bug: T144027
Change-Id: Iec2104cc12df1ed37828b131c6d65a30184a1058
---
M puppet/hieradata/common.yaml
M puppet/modules/role/manifests/citoid.pp
2 files changed, 6 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/vagrant 
refs/changes/20/307320/1

diff --git a/puppet/hieradata/common.yaml b/puppet/hieradata/common.yaml
index 7c3d9fe..fbc6a85 100644
--- a/puppet/hieradata/common.yaml
+++ b/puppet/hieradata/common.yaml
@@ -26,8 +26,9 @@
 cdh::oozie::server::db_root_password: "%{hiera('mysql::root_password')}"
 cdh::oozie::server::heapsize: 64
 
-citoid::port: 1970
+citoid::port: '1970'
 citoid::log_level: info
+role::citoid::url: 
"//%{hiera('role::mediawiki::hostname')}:%{hiera('citoid::port')}/api"
 
 changeprop::port: 7272
 
diff --git a/puppet/modules/role/manifests/citoid.pp 
b/puppet/modules/role/manifests/citoid.pp
index 6b2bb43..de637f1 100644
--- a/puppet/modules/role/manifests/citoid.pp
+++ b/puppet/modules/role/manifests/citoid.pp
@@ -1,13 +1,15 @@
 # == Class: role::citoid
 # Provisions Citoid, a MediaWiki extension which adds an auto-
 # filled citation tool to VisualEditor using the citoid service.
-class role::citoid {
+class role::citoid(
+    $url,
+) {
     include ::role::zotero
     include ::citoid
 
     mediawiki::extension { 'Citoid':
         settings => {
-            wgCitoidServiceUrl => "//{\$wgServer}:${::citoid::port}/api"
+            wgCitoidServiceUrl => $url,
         }
     }
 }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iec2104cc12df1ed37828b131c6d65a30184a1058
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/vagrant
Gerrit-Branch: master
Gerrit-Owner: Mobrovac <mobro...@wikimedia.org>

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

Reply via email to