Attention is currently required from: fixeria, pespin.
osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/37549?usp=email )
Change subject: jobs/update-osmo-…: only trigger container rebuild
......................................................................
Patch Set 3:
This change is ready for review.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/37549?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I3b06a2fc2c7182d0b0a4d13b82f67a9db1be2fd0
Gerrit-Change-Number: 37549
Gerrit-PatchSet: 3
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 23 Jul 2024 09:26:17 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/37549?usp=email )
Change subject: jobs/update-osmo-python-on-slaves: remove
......................................................................
jobs/update-osmo-python-on-slaves: remove
This job should not be needed anymore, as the builds of almost all
jenkins jobs are done in docker where we install osmo-python-tests
inside the containers if it is needed. We already removed the initial
deployment to jenkins nodes via ansible in
I7c87bd516d1bcdbcc21f65909dabbd3ccc38eede ("ansible:
osmocom_build_deps: remove"), let's remove the job that updates
osmo-python-tests too.
I'm doing this now because the jenkins job started failing after
I42ba71e2fd0ceb8466758ea46e9ae33e3d4fcb91 ("contrib/jenkins.sh: use
virtualenv and pip") on some nodes where virtualenv is not available.
The contrib/jenkins.sh script was used for both CI verification and to
actually deploy this python package to the jenkins nodes, which was
not really obvious and also as noted above isn't needed anymore.
Change-Id: I3b06a2fc2c7182d0b0a4d13b82f67a9db1be2fd0
---
D jobs/update-osmo-python-on-slaves.yml
1 file changed, 24 insertions(+), 47 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/49/37549/1
diff --git a/jobs/update-osmo-python-on-slaves.yml b/jobs/update-osmo-python-on-slaves.yml
deleted file mode 100644
index df01409..0000000
--- a/jobs/update-osmo-python-on-slaves.yml
+++ /dev/null
@@ -1,47 +0,0 @@
-- job:
- name: update-osmo-python-on-slaves
- project-type: matrix
- axes:
- - axis:
- type: slave
- name: label
- values:
- - build2-deb10build-ansible
- - build2-deb11build-ansible
- - build4-deb12build-ansible
- - host2-deb11build-ansible
- - lx2-raspbian11build-ansible
- - rpi4-raspbian11build-ansible-1
- - rpi4-raspbian11build-ansible-2
- - rpi4-raspbian11build-ansible-3
- properties:
- - build-discarder:
- days-to-keep: 30
- num-to-keep: 120
- scm:
- - git:
- url: https://gerrit.osmocom.org/python/osmo-python-tests
- branches:
- - 'origin/master'
- skip-tag: true
- wipe-workspace: true
-
- triggers:
- - pollscm:
- cron: "H/5 * * * *"
- ignore-post-commit-hooks: false
-
- builders:
- - shell: './contrib/jenkins.sh'
- description: |
- <b>Auto-generated using Jenkins Job Builder. DO NOT EDIT MANUALLY!</b>
-
- # make sure to update the osmo-python in docker images, rebuilt from osmo-ci/contrib/jenkins.sh
- publishers:
- - trigger:
- project: update-osmo-ci-on-slaves
- threshold: FAILURE
- - email:
- recipients: jenkins-notifications(a)lists.osmocom.org
-
-# vim: expandtab tabstop=2 shiftwidth=2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/37549?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I3b06a2fc2c7182d0b0a4d13b82f67a9db1be2fd0
Gerrit-Change-Number: 37549
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: newchange
Attention is currently required from: fixeria, pespin.
Hello Jenkins Builder, fixeria, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-pcu/+/37548?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Code-Review-1 by fixeria, Code-Review-1 by pespin, Verified+1 by Jenkins Builder
Change subject: pcuif_sock: do not retry when sending PCU version
......................................................................
pcuif_sock: do not retry when sending PCU version
When some client connects to the PCU socket, the PCU sends a TXT
indication with its version number in it. There is a retry mechanism
in place that checks if the BTS got active. In case the BTS is not
active yet, the TXT indication with the version number is repeated.
This mechanism is unnecessary. It even can cause annoyance in cas the
BTS on the other end does not send an info indication because it does
not use GPRS/EGPRS at all. The TXT indication is sent after the PCU
socket connection is made. We use a normal unix domain socket, which
can not lose messages. It is guaranteed that the TXT indication
reaches the client (BTS or BSC).
Related: OS#6270
Change-Id: Iddd3eed113ed288e629b6029f9a56de427de8126
---
M src/pcuif_sock.c
1 file changed, 22 insertions(+), 22 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/48/37548/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/37548?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Iddd3eed113ed288e629b6029f9a56de427de8126
Gerrit-Change-Number: 37548
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: pespin.
Hello Jenkins Builder, daniel,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37544?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
The change is no longer submittable: Verified is unsatisfied now.
Change subject: sgsn: expected-results.xml: Add missing TC_sgsn_context_req_{in,out}
......................................................................
sgsn: expected-results.xml: Add missing TC_sgsn_context_req_{in,out}
Change-Id: I2579958bbf8c28b72e58174e60634ca77c71dfee
---
M sgsn/expected-results.xml
1 file changed, 11 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/44/37544/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37544?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I2579958bbf8c28b72e58174e60634ca77c71dfee
Gerrit-Change-Number: 37544
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: daniel, fixeria, laforge, osmith, pespin.
Hello Jenkins Builder, daniel, fixeria, laforge, osmith,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37545?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Code-Review+1 by fixeria, Code-Review+1 by laforge, Verified+1 by Jenkins Builder
Change subject: sgsn: Move BSSGP_ConnHdlr code to its own file
......................................................................
sgsn: Move BSSGP_ConnHdlr code to its own file
This was we split generic code from test specific code, making it easier
to focus on new test and finding generic functions which can be reused.
This is similar to what we already use in tons of other testsuites.
Related: SYS#5435
Change-Id: I0591cab8cca5195b8648920d5f79acc536a4efac
---
A sgsn/BSSGP_ConnHdlr.ttcn
M sgsn/SGSN_Tests.ttcn
M sgsn/SGSN_Tests_Iu.ttcn
3 files changed, 924 insertions(+), 842 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/45/37545/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37545?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I0591cab8cca5195b8648920d5f79acc536a4efac
Gerrit-Change-Number: 37545
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-MessageType: newpatchset