Subramanya Sastry has uploaded a new change for review.

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

Change subject: ruthenium: Updated update_parsoid.sh to run it as regular user
......................................................................

ruthenium: Updated update_parsoid.sh to run it as regular user

Change-Id: I6b26e89cf824e1ccd4ae0ffdce57258ea47ad601
---
M modules/admin/data/data.yaml
M modules/parsoid/files/parsoid_testing.update_parsoid.sh
2 files changed, 3 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/47/271047/1

diff --git a/modules/admin/data/data.yaml b/modules/admin/data/data.yaml
index 17b49cd..4ceca55 100644
--- a/modules/admin/data/data.yaml
+++ b/modules/admin/data/data.yaml
@@ -26,8 +26,7 @@
     description: RT 5934
     members: [ssastry, cscott, arlolra]
     privileges: ['ALL = NOPASSWD: /usr/sbin/service parsoid *',
-                 'ALL = NOPASSWD: /usr/sbin/service parsoid-rt-client restart',
-                 'ALL = (parsoid) NOPASSWD: /usr/local/bin/update_parsoid.sh']
+                 'ALL = NOPASSWD: /usr/sbin/service parsoid-rt-client restart' 
]
   gerrit-root:
     gid: 703
     description: manage gerrit server
@@ -477,7 +476,6 @@
                'ALL = NOPASSWD: /usr/sbin/service parsoid-vd *',
                'ALL = NOPASSWD: /usr/sbin/service parsoid-vd-client *',
                'ALL = NOPASSWD: /usr/sbin/service diffservice *',
-               'ALL = (parsoid) NOPASSWD: /usr/local/bin/update_parsoid.sh',
                'ALL = NOPASSWD: /bin/journalctl *']
 users:
   rush:
diff --git a/modules/parsoid/files/parsoid_testing.update_parsoid.sh 
b/modules/parsoid/files/parsoid_testing.update_parsoid.sh
index 521a3b2..214f57a 100644
--- a/modules/parsoid/files/parsoid_testing.update_parsoid.sh
+++ b/modules/parsoid/files/parsoid_testing.update_parsoid.sh
@@ -12,7 +12,7 @@
 git log -n 1
 cd ..
 echo 'Restarting parsoid ...'
-service parsoid restart
+sudo service parsoid restart
 echo 'Restarting parsoid-rt testreduce clients ...'
-service parsoid-rt-client restart
+sudo service parsoid-rt-client restart
 echo 'All done!'

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6b26e89cf824e1ccd4ae0ffdce57258ea47ad601
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Subramanya Sastry <[email protected]>

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

Reply via email to