Dzahn has submitted this change and it was merged.

Change subject: swift/admin: put admin group in role/common/swift.yaml
......................................................................


swift/admin: put admin group in role/common/swift.yaml

Trying to put the admin group in ./common/ was wrong and
didnt work. We need ./role/common/ and we did not have one
common file here yet, it's split up into proxy/storage and
esams/eqiad. So add the new common file for all roles.

Bug:T130910
Change-Id: Idd99beecb55cc28b04e4073d626c05b8f7ab22f9
---
M hieradata/common/swift.yaml
A hieradata/role/common/swift.yaml
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  jenkins-bot: Verified
  Dzahn: Looks good to me, approved



diff --git a/hieradata/common/swift.yaml b/hieradata/common/swift.yaml
index 5efac28..a1d8f67 100644
--- a/hieradata/common/swift.yaml
+++ b/hieradata/common/swift.yaml
@@ -1,5 +1,3 @@
-admin::groups:
-  - swift-roots
 # used by ferm
 proxyhosts:
   - ms-fe1001.eqiad.wmnet
diff --git a/hieradata/role/common/swift.yaml b/hieradata/role/common/swift.yaml
new file mode 100644
index 0000000..5b3dfbc
--- /dev/null
+++ b/hieradata/role/common/swift.yaml
@@ -0,0 +1,2 @@
+admin::groups:
+  - swift-roots

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idd99beecb55cc28b04e4073d626c05b8f7ab22f9
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Filippo Giunchedi <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to