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
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
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
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
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
Attention is currently required from: osmith.
Hello fixeria,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/docker-playground/+/27103
to look at the new patch set (#3).
Change subject: ttcn3-bts-test: use osmo-config-merge to generate config files
......................................................................
ttcn3-bts-test: use osmo-config-merge to generate config files
This uses the config files from osmo-ttcn3-hacks.git instead of keeping
local copies around here, which has shown many times will get out of
sync. Copy the files from the git repository checked out in the
ttcn3-bts-test docker image, so we don't need to take care of cloning
the repository again and checking out the same commit as used inside
that image. Original files are copied to *.src.cfg, and then merged with
the local *.confmerge files using osmo-conf-merge (*.gen.cfg).
Unfortunately, osmo-config-merge is not capable of removing text blocks
or substituting specific parts, so we fall-back to using sed where it's
needed. A good example is 'log gsmtap 127.0.0.1', as there is no
command for removing a logging target of such type.
Co-Authored-By: Vadim Yanitskiy <vyanitskiy(a)sysmocom.de>
Co-Authored-By: Oliver Smith <osmith(a)sysmocom.de>
Change-Id: If2d1ba8a775dc6c396f50f9853e633df110329a9
---
M .gitignore
M jenkins-common.sh
D ttcn3-bts-test/fh/osmo-bsc.cfg
A ttcn3-bts-test/fh/osmo-bsc.confmerge
M ttcn3-bts-test/jenkins.sh
D ttcn3-bts-test/oml/osmo-bts.cfg
A ttcn3-bts-test/oml/osmo-bts.confmerge
D ttcn3-bts-test/osmo-bsc.cfg
A ttcn3-bts-test/osmo-bsc.confmerge
D ttcn3-bts-test/osmo-bts.cfg
A ttcn3-bts-test/osmo-bts.confmerge
D ttcn3-bts-test/virtphy/osmo-bts.cfg
A ttcn3-bts-test/virtphy/osmo-bts.confmerge
13 files changed, 336 insertions(+), 851 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/docker-playground refs/changes/03/27103/3
--
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: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: newpatchset