osmith has submitted this change. (
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(-)
Approvals:
laforge: Looks good to me, approved
pespin: Looks good to me, but someone else must approve
Jenkins Builder: Verified
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: 2
Gerrit-Owner: osmith <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged
