osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28415 )
Change subject: jobs/osmocom-obs: run against obs.osmocom.org too
......................................................................
jobs/osmocom-obs: run against obs.osmocom.org too
Split the OBS related jenkins jobs into two, one that uploads to the
new obs.osmocom.org and one that uploads to build.opensuse.org.
Set an alias for "osc" that sets the right server, and set the
expand_aliases shopt so the bash scripts that source common-obs.sh will
not ignore the alias.
Related: OS#5557
Change-Id: Ib271e16e93ebc58ed1ecb0ea881e1f1370e40928
---
M jobs/osmocom-obs.yml
M scripts/common-obs.sh
2 files changed, 22 insertions(+), 9 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, but someone else must approve
pespin: Looks good to me, but someone else must approve
osmith: Looks good to me, approved
diff --git a/jobs/osmocom-obs.yml b/jobs/osmocom-obs.yml
index 3b8b1d8..cfd3bff 100644
--- a/jobs/osmocom-obs.yml
+++ b/jobs/osmocom-obs.yml
@@ -2,23 +2,24 @@
- project:
name: Osmocom_OBS
jobs:
- - Osmocom_OBS_{type}
+ - Osmocom_OBS_{type}_{server}
type:
- nightly
- latest
# - next (disabled: OS#5322)
+ server:
+ - build.opensuse.org:
+ proj: "network:osmocom"
+ - obs.osmocom.org:
+ proj: "osmocom"
- job-template:
- name: 'Osmocom_OBS_{type}'
+ name: 'Osmocom_OBS_{type}_{server}'
project-type: freestyle
defaults: global
description: |
- <ul>
- <li><a href="https://build.opensuse.org/project/show/network:osmocom:{type}">osmocom:{type} home page</a>
- <li><a href="https://build.opensuse.org/project/monitor/network:osmocom:{type}">osmocom:{type} build monitor</a>
- <li><a href="https://osmocom.org/projects/cellular-infrastructure/wiki/Binary_Packages">Wiki: binary packages</a>
- </ul>
- (Generated by job-builder)
+ See <a href="https://osmocom.org/projects/cellular-infrastructure/wiki/Binary_Packages">Wiki: binary packages</a>
+ for more information.
node: obs
parameters:
- string:
@@ -27,7 +28,9 @@
default: 'refs/remotes/origin/master'
builders:
- shell: |
- export PROJ=network:osmocom:{type}
+ export PROJ={proj}:{type}
+ export OBS_SERVER={server}
+
./scripts/osmocom-{type}-packages.sh
scm:
- git:
diff --git a/scripts/common-obs.sh b/scripts/common-obs.sh
index 91d54c9..4de2c55 100644
--- a/scripts/common-obs.sh
+++ b/scripts/common-obs.sh
@@ -29,6 +29,16 @@
exit 1
fi
+if [ -z "$OBS_SERVER" ]; then
+ echo "OBS_SERVER environment variable is not set"
+ exit 1
+fi
+
+# Related configuration sections are in .oscrc (OS#5557)
+echo "Using OBS server: $OBS_SERVER"
+shopt -s expand_aliases
+alias osc="osc -A '$OBS_SERVER'"
+
# Add dependency to all (sub)packages in debian/control and commit the change.
# $1: path to debian/control file
# $2: package name (e.g. "libosmocore")
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28415
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ib271e16e93ebc58ed1ecb0ea881e1f1370e40928
Gerrit-Change-Number: 28415
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28415 )
Change subject: jobs/osmocom-obs: run against obs.osmocom.org too
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28415
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ib271e16e93ebc58ed1ecb0ea881e1f1370e40928
Gerrit-Change-Number: 28415
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 28 Jun 2022 09:59:18 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ci/+/28421
to look at the new patch set (#4).
Change subject: install-coverity: update to 2021.12
......................................................................
install-coverity: update to 2021.12
Starting in February 2022, coverity stopped to support the old
2019.03 version that we were using. Unfortunately that didn't
generate any e-mail notification to the project admins so it
remained undetected until now :(
What sucks is that the filename states 2021.12 but then the
actual directory created upon extraction is 2021.12.1, meaning
our ansible rule breaks. We have to rename the tarball to 2021.12.1
before running the playbook...
Closes: OS#5594
Change-Id: I950834cc2ad70979a53a563610bbc9feef1f5f20
---
M ansible/roles/install-coverity/defaults/main.yml
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/21/28421/4
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28421
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I950834cc2ad70979a53a563610bbc9feef1f5f20
Gerrit-Change-Number: 28421
Gerrit-PatchSet: 4
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ci/+/28421
to look at the new patch set (#3).
Change subject: install-coverity: update to 2021.12
......................................................................
install-coverity: update to 2021.12
Starting in February 2022, coverity stopped to support the old
2019.03 version that we were using. Unfortunately that didn't
generate any e-mail notification to the project admins so it
remained undetected until now :(
What sucks is that the filename states 2021.12 but then the
actual directory created upon extraction is 2021.12.1, meaning
our ansible rule breaks. We have to rename the tarball to 2021.12.1
before running the playbook...
Change-Id: I950834cc2ad70979a53a563610bbc9feef1f5f20
---
M ansible/roles/install-coverity/defaults/main.yml
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/21/28421/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28421
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I950834cc2ad70979a53a563610bbc9feef1f5f20
Gerrit-Change-Number: 28421
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ci/+/28421
to look at the new patch set (#2).
Change subject: install-coverity: update to 2021.12
......................................................................
install-coverity: update to 2021.12
Starting in February 2022, coverity stopped to support the old
2019.03 version that we were using. Unfortunately that didn't
generate any e-mail notification to the project admins so it
remained undetected until now :(
Change-Id: I950834cc2ad70979a53a563610bbc9feef1f5f20
---
M ansible/roles/install-coverity/defaults/main.yml
M ansible/roles/install-coverity/tasks/main.yml
2 files changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/21/28421/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28421
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I950834cc2ad70979a53a563610bbc9feef1f5f20
Gerrit-Change-Number: 28421
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset
laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/28421 )
Change subject: install-coverity: update to 2021.12
......................................................................
install-coverity: update to 2021.12
Starting in February 2022, coverity stopped to support the old
2019.03 version that we were using. Unfortunately that didn't
generate any e-mail notification to the project admins so it
remained undetected until now :(
Change-Id: I950834cc2ad70979a53a563610bbc9feef1f5f20
---
M ansible/roles/install-coverity/defaults/main.yml
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/21/28421/1
diff --git a/ansible/roles/install-coverity/defaults/main.yml b/ansible/roles/install-coverity/defaults/main.yml
index 6b97781..b823ba6 100644
--- a/ansible/roles/install-coverity/defaults/main.yml
+++ b/ansible/roles/install-coverity/defaults/main.yml
@@ -1,4 +1,4 @@
---
-coverity_version: 2019.03
+coverity_version: 2021.12
coverity_installer_file: "cov-analysis-linux64-{{ coverity_version }}.tar.gz"
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28421
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I950834cc2ad70979a53a563610bbc9feef1f5f20
Gerrit-Change-Number: 28421
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: newchange
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ci/+/28349
to look at the new patch set (#2).
Change subject: update git URLs (git -> https; gitea/gerrit)
......................................................................
update git URLs (git -> https; gitea/gerrit)
Change-Id: Ia44e4127ff87a3d4d747225d3258188a0a891f1b
---
M ansible/hosts
M ansible/roles/gsm-tester/tasks/main.yml
M ansible/roles/osmocom-workstation/tasks/git-repos.yml
M contrib/jenkins.sh
M coverity/prepare_source_Osmcocom.sh
M jobs/coverity.yml
M jobs/master-builds.yml
M jobs/octsim_osmo-ccid-firmware.yml
M jobs/osmo-gsm-manuals-trigger.yml
M jobs/osmo-gsm-tester-builder.yml
M jobs/osmo-gsm-tester-runner.yml
M jobs/osmocom-build-tags-against-master.yml
M jobs/osmocom-depcheck.yml
M jobs/osmocom-list-commits.yml
M jobs/osmocom-obs.yml
M jobs/osmocom-release-tarballs.yml
M jobs/registry-rebuild-upload-fpga-build.yml
M jobs/registry-rebuild-upload-titan.yml
M jobs/registry-triggers.yml
M jobs/repo-install-test.yml
M jobs/ttcn3-testsuites-kernel-git.yml
M jobs/ttcn3-testsuites.yml
M jobs/update-osmo-ci-on-slaves.yml
M qemu-kvm/ggsn-kernel-gtp/postinst.sh
M scripts/osmo-build.sh
M scripts/osmo-depcheck/config.py
M scripts/osmo-depcheck/dependencies.py
M scripts/osmo-deps.sh
M scripts/osmo-layer1-headers.sh
29 files changed, 36 insertions(+), 35 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/49/28349/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28349
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ia44e4127ff87a3d4d747225d3258188a0a891f1b
Gerrit-Change-Number: 28349
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset
Attention is currently required from: osmith.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28415 )
Change subject: jobs/osmocom-obs: run against obs.osmocom.org too
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28415
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ib271e16e93ebc58ed1ecb0ea881e1f1370e40928
Gerrit-Change-Number: 28415
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 28 Jun 2022 09:12:51 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: fixeria, pespin.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/28328 )
Change subject: lchan_select: implement dynamic selection mode for assignment
......................................................................
Patch Set 5: Code-Review+1
(1 comment)
File src/osmo-bsc/lchan_select.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/28328/comment/3b7a517e_0cdb6a94
PS2, Line 231: qsort(&trx_list[0], bts->num_trx, sizeof(trx), &qsort_func);
> I though about having a separate function for dynamic mode, but then it would be 80% identical to th […]
I can find arguments in favor of both ways, I don't see a clear advantage of either over the other. So in the interest of not dragging this on any loger, I'd be in favor of merging it as-is
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/28328
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I1b7a0d706976b73cc5c30a8714b830811addfe8d
Gerrit-Change-Number: 28328
Gerrit-PatchSet: 5
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 28 Jun 2022 09:11:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment