Yuvipanda has submitted this change and it was merged. Change subject: Trebuchet group wikidev; mw-staging owner mwdeploy ......................................................................
Trebuchet group wikidev; mw-staging owner mwdeploy Change ownership of files on {tin,deployment-bastion} to correct recent permissions problems that arose from combining role::deployment::server::{production,labs} into role::deployment::server (I3e947637b49ce2a94128e21db35798a49e8d45e8) Bug: T94054, T94261 Change-Id: I0b391b1c50857794d1526edc2c80e3c87b67ef8c --- M hieradata/labs/deployment-prep/common.yaml M hieradata/labs/staging/common.yaml M manifests/role/deployment.pp M modules/scap/manifests/master.pp 4 files changed, 2 insertions(+), 7 deletions(-) Approvals: Yuvipanda: Verified; Looks good to me, approved diff --git a/hieradata/labs/deployment-prep/common.yaml b/hieradata/labs/deployment-prep/common.yaml index 9c2b300..d6fd243 100644 --- a/hieradata/labs/deployment-prep/common.yaml +++ b/hieradata/labs/deployment-prep/common.yaml @@ -182,6 +182,5 @@ "role::url_downloader::url_downloader_ip": 10.68.16.135 "zotero::http_proxy": deployment-urldownloader.eqiad.wmflabs:8080 "role::trebuchet::deployment_server": deployment-bastion.eqiad.wmflabs -"role::deployment::server::deployment_group": 'project-deployment-prep' "dsh::config::group_source": 'puppet:///modules/beta/dsh/group' "mediawiki::users::mwdeploy_pub_key": 'ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQDFwlmBBBJAr1GI+vuYjFh5vq0YIVa5fqE5DZdpzUZISlQ0Kt+9bIr2qNHIj+Jl5Bc6ZY1mkh8l693tAHVx+8tayoiFWYNs9IVsxR+iHgOOhAdDIBXaHaUattdiye5bQmdvJVXaVegckNX2gbmUCOc09jvZvlk3blKFTSEpZRU8dmpXQzKdZgaAq2VTajAegoFnuN9FbC7hzBPA+1NxFNKn94eIeFPSlo5rWr44OEb5Uy3O0B5c6WPM+IgfiygetP+yGL4cKv7qEjZ0Sxok/Rh1lBh1vP1YQ/Mc6tMV0s+kOv7Wz+P88bfU1/uWvy479OZdfh3NQqDTrLzqHwVW1vef root@deployment-salt' diff --git a/hieradata/labs/staging/common.yaml b/hieradata/labs/staging/common.yaml index 51e4f1e..41d510e 100644 --- a/hieradata/labs/staging/common.yaml +++ b/hieradata/labs/staging/common.yaml @@ -6,6 +6,5 @@ salt::master::salt_pillar_roots: { base: [ '/srv/pillars' ] } salt::master::salt_module_roots: { base: [ '/srv/salt/_modules' ] } salt::master::salt_returner_roots: { base: [ '/srv/salt/_returners' ] } -role::deployment::server::deployment_group: 'project-staging' role::deployment::salt_masters::deployment_server: staging-tin.eqiad.wmflabs role::trebuchet::deployment_server: staging-tin.eqiad.wmflabs diff --git a/manifests/role/deployment.pp b/manifests/role/deployment.pp index e9c5d8c..636cd95 100644 --- a/manifests/role/deployment.pp +++ b/manifests/role/deployment.pp @@ -156,10 +156,7 @@ $deployable_networks = $::network::constants::deployable_networks include mediawiki - - class { 'scap::master': - deployment_group => $deployment_group, - } + include scap::master if $::realm != 'labs' { include wikitech::wiki::passwords diff --git a/modules/scap/manifests/master.pp b/modules/scap/manifests/master.pp index 51720f1..40ca837 100644 --- a/modules/scap/manifests/master.pp +++ b/modules/scap/manifests/master.pp @@ -1,6 +1,6 @@ # = class: scap::master # -# Sets up a scap master (currently tin). +# Sets up a scap master (currently tin) class scap::master( $common_path = '/srv/mediawiki', $common_source_path = '/srv/mediawiki-staging', -- To view, visit https://gerrit.wikimedia.org/r/199988 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0b391b1c50857794d1526edc2c80e3c87b67ef8c Gerrit-PatchSet: 3 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Thcipriani <tcipri...@wikimedia.org> Gerrit-Reviewer: BryanDavis <bda...@wikimedia.org> Gerrit-Reviewer: Hashar <has...@free.fr> Gerrit-Reviewer: Thcipriani <tcipri...@wikimedia.org> Gerrit-Reviewer: Yuvipanda <yuvipa...@gmail.com> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits