Attention is currently required from: laforge, osmith.
Hello Jenkins Builder, laforge,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ci/+/38058?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Code-Review+1 by laforge
Change subject: Stop building openbsc
......................................................................
Stop building openbsc
With recent changes in libosmo-sccp, openbsc does not build anymore.
The openbsc.git repo has been unmaintained/unsupported for 7 years, so
let's finally remove it instead of fixing this.
Related: libosmo-sccp I299a32a2f8f61477cf49c8126567fef2092d5407
Change-Id: Ie58f54c3f2a224fb1e3a0c95bd3ed85e7859fdc9
---
M jobs/gerrit-verifications.yml
M jobs/master-builds.yml
M jobs/osmo-gsm-tester-builder.yml
M scripts/osmocom-build-old-tags-against-master.sh
4 files changed, 5 insertions(+), 39 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/58/38058/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38058?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ie58f54c3f2a224fb1e3a0c95bd3ed85e7859fdc9
Gerrit-Change-Number: 38058
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Attention is currently required from: osmith.
laforge has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ci/+/38058?usp=email )
Change subject: Stop building openbsc
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/osmo-ci/+/38058/comment/b0435510_f6f96cb5?usp=… :
PS1, Line 9: With recent changes in libosmo-sccp, openbsc does not build anymore.
: Harald suggested that we might want to stop building openbsc.git
: altogether now.
I think the rationale should less be "Harald said ..." but more "the openbsc.git repo has been unmaintained/unsupported for 7 years", and that at some point enough is enough.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38058?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ie58f54c3f2a224fb1e3a0c95bd3ed85e7859fdc9
Gerrit-Change-Number: 38058
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 09 Sep 2024 09:16:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Attention is currently required from: pespin.
osmith has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/docker-playground/+/37973?usp=email )
Change subject: debian-bookworm-build-arm: FROM: add arm32v7
......................................................................
Patch Set 1:
(1 comment)
File debian-bookworm-build-arm/Dockerfile:
https://gerrit.osmocom.org/c/docker-playground/+/37973/comment/695a2cfa_5b7… :
PS1, Line 6: FROM ${REGISTRY}/arm32v7/${UPSTREAM_DISTRO}
> we may want to change this file name to mention it's arm32? or it's already implicit vs arm64?
IMHO not worth changing, if we add arm64 too we could name it -arm64
--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/37973?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I93986569a9f492237da28fbc534ac5ccd176f120
Gerrit-Change-Number: 37973
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 09 Sep 2024 09:04:15 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Attention is currently required from: pespin.
osmith has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ci/+/37975?usp=email )
Change subject: jobs/{gerrit,master,update-osmo-ci}: add aarch64 case
......................................................................
Patch Set 1:
(1 comment)
File jobs/gerrit-verifications.yml:
https://gerrit.osmocom.org/c/osmo-ci/+/37975/comment/fd064ada_a267ae2f?usp=… :
PS1, Line 181: arm*|aarch64) DOCKER_IMG="$DOCKER_IMG-arm" ;;
> didn't you just change this docker image in the previous commit to always pull an arm32?
Yes. But the code here runs on the host, it will not use the -arm image if "$(arch)" on the host says "aarch64" without this patch.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/37975?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Iaf987d05870d842cd297f5f14c8542bf75835414
Gerrit-Change-Number: 37975
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 09 Sep 2024 09:02:47 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>