Dzahn has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/404063 )
Change subject: parsoid::testing: convert role to profile ...................................................................... parsoid::testing: convert role to profile Change-Id: Ic1ed5e6719e1ec2c5cf017a8667b8fd3fedd076e --- A modules/profile/manifests/parsoid/testing.pp M modules/role/manifests/parsoid/testing.pp 2 files changed, 63 insertions(+), 59 deletions(-) Approvals: jenkins-bot: Verified Arlolra: Looks good to me, but someone else must approve Dzahn: Looks good to me, approved diff --git a/modules/profile/manifests/parsoid/testing.pp b/modules/profile/manifests/parsoid/testing.pp new file mode 100644 index 0000000..a7a3eb1 --- /dev/null +++ b/modules/profile/manifests/parsoid/testing.pp @@ -0,0 +1,62 @@ +# Parsoid roundtrip testing, Parsoid & PHP parser visual diff testing +class profile::parsoid::testing ( + $parsoid_port = hiera('parsoid::testing::parsoid_port'), +) { + class { '::parsoid': + port => $parsoid_port, + settings_file => '/srv/deployment/parsoid/deploy/src/localsettings.js', + deployment => 'git', + } + + file { '/usr/local/bin/update_parsoid.sh': + source => 'puppet:///modules/parsoid/parsoid_testing.update_parsoid.sh', + owner => 'root', + group => 'root', + mode => '0555', + } + + # Use this parsoid instance for parsoid rt-testing + file { '/srv/deployment/parsoid/deploy/src/localsettings.js': + content => template('testreduce/parsoid-rt-client.rttest.localsettings.js.erb'), + owner => 'root', + group => 'wikidev', + mode => '0444', + before => Service['parsoid'], + } + + # mysql client and configuration to provide command line access to + # parsoid testing database + include ::passwords::testreduce::mysql + $parsoid_cli_password = $passwords::testreduce::mysql::mysql_client_pass + $parsoid_test_db_host = 'm5-master.eqiad.wmnet' + + package { [ + 'mysql-client', + ]: ensure => present, + } + + file { '/etc/my.cnf': + content => template('role/mariadb/mysqld_config/parsoid_testing.my.cnf'), + owner => 'root', + group => 'parsoid-test-roots', + mode => '0440', + } + + nginx::site { 'nginx-parsoid-testing': + content => template('parsoid/parsoid-testing.nginx.conf.erb'), + notify => Service['nginx'], + } + + ferm::service { 'nginx-parsoid-testing': + proto => 'tcp', + port => 8001, + srange => '$PRODUCTION_NETWORKS', + } + + # Presented by the @remote links shown on parsoid-rt-tests.wikimedia.org + ferm::service { 'parsoid-testing': + proto => 'tcp', + port => 8142, + srange => '$PRODUCTION_NETWORKS', + } +} diff --git a/modules/role/manifests/parsoid/testing.pp b/modules/role/manifests/parsoid/testing.pp index ec686c4..a2825b8 100644 --- a/modules/role/manifests/parsoid/testing.pp +++ b/modules/role/manifests/parsoid/testing.pp @@ -5,63 +5,5 @@ description => 'Parsoid server (rt-testing, visual-diffing, etc.)' } - $parsoid_port = hiera('parsoid::testing::parsoid_port') - - class { '::parsoid': - port => $parsoid_port, - settings_file => '/srv/deployment/parsoid/deploy/src/localsettings.js', - deployment => 'git', - } - - file { '/usr/local/bin/update_parsoid.sh': - source => 'puppet:///modules/parsoid/parsoid_testing.update_parsoid.sh', - owner => 'root', - group => 'root', - mode => '0555', - } - - # Use this parsoid instance for parsoid rt-testing - file { '/srv/deployment/parsoid/deploy/src/localsettings.js': - content => template('testreduce/parsoid-rt-client.rttest.localsettings.js.erb'), - owner => 'root', - group => 'wikidev', - mode => '0444', - before => Service['parsoid'], - } - - # mysql client and configuration to provide command line access to - # parsoid testing database - include ::passwords::testreduce::mysql - $parsoid_cli_password = $passwords::testreduce::mysql::mysql_client_pass - $parsoid_test_db_host = 'm5-master.eqiad.wmnet' - - package { [ - 'mysql-client', - ]: ensure => present, - } - - file { '/etc/my.cnf': - content => template('role/mariadb/mysqld_config/parsoid_testing.my.cnf'), - owner => 'root', - group => 'parsoid-test-roots', - mode => '0440', - } - - nginx::site { 'nginx-parsoid-testing': - content => template('parsoid/parsoid-testing.nginx.conf.erb'), - notify => Service['nginx'], - } - - ferm::service { 'nginx-parsoid-testing': - proto => 'tcp', - port => 8001, - srange => '$PRODUCTION_NETWORKS', - } - - # Presented by the @remote links shown on parsoid-rt-tests.wikimedia.org - ferm::service { 'parsoid-testing': - proto => 'tcp', - port => 8142, - srange => '$PRODUCTION_NETWORKS', - } + include ::profile::parsoid::testing } -- To view, visit https://gerrit.wikimedia.org/r/404063 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic1ed5e6719e1ec2c5cf017a8667b8fd3fedd076e Gerrit-PatchSet: 4 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Dzahn <dz...@wikimedia.org> Gerrit-Reviewer: Arlolra <abrea...@wikimedia.org> Gerrit-Reviewer: Dzahn <dz...@wikimedia.org> Gerrit-Reviewer: Giuseppe Lavagetto <glavage...@wikimedia.org> Gerrit-Reviewer: Mobrovac <mobro...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits