osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/docker-playground/+/37973?usp=email )
Change subject: debian-bookworm-build-arm: FROM: add arm32v7
......................................................................
debian-bookworm-build-arm: FROM: add arm32v7
Use the arm32v7 architecture explicitly, instead of implicitly using the
same architecture as the host. That way we can use the arm32v7 image on
a raspberry pi 4, even if the host system is 64 bit arm.
Using 32 bit arm instead of aarch64 is needed for the osmo-trx jenkins
jobs, which test -mfpu=neon and -mfpu=neon-vfpv4 build flags.
Related: https://github.com/docker-library/official-images#architectures-other-than-…
Change-Id: I93986569a9f492237da28fbc534ac5ccd176f120
---
M debian-bookworm-build-arm/Dockerfile
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/docker-playground refs/changes/73/37973/1
diff --git a/debian-bookworm-build-arm/Dockerfile b/debian-bookworm-build-arm/Dockerfile
index 9b725a7..a6a1b21 100644
--- a/debian-bookworm-build-arm/Dockerfile
+++ b/debian-bookworm-build-arm/Dockerfile
@@ -3,7 +3,7 @@
# See master-builds.yml, gerrit-verifications.yml in osmo-ci.git.
ARG REGISTRY=docker.io
ARG UPSTREAM_DISTRO=debian:bookworm
-FROM ${REGISTRY}/${UPSTREAM_DISTRO}
+FROM ${REGISTRY}/arm32v7/${UPSTREAM_DISTRO}
# Arguments used after FROM must be specified again
ARG OSMOCOM_REPO_MIRROR="https://downloads.osmocom.org"
--
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: newchange
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>
Attention is currently required from: dexter, laforge.
fixeria has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/37819?usp=email )
Change subject: Testsuite for pySim-shell with real cards
......................................................................
Patch Set 15:
(1 comment)
File tests/pySim-shell_test/ara_m/test.py:
https://gerrit.osmocom.org/c/pysim/+/37819/comment/01c1a29e_f8a8ed30?usp=em… :
PS15, Line 27: self.setup(__file__)
> `inspect.getfile(MyClass)` or `inspect.getfile(inst.__class__)` should give you the file name.
Either what @laforge@osmocom.org suggested, or adding a class variable should work.
```
class test_case(UnittestUtils):
testcasepath = __file__
```
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/37819?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ieae1330767a6e55e62437f5f988a0d33b727b5de
Gerrit-Change-Number: 37819
Gerrit-PatchSet: 15
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 02 Sep 2024 12:34:56 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>
Attention is currently required from: dexter, fixeria.
laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/37819?usp=email )
Change subject: Testsuite for pySim-shell with real cards
......................................................................
Patch Set 15:
(2 comments)
Patchset:
PS15:
a
File tests/pySim-shell_test/ara_m/test.py:
https://gerrit.osmocom.org/c/pysim/+/37819/comment/f421a278_40b93c6c?usp=em… :
PS15, Line 27: self.setup(__file__)
> Initially I had it in the constructor of UnittestUtils, but I ran into problems because I want t dis […]
`inspect.getfile(MyClass)` or `inspect.getfile(inst.__class__)` should give you the file name.
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/37819?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ieae1330767a6e55e62437f5f988a0d33b727b5de
Gerrit-Change-Number: 37819
Gerrit-PatchSet: 15
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 02 Sep 2024 09:45:12 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>
Attention is currently required from: fixeria, laforge.
dexter has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/37819?usp=email )
Change subject: Testsuite for pySim-shell with real cards
......................................................................
Patch Set 15:
(2 comments)
File tests/pySim-shell_test/ara_m/test.py:
https://gerrit.osmocom.org/c/pysim/+/37819/comment/007c70ef_b4cb60d7?usp=em… :
PS15, Line 27: self.setup(__file__)
> Calling this in each test case can be avoided by defining method `setUp` in `UnittestUtils`. […]
Initially I had it in the constructor of UnittestUtils, but I ran into problems because I want t display the unittest name, which I deduct from the file name. Since I could not get the file name through inspection methods I gave up and opted for having this setup method. Do you have any ideas? I would love to get rid of this method call.
https://gerrit.osmocom.org/c/pysim/+/37819/comment/ce417b64_c80a97cd?usp=em… :
PS15, Line 35: self.cleanup()
> Likewise, calling this in each test case can be avoided by defining method `tearDown` in `UnittestUt […]
This could actually work. I will try it.
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/37819?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ieae1330767a6e55e62437f5f988a0d33b727b5de
Gerrit-Change-Number: 37819
Gerrit-PatchSet: 15
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 02 Sep 2024 08:10:52 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
osmith has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37972?usp=email )
Change subject: Update key for Osmocom OBS
......................................................................
Patch Set 1: Code-Review+2 Verified+1
(1 comment)
Patchset:
PS1:
Trivial fix, without this currently all ttcn3 testsuites using testenv are failing. I'll merge it now to have it fixed quickly.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37972?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I732b379b8ac70f5e2139520a07183060a9c1a7f0
Gerrit-Change-Number: 37972
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Mon, 02 Sep 2024 07:25:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes