- gerrit-log - lists.osmocom.org

Change in pysim[master]: contrib/jenkins.sh: first upload manuals, then execute physical tests
by laforge 10 Feb '22

10 Feb '22

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/27122 ) Change subject: contrib/jenkins.sh: first upload manuals, then execute physical tests ...................................................................... contrib/jenkins.sh: first upload manuals, then execute physical tests The tests with physical cards should not prevent upload of the manuals Change-Id: I8aecb4ce211cbcc3890886ef24b04d01c510b6da --- M contrib/jenkins.sh 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index a01c012..480f3e5 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -47,10 +47,10 @@ pip3 install -e 'git+https://github.com/osmocom/sphinx-argparse@master#egg=sphinx-argparse' (cd docs && make html latexpdf) -# run the test with physical cards -cd pysim-testdata -../tests/pysim-test.sh - if [ "$WITH_MANUALS" = "1" ] && [ "$PUBLISH" = "1" ]; then make -C "$base/docs" publish fi + +# run the test with physical cards +cd pysim-testdata +../tests/pysim-test.sh -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27122 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I8aecb4ce211cbcc3890886ef24b04d01c510b6da Gerrit-Change-Number: 27122 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged

1 0

Change in pysim[master]: contrib/jenkins.sh: Fix PUBLISH
by laforge 10 Feb '22

10 Feb '22

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/27123 ) Change subject: contrib/jenkins.sh: Fix PUBLISH ...................................................................... contrib/jenkins.sh: Fix PUBLISH There is no $base in this script, and the current form renders: make: *** /docs: No such file or directory. Stop. Build step 'Execute shell' marked build as failure Change-Id: Ifcf27f7497daeb285dfb364bff20d0c861c77dcb Related: OS#5271 --- M contrib/jenkins.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index 480f3e5..0fd2473 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -48,7 +48,7 @@ (cd docs && make html latexpdf) if [ "$WITH_MANUALS" = "1" ] && [ "$PUBLISH" = "1" ]; then - make -C "$base/docs" publish + make -C "docs" publish fi # run the test with physical cards -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27123 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ifcf27f7497daeb285dfb364bff20d0c861c77dcb Gerrit-Change-Number: 27123 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged

1 0

Change in simtrace2[master]: cardem: set more reasonable interrupt priorities
by Hoernchen 10 Feb '22

10 Feb '22

Attention is currently required from: laforge, lynxis lazus. Hoernchen has posted comments on this change. ( https://gerrit.osmocom.org/c/simtrace2/+/26995 ) Change subject: cardem: set more reasonable interrupt priorities ...................................................................... Patch Set 2: (1 comment) Patchset: PS2: I'd prefer a less scatted approach to setting the priorities, since anyone ever cares about are the relative values, so setting all in one place allows figuring that out without having to grep the code and tracking down control flow. -- To view, visit https://gerrit.osmocom.org/c/simtrace2/+/26995 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: simtrace2 Gerrit-Branch: master Gerrit-Change-Id: Ie6c97d61d8da3990b6e44144f36cb6d37d194307 Gerrit-Change-Number: 26995 Gerrit-PatchSet: 2 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Wed, 09 Feb 2022 17:21:21 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in pysim[master]: contrib/jenkins.sh: Fix PUBLISH
by laforge 10 Feb '22

10 Feb '22

laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/27123 ) Change subject: contrib/jenkins.sh: Fix PUBLISH ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27123 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ifcf27f7497daeb285dfb364bff20d0c861c77dcb Gerrit-Change-Number: 27123 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: Jenkins Builder Gerrit-Comment-Date: Wed, 09 Feb 2022 17:08:03 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in pysim[master]: contrib/jenkins.sh: first upload manuals, then execute physical tests
by laforge 10 Feb '22

10 Feb '22

laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/27122 ) Change subject: contrib/jenkins.sh: first upload manuals, then execute physical tests ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27122 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I8aecb4ce211cbcc3890886ef24b04d01c510b6da Gerrit-Change-Number: 27122 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: Jenkins Builder Gerrit-Comment-Date: Wed, 09 Feb 2022 17:07:53 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in pysim[master]: contrib/jenkins.sh: Fix PUBLISH
by laforge 10 Feb '22

10 Feb '22

laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/27123 ) Change subject: contrib/jenkins.sh: Fix PUBLISH ...................................................................... contrib/jenkins.sh: Fix PUBLISH There is no $base in this script, and the current form renders: make: *** /docs: No such file or directory. Stop. Build step 'Execute shell' marked build as failure Change-Id: Ifcf27f7497daeb285dfb364bff20d0c861c77dcb Related: OS#5271 --- M contrib/jenkins.sh 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/23/27123/1 diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index 480f3e5..0fd2473 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -48,7 +48,7 @@ (cd docs && make html latexpdf) if [ "$WITH_MANUALS" = "1" ] && [ "$PUBLISH" = "1" ]; then - make -C "$base/docs" publish + make -C "docs" publish fi # run the test with physical cards -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27123 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ifcf27f7497daeb285dfb364bff20d0c861c77dcb Gerrit-Change-Number: 27123 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-MessageType: newchange

