October 2024 - gerrit-log - lists.osmocom.org

[S] Change in osmo-ci[master]: jobs/master-builds: add param BRANCH
by pespin 02 Oct '24

02 Oct '24

Attention is currently required from: osmith. pespin has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ci/+/38319?usp=email ) Change subject: jobs/master-builds: add param BRANCH ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38319?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: I5fb64024dc4ad70cb3dd019bd505aacc58d14380 Gerrit-Change-Number: 38319 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Wed, 02 Oct 2024 10:05:03 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in osmo-ci[master]: jobs/simtester-sanitize: new job
by pespin 02 Oct '24

02 Oct '24

Attention is currently required from: osmith. pespin has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ci/+/38318?usp=email ) Change subject: jobs/simtester-sanitize: new job ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38318?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: Ie1846b031224a2a9604c22e7d81016c08e217bbc Gerrit-Change-Number: 38318 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Wed, 02 Oct 2024 10:04:11 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in osmo-ci[master]: jobs/osmocom-obs-nightly-asan: exclude pyosmocom
by pespin 02 Oct '24

02 Oct '24

Attention is currently required from: osmith. pespin has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ci/+/38317?usp=email ) Change subject: jobs/osmocom-obs-nightly-asan: exclude pyosmocom ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38317?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: I5f3c53ca55661970e29b9d4c10b7c9dbff53f00f Gerrit-Change-Number: 38317 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Wed, 02 Oct 2024 10:03:36 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XL] Change in pysim[master]: tests: sanitize all cards before running tests
by osmith 02 Oct '24

02 Oct '24

Attention is currently required from: dexter, fixeria, laforge. osmith has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/38254?usp=email ) Change subject: tests: sanitize all cards before running tests ...................................................................... Patch Set 3: (2 comments) Patchset: PS1: > Restoring the cards in parallel should be possible. I see no problem there. […] As discussed with dexter, added a separate jenkins job that runs nightly and can be triggered manually if needed: * https://jenkins.osmocom.org/jenkins/job/simtester-sanitize/ * https://gerrit.osmocom.org/c/osmo-ci/+/38318 File contrib/jenkins.sh: https://gerrit.osmocom.org/c/pysim/+/38254/comment/8405cc9c_089aad9c?usp=em… : PS3, Line 36: tests/card_sanitizer/card_sanitizer.py this needs to be removed, as discussed a separate jenkins job calls the script now. * https://jenkins.osmocom.org/jenkins/job/simtester-sanitize/ * https://gerrit.osmocom.org/c/osmo-ci/+/38318 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38254?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: I42eaf61280968518164f2280245136fd30a603ce Gerrit-Change-Number: 38254 Gerrit-PatchSet: 3 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Wed, 02 Oct 2024 10:02:23 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <laforge(a)osmocom.org> Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>

1 0

[S] Change in osmo-ci[master]: jobs/master-builds: add param BRANCH
by osmith 02 Oct '24

02 Oct '24

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/38319?usp=email ) Change subject: jobs/master-builds: add param BRANCH ...................................................................... jobs/master-builds: add param BRANCH Sometimes it is desirable to run master jobs from a different branch: * As discussed with Philipp: running pysim CI on a branch with many patches once before submitting to gerrit (it takes much longer in gerrit to run on each individual patch) * When adding/modifying master jobs to test if they still work as expected with additional patches in a branch. It is undesirable to have mail notifications if not running on master, the next patch will make it easy to disable those. Change-Id: I5fb64024dc4ad70cb3dd019bd505aacc58d14380 --- M jobs/master-builds.yml 1 file changed, 10 insertions(+), 5 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/19/38319/1 diff --git a/jobs/master-builds.yml b/jobs/master-builds.yml index 730c68c..ee6947f 100644 --- a/jobs/master-builds.yml +++ b/jobs/master-builds.yml @@ -74,16 +74,14 @@ gerrit_url: 'https://gerrit.osmocom.org' gitea_url: 'https://gitea.osmocom.org' repos_url: '{gerrit_url}/{repos}' - branches: - - master + branch: master ssh: - docs_ftp.osmocom.org # in alphabetical order repos: - asn1c: - branches: - - osmo-iuh/master + branch: osmo-iuh/master - libasn1c: # Full triggers would be: @@ -632,10 +630,17 @@ name: '{obj:a4_name}' values: '{obj:a4}' + parameters: + - string: + name: BRANCH + description: git branch + default: '{obj:branch}' + scm: - git: url: '{obj:repos_url}' - branches: '{obj:branches}' + branches: + - '$BRANCH' refspec: name: wipe-workspace: false -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38319?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I5fb64024dc4ad70cb3dd019bd505aacc58d14380 Gerrit-Change-Number: 38319 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de>

1 0

[S] Change in osmo-ci[master]: jobs/simtester-sanitize: new job
by osmith 02 Oct '24

02 Oct '24

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/38318?usp=email ) Change subject: jobs/simtester-sanitize: new job ...................................................................... jobs/simtester-sanitize: new job Reset SIM cards every night, and when the job gets triggered manually. Depends: pysim I42eaf61280968518164f2280245136fd30a603ce Change-Id: Ie1846b031224a2a9604c22e7d81016c08e217bbc --- M jobs/README.md A jobs/simtester-sanitize.yml 2 files changed, 41 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/18/38318/1 diff --git a/jobs/README.md b/jobs/README.md index 7c81af5..56b5dfd 100644 --- a/jobs/README.md +++ b/jobs/README.md @@ -167,6 +167,7 @@ 00:XX registry-update-base-images 01:XX osmocom-build-tags-against-master 01:XX osmocom-list-commits + 01:XX simtester-sanitize 02:XX registry-triggers 02:XX registry-rebuild-upload-fpga-build (weekly) diff --git a/jobs/simtester-sanitize.yml b/jobs/simtester-sanitize.yml new file mode 100644 index 0000000..7cb7f26 --- /dev/null +++ b/jobs/simtester-sanitize.yml @@ -0,0 +1,40 @@ +- job: + name: "simtester-sanitize" + project-type: freestyle + defaults: global + node: simtester + description: | + Reset the SIM cards in simtester to a defined state. + builders: + - shell: | + virtualenv -p python3 venv --system-site-packages + . venv/bin/activate + pip install -r requirements.txt + pip install pyshark + tests/card_sanitizer/card_sanitizer.py + properties: + - build-discarder: + days-to-keep: 30 + num-to-keep: 30 + parameters: + - string: + name: BRANCH + description: pysim.git branch + default: 'origin/master' + - string: + name: EMAIL_NOTIFICATIONS + description: For failed build notifications, set to empty to disable + default: 'jenkins-notifications(a)lists.osmocom.org' + scm: + - git: + url: https://gerrit.osmocom.org/pysim + git-config-name: 'Jenkins Builder' + git-config-email: 'jenkins(a)osmocom.org' + branches: + - '$BRANCH' + triggers: + - timed: "H 01 * * *" + publishers: + - email: + recipients: '$EMAIL_NOTIFICATIONS' + notify-every-unstable-build: true -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38318?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ie1846b031224a2a9604c22e7d81016c08e217bbc Gerrit-Change-Number: 38318 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de>

1 0

[XS] Change in osmo-ci[master]: jobs/master-builds: add param EMAIL_NOTIFICATIONS
by osmith 02 Oct '24

02 Oct '24

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/38320?usp=email ) Change subject: jobs/master-builds: add param EMAIL_NOTIFICATIONS ...................................................................... jobs/master-builds: add param EMAIL_NOTIFICATIONS Make it easy to disable mail notifications if running with a different branch than master. Change-Id: Ia8fa57f1efd04684f08db4377c42358c534faff9 --- M jobs/master-builds.yml 1 file changed, 5 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/20/38320/1 diff --git a/jobs/master-builds.yml b/jobs/master-builds.yml index ee6947f..f711d4d 100644 --- a/jobs/master-builds.yml +++ b/jobs/master-builds.yml @@ -635,6 +635,10 @@ name: BRANCH description: git branch default: '{obj:branch}' + - string: + name: EMAIL_NOTIFICATIONS + description: For failed build notifications, set to empty to disable + default: '{obj:email}' scm: - git: @@ -675,7 +679,7 @@ - trigger: project: 'Osmocom_OBS_master' - email: - recipients: '{obj:email}' + recipients: '$EMAIL_NOTIFICATIONS' send-to-individuals: true wrappers: - ssh-agent-credentials: -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38320?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ia8fa57f1efd04684f08db4377c42358c534faff9 Gerrit-Change-Number: 38320 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de>

1 0

[XS] Change in osmo-ci[master]: jobs/osmocom-obs-nightly-asan: exclude pyosmocom
by osmith 02 Oct '24

02 Oct '24

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/38317?usp=email ) Change subject: jobs/osmocom-obs-nightly-asan: exclude pyosmocom ...................................................................... jobs/osmocom-obs-nightly-asan: exclude pyosmocom Order entries alphabetically while at it. Change-Id: I5f3c53ca55661970e29b9d4c10b7c9dbff53f00f --- M jobs/osmocom-obs-nightly-asan.yml 1 file changed, 2 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/17/38317/1 diff --git a/jobs/osmocom-obs-nightly-asan.yml b/jobs/osmocom-obs-nightly-asan.yml index e3bf098..bd5480e 100644 --- a/jobs/osmocom-obs-nightly-asan.yml +++ b/jobs/osmocom-obs-nightly-asan.yml @@ -23,13 +23,14 @@ # We only care about Osmocom packages that run in the TTCN-3 testsuites # for the asan repository. EXCLUDE_PACKAGES=" - erlang/osmo_dia2gsup erlang/osmo-epdg erlang/osmo-s1gw + erlang/osmo_dia2gsup osmo-bsc-nat osmo-fl2k osmo-gsm-manuals python/osmo-python-tests + python/pyosmocom rtl-sdr " -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38317?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I5f3c53ca55661970e29b9d4c10b7c9dbff53f00f Gerrit-Change-Number: 38317 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de>

1 0

[XS] Change in osmo-msc[master]: Release BSS connection when SS message is rejected
by fixeria 01 Oct '24

01 Oct '24

Attention is currently required from: jolly, neels, pespin. fixeria has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/osmo-msc/+/38305?usp=email ) Change subject: Release BSS connection when SS message is rejected ...................................................................... Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38305?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Ic6765e5d480735e67d97f0f560da24653b26d487 Gerrit-Change-Number: 38305 Gerrit-PatchSet: 3 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Tue, 01 Oct 2024 17:13:26 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in osmo-msc[master]: msc/gsup_client_mux.h: remove include of itself
by dexter 01 Oct '24

01 Oct '24

Attention is currently required from: lynxis lazus. dexter has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-msc/+/38108?usp=email ) Change subject: msc/gsup_client_mux.h: remove include of itself ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38108?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: I89ffdb98e98133bd28023a19076f3a5f02e520d2 Gerrit-Change-Number: 38108 Gerrit-PatchSet: 2 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Tue, 01 Oct 2024 14:05:40 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0