osmith submitted this change.

View Change


Approvals: laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified
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(-)

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 change 33992. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I795f821783b52a6a9cd1967d33f67b656efccf22
Gerrit-Change-Number: 33992
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged