osmith has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-ci/+/33992 )


Change subject: scripts/jenkins-gerrit: sort jobs list
......................................................................

scripts/jenkins-gerrit: sort jobs list

Having two debian entries in there in random order looks weird so sort
the job lists.

Change-Id: I795f821783b52a6a9cd1967d33f67b656efccf22
---
M scripts/jenkins-gerrit/comment_generate.py
1 file changed, 15 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/92/33992/1

diff --git a/scripts/jenkins-gerrit/comment_generate.py 
b/scripts/jenkins-gerrit/comment_generate.py
index 147faac..e127c4c 100755
--- a/scripts/jenkins-gerrit/comment_generate.py
+++ b/scripts/jenkins-gerrit/comment_generate.py
@@ -171,10 +171,10 @@


 def get_jobs_list_str(jobs):
-    ret = ""
+    lines = []
     for job in jobs:
-        ret += f"* [{get_job_short_name(job)}] {job['url']}/consoleFull\n"
-    return ret
+        lines += [f"* [{get_job_short_name(job)}] {job['url']}/consoleFull\n"]
+    return "".join(sorted(lines))


 def get_comment_result(build_url, notify_on_success):

--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/33992
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I795f821783b52a6a9cd1967d33f67b656efccf22
Gerrit-Change-Number: 33992
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <[email protected]>
Gerrit-MessageType: newchange

Reply via email to