osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29794 )
Change subject: jenkins-gerrit: add note for build fails/pkgs pass
......................................................................
jenkins-gerrit: add note for build fails/pkgs pass
One might be wondering what happened if the build job failed, but
building the binary packages succeeded. Since we run 'make check' in
all cases. Add this short explanation:
The build job(s) failed, but deb/rpm jobs passed.
We don't enable external/vty tests when building
packages, so maybe those failed. Check the logs.
Related: OS#2385
Change-Id: I70027ec970a220c3ddfa766302faec7bd8752118
---
M scripts/jenkins-gerrit/pipeline_summary.py
1 file changed, 8 insertions(+), 0 deletions(-)
Approvals:
laforge: Looks good to me, but someone else must approve
osmith: Looks good to me, approved
msuraev: Looks good to me, but someone else must approve
Jenkins Builder: Verified
diff --git a/scripts/jenkins-gerrit/pipeline_summary.py b/scripts/jenkins-gerrit/pipeline_summary.py
index a47ab7c..95ae821 100755
--- a/scripts/jenkins-gerrit/pipeline_summary.py
+++ b/scripts/jenkins-gerrit/pipeline_summary.py
@@ -156,6 +156,14 @@
summary += f"{len(jobs['passed'])} passed:\n"
summary += get_jobs_list_str(jobs["passed"])
+ if "build" in pipeline and "deb" in pipeline and "rpm" in pipeline and \
+ not pipeline["build"]["passed"] and pipeline["deb"]["passed"] \
+ and pipeline["rpm"]["passed"]:
+ summary += "\n"
+ summary += "The build job(s) failed, but deb/rpm jobs passed.\n"
+ summary += "We don't enable external/vty tests when building\n"
+ summary += "packages, so maybe those failed. Check the logs.\n"
+
if "lint" in pipeline and not pipeline["lint"]["passed"]:
summary += "\n"
summary += "Please fix the linting errors. More information:\n"
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29794
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I70027ec970a220c3ddfa766302faec7bd8752118
Gerrit-Change-Number: 29794
Gerrit-PatchSet: 3
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
Attention is currently required from: pespin, fixeria.
osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29794 )
Change subject: jenkins-gerrit: add note for build fails/pkgs pass
......................................................................
Patch Set 2: Code-Review+2
(1 comment)
Patchset:
PS2:
> I would really avoid clogging the comments with more and more text. […]
Got 2x +1, it seems this is useful for others. I'll try to not add much more text though.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29794
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I70027ec970a220c3ddfa766302faec7bd8752118
Gerrit-Change-Number: 29794
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(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-Comment-Date: Fri, 21 Oct 2022 08:34:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29831 )
Change subject: {ns,sgsn,gbproxy}: gen_links.sh: do not link unused modules
......................................................................
Patch Set 2:
This change is ready for review.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29831
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: I8fddd010e2a3ce88c892887fc571ae54d84e931f
Gerrit-Change-Number: 29831
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Comment-Date: Thu, 20 Oct 2022 20:37:58 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: fixeria.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29817
to look at the new patch set (#5).
Change subject: library: move IA/IAR Rest Octets to GSM_RestOctets.ttcn
......................................................................
library: move IA/IAR Rest Octets to GSM_RestOctets.ttcn
Change-Id: Icd96fc461e37d1346950a4fa158e0421bd7330c5
---
M library/GSM_RR_Types.ttcn
M library/GSM_RestOctets.ttcn
M library/L1CTL_PortType.ttcn
M library/LAPDm_RAW_PT.ttcn
M pcu/GPRS_Components.ttcn
M pcu/PCU_Tests.ttcn
M pcu/gen_links.sh
M selftest/gen_links.sh
8 files changed, 271 insertions(+), 248 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/17/29817/5
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29817
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: Icd96fc461e37d1346950a4fa158e0421bd7330c5
Gerrit-Change-Number: 29817
Gerrit-PatchSet: 5
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: pespin.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29829 )
Change subject: bsc: Add tests for MGW pool features
......................................................................
Patch Set 1: Code-Review+1
(2 comments)
File bsc/BSC_Tests.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29829/comment/4c371883_ad6c…
PS1, Line 11845: COORD_CMD_ESTABLISHED
I guess this should be private too?
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29829/comment/c1a2de1a_2840…
PS1, Line 12266: /* this conditional canbe dropped once osmo-bsc >1.10.0 is released: */
Mark with TODO? Otherwise it's easy to miss this one when making a new release...
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29829
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: I6a8c30309be406e37190dc67b6ee5af13e1b9e68
Gerrit-Change-Number: 29829
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 20 Oct 2022 19:36:56 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: pespin.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29828 )
Change subject: bsc: Support running several MGW node emulations
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29828
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: Ic84726c09ad3c6bea3ce6f0198db862d7dbb6dd3
Gerrit-Change-Number: 29828
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 20 Oct 2022 19:31:01 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment