Rush has submitted this change and it was merged.
Change subject: tools: sge colletor set SGE_ROOT
......................................................................
tools: sge colletor set SGE_ROOT
Add env context for qstat similar to qconf
Bug: T140999
Change-Id: If243a00f509ad19627fe83520e9861527cbd4f4a
---
M modules/toollabs/files/monitoring/sge.py
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
Rush: Verified; Looks good to me, approved
diff --git a/modules/toollabs/files/monitoring/sge.py
b/modules/toollabs/files/monitoring/sge.py
index ba3079b..2e08579 100644
--- a/modules/toollabs/files/monitoring/sge.py
+++ b/modules/toollabs/files/monitoring/sge.py
@@ -38,7 +38,9 @@
:note: shell=true is to honor askerisk via subprocess
"""
job_query = "/usr/bin/qstat -q %s -u '*'" % (queue,)
- return subprocess.check_output([job_query], shell=True)
+ return subprocess.check_output([job_query],
+ env={"SGE_ROOT":
'/data/project/.system/gridengine/'},
+ shell=True)
def get_queues(self):
""" retrieve list of queues
--
To view, visit https://gerrit.wikimedia.org/r/305588
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If243a00f509ad19627fe83520e9861527cbd4f4a
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Rush <r...@wikimedia.org>
Gerrit-Reviewer: Coren <m...@uberbox.org>
Gerrit-Reviewer: Hashar <has...@free.fr>
Gerrit-Reviewer: Merlijn van Deen <valhall...@arctus.nl>
Gerrit-Reviewer: Rush <r...@wikimedia.org>
Gerrit-Reviewer: Yuvipanda <yuvipa...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