1 0

Change in pysim[master]: contrib/jenkins.sh: first upload manuals, then execute physical tests
by laforge 10 Feb '22

10 Feb '22

laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/27122 ) Change subject: contrib/jenkins.sh: first upload manuals, then execute physical tests ...................................................................... contrib/jenkins.sh: first upload manuals, then execute physical tests The tests with physical cards should not prevent upload of the manuals Change-Id: I8aecb4ce211cbcc3890886ef24b04d01c510b6da --- M contrib/jenkins.sh 1 file changed, 4 insertions(+), 4 deletions(-) git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/22/27122/1 diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index a01c012..480f3e5 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -47,10 +47,10 @@ pip3 install -e 'git+https://github.com/osmocom/sphinx-argparse@master#egg=sphinx-argparse' (cd docs && make html latexpdf) -# run the test with physical cards -cd pysim-testdata -../tests/pysim-test.sh - if [ "$WITH_MANUALS" = "1" ] && [ "$PUBLISH" = "1" ]; then make -C "$base/docs" publish fi + +# run the test with physical cards +cd pysim-testdata +../tests/pysim-test.sh -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27122 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I8aecb4ce211cbcc3890886ef24b04d01c510b6da Gerrit-Change-Number: 27122 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-MessageType: newchange

1 0

Change in osmo-ci[master]: jobs/master-builds.yml: Add master-pysim to publish manual
by fixeria 10 Feb '22

10 Feb '22

Attention is currently required from: laforge. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/27121 ) Change subject: jobs/master-builds.yml: Add master-pysim to publish manual ...................................................................... Patch Set 1: Code-Review+1 (1 comment) Commit Message: https://gerrit.osmocom.org/c/osmo-ci/+/27121/comment/5b376f43_69815106 PS1, Line 16: Closes: OS#5271 I believe this change is related to my old ticket too: https://osmocom.org/issues/4384 "Jenkins program-read verification on daily (or weekly) basis". -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/27121 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I4dce05cdb5f2c574256378d5fb66cff1c329c141 Gerrit-Change-Number: 27121 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Wed, 09 Feb 2022 16:54:41 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in osmo-ci[master]: jobs/master-builds.yml: Add master-pysim to publish manual
by laforge 10 Feb '22

10 Feb '22

laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/27121 ) Change subject: jobs/master-builds.yml: Add master-pysim to publish manual ...................................................................... jobs/master-builds.yml: Add master-pysim to publish manual We only had gerrit build verification for pysim so far, but no 'master' job that periodically builds/tests master (and publishes the manual). As a reult, the pySim manual was still the original version from April 2021, while the code has of course progressed quite a bit in git since. Change-Id: I4dce05cdb5f2c574256378d5fb66cff1c329c141 Closes: OS#5271 --- M jobs/master-builds.yml 1 file changed, 5 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/21/27121/1 diff --git a/jobs/master-builds.yml b/jobs/master-builds.yml index 0b226d1..95efdd6 100644 --- a/jobs/master-builds.yml +++ b/jobs/master-builds.yml @@ -371,6 +371,11 @@ - rtl-sdr: email: laforge(a)gnumonks.org steve(a)steve-m.de + - pysim: + concurrent: false + slave_axis: !!python/tuple [simtester] + cmd: 'PUBLISH=1 WITH_MANUALS=1 /usr/bin/timeout 30m ./contrib/jenkins.sh' + - sysmo-usim-tool: concurrent: false cmd: cd ./tests; ./run-tests -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/27121 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I4dce05cdb5f2c574256378d5fb66cff1c329c141 Gerrit-Change-Number: 27121 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-MessageType: newchange

1 0

Change in docker-playground[master]: ttcn3-bts-test: use osmo-config-merge to generate config files
by osmith 10 Feb '22

10 Feb '22

Attention is currently required from: fixeria. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/27103 ) Change subject: ttcn3-bts-test: use osmo-config-merge to generate config files ...................................................................... Patch Set 3: (2 comments) File jenkins-common.sh: https://gerrit.osmocom.org/c/docker-playground/+/27103/comment/eafa5d08_25a… PS1, Line 422: osmo-ttcn3-hacks > Do we have such a testsuite? Maybe rather use 'ttcn3-bts-test' as an example? right, that's what I wanted to put there :) https://gerrit.osmocom.org/c/docker-playground/+/27103/comment/635109f1_692… PS1, Line 422: name > 'image' I guess? Done -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/27103 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: If2d1ba8a775dc6c396f50f9853e633df110329a9 Gerrit-Change-Number: 27103 Gerrit-PatchSet: 3 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Wed, 09 Feb 2022 16:39:10 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: comment

1 0

2026

2025

2024

2023

2022

gerrit-log