June 2026 - gerrit-log - lists.osmocom.org

[XS] Change in osmo-ttcn3-hacks[master]: Revert "bts: limit stderr logging to ERROR to avoid long write() to e...
by fixeria 29 Jun '26

29 Jun '26

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42893?usp=email ) Change subject: Revert "bts: limit stderr logging to ERROR to avoid long write() to ext4 fs" ...................................................................... Revert "bts: limit stderr logging to ERROR to avoid long write() to ext4 fs" This reverts commit 3c798765f04dd7428c76e3c5cbf3f2ee27a0af4d. Reducing logging verbosity did not help: we're still seeing the clock skew errors and restarts. This makes debugging harder, and is not relevant anymore because we've switched to tmpfs [1]. Related: [1] docker-playground.git Id9a93f7149ef7e9bfde1f4fe3a8299ba46645d50 Change-Id: Id234394fda1ce752f93448e2fc03887a097f6b9d --- M bts/osmo-bts.cfg 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/bts/osmo-bts.cfg b/bts/osmo-bts.cfg index b2f5190..7ac38a1 100644 --- a/bts/osmo-bts.cfg +++ b/bts/osmo-bts.cfg @@ -16,7 +16,7 @@ logging print category-hex 0 logging print level 1 ! - logging level set-all error + logging level set-all notice !Disabled due to stderr redirect to ext4 filesystem creating long writes of >500ms (OS#6794): !logging level rsl info !logging level meas info -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42893?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Id234394fda1ce752f93448e2fc03887a097f6b9d Gerrit-Change-Number: 42893 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>

1 0

[S] Change in osmo-ttcn3-hacks[master]: Revert "bts: limit stderr logging to NOTICE to avoid long write() to ...
by fixeria 29 Jun '26

29 Jun '26

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42894?usp=email ) Change subject: Revert "bts: limit stderr logging to NOTICE to avoid long write() to ext4 fs" ...................................................................... Revert "bts: limit stderr logging to NOTICE to avoid long write() to ext4 fs" This reverts commit c3ab9ec33f37c6837fb8bcb0fa0bbd2e98781fb7. Reducing logging verbosity did not help: we're still seeing the clock skew errors and restarts. This makes debugging harder, and is not relevant anymore because we've switched to tmpfs [1]. Related: [1] docker-playground.git Id9a93f7149ef7e9bfde1f4fe3a8299ba46645d50 Change-Id: Ief6c3a6ca3053ccf745a186f7bb440e8540336f4 --- M bts/osmo-bts.cfg 1 file changed, 10 insertions(+), 11 deletions(-) Approvals: osmith: Looks good to me, approved Jenkins Builder: Verified diff --git a/bts/osmo-bts.cfg b/bts/osmo-bts.cfg index 7ac38a1..c583d32 100644 --- a/bts/osmo-bts.cfg +++ b/bts/osmo-bts.cfg @@ -17,17 +17,16 @@ logging print level 1 ! logging level set-all notice - !Disabled due to stderr redirect to ext4 filesystem creating long writes of >500ms (OS#6794): - !logging level rsl info - !logging level meas info - !logging level pag info - !logging level l1c info - !logging level dsp info - !logging level pcu info - !logging level trx info - !logging level osmux info - !logging level lmib info - !logging level lmux info + logging level rsl info + logging level meas info + logging level pag info + logging level l1c info + logging level dsp info + logging level pcu info + logging level trx info + logging level osmux info + logging level lmib info + logging level lmux info ! line vty no login -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42894?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Ief6c3a6ca3053ccf745a186f7bb440e8540336f4 Gerrit-Change-Number: 42894 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>

1 0

[S] Change in onomondo-eim[master]: contrib/jenkins: run 'rebar3 fmt --check'
by osmith 29 Jun '26

29 Jun '26

Attention is currently required from: dexter, fixeria. osmith has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/onomondo-eim/+/42868?usp=email ) Change subject: contrib/jenkins: run 'rebar3 fmt --check' ...................................................................... Patch Set 3: (1 comment) Patchset: PS3: Pushed a new version that displays the diff on error: https://jenkins.osmocom.org/jenkins/job/gerrit-onomondo-eim-build/a1=defaul… I'll submit a patch before this to make it pass now. -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42868?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: I271c90dace6e7b6f052b38054815e6dfcbfa60f5 Gerrit-Change-Number: 42868 Gerrit-PatchSet: 3 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Mon, 29 Jun 2026 09:57:15 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[S] Change in osmo-dev[master]: src/grd: use sys.exit
by osmith 29 Jun '26

29 Jun '26

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-dev/+/42901?usp=email ) Change subject: src/grd: use sys.exit ...................................................................... src/grd: use sys.exit The exit function is a constant from the site module, which "should not be used in programs". Replace it with sys.exit() Related: https://docs.python.org/3/library/constants.html#constants-added-by-the-sit… Change-Id: I95db013e36242d5126ce07b8a392e6dc0a0ecfdd --- M src/grd 1 file changed, 7 insertions(+), 6 deletions(-) Approvals: daniel: Looks good to me, but someone else must approve Jenkins Builder: Verified fixeria: Looks good to me, approved diff --git a/src/grd b/src/grd index 64de140..d86225d 100755 --- a/src/grd +++ b/src/grd @@ -8,8 +8,9 @@ import json import os import subprocess -import urllib.request +import sys import urllib.parse +import urllib.request def get_topdir(): @@ -22,14 +23,14 @@ ).stdout.rstrip() except subprocess.CalledProcessError: print("ERROR: not running inside a git repository") - exit(1) + sys.exit(1) def get_config_path(): ret = f"{get_topdir()}/.gitreview" if not os.path.exists(ret): print(f"ERROR: config not found: {ret}") - exit(1) + sys.exit(1) return ret @@ -72,7 +73,7 @@ try: return subprocess.run(cmd, check=True) except subprocess.CalledProcessError: - exit(1) + sys.exit(1) def git_cherry_pick_fetch_head(): @@ -82,7 +83,7 @@ try: return subprocess.run(cmd, check=True) except subprocess.CalledProcessError: - exit(1) + sys.exit(1) def git_checkout_fetch_head(patch_id, rev): @@ -92,7 +93,7 @@ try: return subprocess.run(cmd, check=True) except subprocess.CalledProcessError: - exit(1) + sys.exit(1) desc = "git review download: fetch and checkout a patch from gerrit" -- To view, visit https://gerrit.osmocom.org/c/osmo-dev/+/42901?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-dev Gerrit-Branch: master Gerrit-Change-Id: I95db013e36242d5126ce07b8a392e6dc0a0ecfdd Gerrit-Change-Number: 42901 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>

1 0

[S] Change in osmo-dev[master]: src/grd: apply ruff formatter
by osmith 29 Jun '26

29 Jun '26

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-dev/+/42900?usp=email ) Change subject: src/grd: apply ruff formatter ...................................................................... src/grd: apply ruff formatter The src/grd script is not part of the default "include" pattern as the script does not end in ".py" (it used to be a shell script earlier). Add it to the config explicitly and run "ruff format". Change-Id: Id8da8e6b3325c8301276fd164ba218e8c5a95951 --- M .ruff.toml M src/grd 2 files changed, 35 insertions(+), 12 deletions(-) Approvals: osmith: Looks good to me, approved daniel: Looks good to me, but someone else must approve Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve diff --git a/.ruff.toml b/.ruff.toml index 6c2df50..c673c3a 100644 --- a/.ruff.toml +++ b/.ruff.toml @@ -6,6 +6,10 @@ "osmo-cn-latest/provision-hlr.py", "sysmobts-calib.py", ] +include = [ + "*.py", + "src/grd", +] [lint] # E741: Ambiguous variable name: `l` diff --git a/src/grd b/src/grd index e149c6f..64de140 100755 --- a/src/grd +++ b/src/grd @@ -14,9 +14,12 @@ def get_topdir(): try: - return subprocess.run(["git", "rev-parse", "--show-toplevel"], - check=True, capture_output=True, - encoding="UTF-8").stdout.rstrip() + return subprocess.run( + ["git", "rev-parse", "--show-toplevel"], + check=True, + capture_output=True, + encoding="UTF-8", + ).stdout.rstrip() except subprocess.CalledProcessError: print("ERROR: not running inside a git repository") exit(1) @@ -71,8 +74,9 @@ except subprocess.CalledProcessError: exit(1) + def git_cherry_pick_fetch_head(): - cmd = ["git", "cherry-pick", "FETCH_HEAD"]; + cmd = ["git", "cherry-pick", "FETCH_HEAD"] print(f"+ {' '.join(cmd)}") try: @@ -80,6 +84,7 @@ except subprocess.CalledProcessError: exit(1) + def git_checkout_fetch_head(patch_id, rev): cmd = ["git", "checkout", "-B", f"gerrit/{patch_id}_{rev}", "FETCH_HEAD"] print(f"+ {' '.join(cmd)}") @@ -92,14 +97,28 @@ desc = "git review download: fetch and checkout a patch from gerrit" parser = argparse.ArgumentParser(description=desc) -parser.add_argument("patch_id", type=int, - help="gerrit review ID") -parser.add_argument("-c", "--cherry-pick", action="store_true", - help="cherry-pick into current branch instead of " - "fetching into a branch") -parser.add_argument("-r", "--revision", type=int, - help="patchset revision, default is latest") -parser.add_argument("-v", "--verbose", action="store_true") +parser.add_argument( + "patch_id", + type=int, + help="gerrit review ID", +) +parser.add_argument( + "-c", + "--cherry-pick", + action="store_true", + help="cherry-pick into current branch instead of fetching into a branch", +) +parser.add_argument( + "-r", + "--revision", + type=int, + help="patchset revision, default is latest", +) +parser.add_argument( + "-v", + "--verbose", + action="store_true", +) args = parser.parse_args() host, project = get_config() -- To view, visit https://gerrit.osmocom.org/c/osmo-dev/+/42900?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-dev Gerrit-Branch: master Gerrit-Change-Id: Id8da8e6b3325c8301276fd164ba218e8c5a95951 Gerrit-Change-Number: 42900 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>

1 0

[S] Change in osmo-dev[master]: src/grd: use origin url to get get host + project
by osmith 29 Jun '26

29 Jun '26

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-dev/+/42902?usp=email ) Change subject: src/grd: use origin url to get get host + project ...................................................................... src/grd: use origin url to get get host + project For most projects we can get the gerrit host and project from the git remote URL already. Do this when a git repository has no ".gitreview" file so it works even if the file is not present. Change-Id: Ib2b201e5238ba7036c6121e8875ee85c73da8751 --- M src/grd 1 file changed, 28 insertions(+), 4 deletions(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve daniel: Looks good to me, but someone else must approve osmith: Looks good to me, approved diff --git a/src/grd b/src/grd index d86225d..3c0cf3f 100755 --- a/src/grd +++ b/src/grd @@ -28,14 +28,38 @@ def get_config_path(): ret = f"{get_topdir()}/.gitreview" - if not os.path.exists(ret): - print(f"ERROR: config not found: {ret}") - sys.exit(1) - return ret + return ret if os.path.exists(ret) else None + + +def exit_gerrit_host_project_unknown(): + print("ERROR: failed to get gerrit host and project") + print("Consider adding a .gitreview file:") + print(" [gerrit]") + print(" host=gerrit.osmocom.org") + print(" project=…") + sys.exit(1) + + +def get_config_from_origin_url(): + url = subprocess.run( + ["git", "config", "--get", "remote.origin.url"], + check=True, + capture_output=True, + encoding="UTF-8", + ).stdout.rstrip() + + if not url.startswith("https://gerrit.osmocom.org/"): + exit_gerrit_host_project_unknown() + + host = url.split("/")[2] + project = "/".join(url.split("/")[3:]) + return host, project def get_config(): config_path = get_config_path() + if not config_path: + return get_config_from_origin_url() config = configparser.ConfigParser() config.read(config_path) return config["gerrit"]["host"], config["gerrit"]["project"] -- To view, visit https://gerrit.osmocom.org/c/osmo-dev/+/42902?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-dev Gerrit-Branch: master Gerrit-Change-Id: Ib2b201e5238ba7036c6121e8875ee85c73da8751 Gerrit-Change-Number: 42902 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>

1 0

[S] Change in osmo-dev[master]: src/grd: use origin url to get get host + project
by osmith 29 Jun '26

29 Jun '26

osmith has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-dev/+/42902?usp=email ) Change subject: src/grd: use origin url to get get host + project ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-dev/+/42902?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-dev Gerrit-Branch: master Gerrit-Change-Id: Ib2b201e5238ba7036c6121e8875ee85c73da8751 Gerrit-Change-Number: 42902 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Mon, 29 Jun 2026 09:56:13 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in osmo-dev[master]: src/grd: apply ruff formatter
by osmith 29 Jun '26

29 Jun '26

osmith has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-dev/+/42900?usp=email ) Change subject: src/grd: apply ruff formatter ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-dev/+/42900?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-dev Gerrit-Branch: master Gerrit-Change-Id: Id8da8e6b3325c8301276fd164ba218e8c5a95951 Gerrit-Change-Number: 42900 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Mon, 29 Jun 2026 09:56:02 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in osmo-dev[master]: src/grd: use origin url to get get host + project
by daniel 29 Jun '26

29 Jun '26

Attention is currently required from: osmith. daniel has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-dev/+/42902?usp=email ) Change subject: src/grd: use origin url to get get host + project ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-dev/+/42902?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-dev Gerrit-Branch: master Gerrit-Change-Id: Ib2b201e5238ba7036c6121e8875ee85c73da8751 Gerrit-Change-Number: 42902 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Mon, 29 Jun 2026 09:55:46 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in onomondo-eim[master]: contrib/jenkins: run 'rebar3 fmt --check'
by osmith 29 Jun '26

29 Jun '26

Attention is currently required from: dexter, fixeria, osmith. Hello Jenkins Builder, dexter, fixeria, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/onomondo-eim/+/42868?usp=email to look at the new patch set (#3). The following approvals got outdated and were removed: Verified-1 by Jenkins Builder Change subject: contrib/jenkins: run 'rebar3 fmt --check' ...................................................................... contrib/jenkins: run 'rebar3 fmt --check' The pull request "v0.0.2" had a patch that formatted everything with erlfmt. Let's run it in CI to keep the style consistent. Related: https://github.com/onomondo/onomondo-eim/pull/10/commits/8bafdb3839d868d6bc… Change-Id: I271c90dace6e7b6f052b38054815e6dfcbfa60f5 --- M contrib/jenkins.sh 1 file changed, 10 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/onomondo-eim refs/changes/68/42868/3 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42868?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: I271c90dace6e7b6f052b38054815e6dfcbfa60f5 Gerrit-Change-Number: 42868 Gerrit-PatchSet: 3 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de>

1 0

2026

2025

2024

2023

2022

gerrit-log June 2026