Attention is currently required from: fixeria, msuraev.
osmith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ci/+/29604 )

Change subject: jobs/gerrit-binpkgs: new job for deb, rpm packages
......................................................................


Patch Set 4:

(2 comments)

File jobs/gerrit-binpkgs.yml:

https://gerrit.osmocom.org/c/osmo-ci/+/29604/comment/90ba0591_aea18a63
PS4, Line 16:     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 1 […]
Better use of resources. Until we have migrated all nodes at to debian 11, 
limiting to debian 11 here means the job would only run on the 
build2-deb11build-ansible node 
(https://jenkins.osmocom.org/jenkins/label/osmocom-gerrit-debian11/). If 
somebody submits a large patchset to gerrit, then running the gerrit 
verifications doesn't take as long if more nodes can be used. I did not include 
osmocom-gerrit-debian9 because the python version there is so old that the 
script doesn't run.

Note that the debian version here is the one of the jenkins node that runs the 
python script. The script runs with a --docker argument, and the actual package 
build runs in debian 11.


https://gerrit.osmocom.org/c/osmo-ci/+/29604/comment/cef36a91_5dfcc10a
PS4, Line 86:           rpm) distro="centos:8" ;;
> That's EoL as well - see https://www.centos. […]
True. But a customer is using it and we decided to use almalinux:8 in our 
infrastructure. centos:8 gets translated to almalinux:8 here:
https://gitea.osmocom.org/osmocom/osmo-ci/src/commit/c449c5de8d9d891553db832ff074782fa02e4d39/scripts/obs/lib/docker.py#L22-L24

If you prefer I could refactor it so it says almalinux:8 here instead of 
translating it later on. Or add a comment here.



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

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I7ca8869c2e9f2e7c74a360933be12b5c2b47c2fc
Gerrit-Change-Number: 29604
Gerrit-PatchSet: 4
Gerrit-Owner: osmith <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-CC: msuraev <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Attention: msuraev <[email protected]>
Gerrit-Comment-Date: Mon, 10 Oct 2022 11:41:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: msuraev <[email protected]>
Gerrit-MessageType: comment

Reply via email to