Dzahn has submitted this change and it was merged.

Change subject: rename ypanda to yuvipanda
......................................................................


rename ypanda to yuvipanda

the (LDAP) user name should
match the class name or we
always run into issues.

like f.e. Ic0147a4ebc
or 'Could not find class accounts::yuvipanda'

RT #7068

Change-Id: Ia8c1b258854741d540b42839bfd37d8b0ca5823d
---
M manifests/admins.pp
M manifests/role/analytics.pp
M manifests/site.pp
3 files changed, 5 insertions(+), 5 deletions(-)

Approvals:
  Yuvipanda: Looks good to me, but someone else must approve
  jenkins-bot: Verified
  Dzahn: Looks good to me, approved



diff --git a/manifests/admins.pp b/manifests/admins.pp
index b4c676e..cc2dc96 100644
--- a/manifests/admins.pp
+++ b/manifests/admins.pp
@@ -2807,7 +2807,7 @@
     }
 
     # RT 4519
-    class ypanda inherits baseaccount {
+    class yuvipanda inherits baseaccount {
         $username = 'yuvipanda'
         $realname = 'Yuvi Panda'
         $uid      = '630'
diff --git a/manifests/role/analytics.pp b/manifests/role/analytics.pp
index 4316757..8d106c3 100644
--- a/manifests/role/analytics.pp
+++ b/manifests/role/analytics.pp
@@ -62,7 +62,7 @@
         accounts::ironholds, # RT 5831
         accounts::dartar,    # RT 5835
         accounts::halfak,    # RT 5836
-        accounts::ypanda,    # RT 6103
+        accounts::yuvipanda, # RT 6103
         accounts::csalvia,   # RT 6664
         accounts::nuria,     # RT 6683
         accounts::sahar      # RT 6767
diff --git a/manifests/site.pp b/manifests/site.pp
index bda08ef..3c7e884 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -294,7 +294,7 @@
     include accounts::maryana
     include accounts::msyed
     include accounts::swalling
-    include accounts::ypanda
+    include accounts::yuvipanda
     include accounts::haithams
 }
 
@@ -2202,7 +2202,7 @@
     include accounts::dsc
     include accounts::milimetric
     include accounts::rfaulk   # RT 4258
-    include accounts::ypanda   # RT 4687
+    include accounts::yuvipanda   # RT 4687
     include accounts::qchris   # RT 5474
     include accounts::tnegrin  # RT 5391
 
@@ -2297,7 +2297,7 @@
     include accounts::leila        # RT 6765
     include accounts::gdubuc       # RT 7074
     include accounts::haithams     # RT 7387
-    include accounts::ypanda
+    include accounts::yuvipanda
 
     # Allow Christian to sudo -u stats
     # to debug and test stats' automated cron jobs.

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia8c1b258854741d540b42839bfd37d8b0ca5823d
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: ArielGlenn <ar...@wikimedia.org>
Gerrit-Reviewer: Dzahn <dz...@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

Reply via email to