Attention is currently required from: osmith. msuraev has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29671 ) Change subject: jobs/gerrit-verifications: write summary comment ...................................................................... Patch Set 1: (2 comments) File jobs/gerrit-pipeline-result.yml: https://gerrit.osmocom.org/c/osmo-ci/+/29671/comment/ae341e30_acd70913 PS1, Line 12: node: osmocom-gerrit-debian10 || osmocom-gerrit-debian11 What's the reason for using Debian 10? I though the plan is to migrate everything we can to Debian 11. File scripts/jenkins-gerrit/pipeline_summary.py: https://gerrit.osmocom.org/c/osmo-ci/+/29671/comment/399112c5_f5fc2401 PS1, Line 174: def main(): Why extra function? You can just move it directly below where main() is called. -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29671 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Idcab969e1b5ca4e0f1383bee8f36f2d1aac4f624 Gerrit-Change-Number: 29671 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: msuraev <msuraev(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Sat, 08 Oct 2022 19:10:46 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment