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

[S] Change in osmo-ttcn3-hacks[master]: asterisk/expected-results.xml: Add missing TC_ims_registration

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36852?usp=email ) Change subject: asterisk/expected-results.xml: Add missing TC_ims_registration ...................................................................... asterisk/expected-results.xml: Add missing TC_ims_registration Change-Id: Ie4f589a398f93b429a8b4a93da1b80784b4013ea --- M asterisk/expected-results.xml 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/asterisk/expected-results.xml b/asterisk/expected-results.xml index 745a4d7..49f92bf 100644 --- a/asterisk/expected-results.xml +++ b/asterisk/expected-results.xml @@ -5,4 +5,5 @@ <testcase classname='Asterisk_Tests' name='TC_internal_call_all_2registered' time='MASKED'/> <testcase classname='Asterisk_Tests' name='TC_internal_call_all_3registered' time='MASKED'/> <testcase classname='Asterisk_Tests' name='TC_internal_call_all_4registered' time='MASKED'/> + <testcase classname='Asterisk_Tests' name='TC_ims_registration' time='MASKED'/> </testsuite> -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36852?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: Ie4f589a398f93b429a8b4a93da1b80784b4013ea Gerrit-Change-Number: 36852 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

1 year, 1 month

1
0
0 0

[S] Change in osmo-ttcn3-hacks[master]: ttcn3-tcpdump-start.sh: Log issue with packet dumper not properly sta...

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36853?usp=email ) Change subject: ttcn3-tcpdump-start.sh: Log issue with packet dumper not properly started ...................................................................... ttcn3-tcpdump-start.sh: Log issue with packet dumper not properly started Change-Id: I50af5eb16a000c4f3585a3458f9130a349eff097 --- M ttcn3-tcpdump-start.sh 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve osmith: Looks good to me, approved diff --git a/ttcn3-tcpdump-start.sh b/ttcn3-tcpdump-start.sh index 36aa320..8564cd5 100755 --- a/ttcn3-tcpdump-start.sh +++ b/ttcn3-tcpdump-start.sh @@ -109,6 +109,7 @@ sleep 1 i=$((i+1)) if [ $i -eq 10 ]; then + echo "Packet dumper didn't start filling pcap file after $i seconds!!!" break fi done -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36853?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: I50af5eb16a000c4f3585a3458f9130a349eff097 Gerrit-Change-Number: 36853 Gerrit-PatchSet: 3 Gerrit-Owner: pespin <pespin(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

1 year, 1 month

1
0
0 0

[S] Change in osmo-ttcn3-hacks[master]: asterisk/AMI_Functions: Add workaround for empty 'Hint: ' field

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36854?usp=email ) Change subject: asterisk/AMI_Functions: Add workaround for empty 'Hint: ' field ...................................................................... asterisk/AMI_Functions: Add workaround for empty 'Hint: ' field It sometimes shows up empty, let's skip it to avoid breaking the decoder for now, similar to what we do with other fields. Change-Id: I50fb51fab3755e4d63f3a0bc06d90b42814231d2 --- M asterisk/AMI_Functions.ttcn 1 file changed, 13 insertions(+), 0 deletions(-) Approvals: laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/asterisk/AMI_Functions.ttcn b/asterisk/AMI_Functions.ttcn index 1f3dc5e..c05403e 100644 --- a/asterisk/AMI_Functions.ttcn +++ b/asterisk/AMI_Functions.ttcn @@ -261,6 +261,7 @@ patched_txt := f_str_replace(patched_txt, "DestExten: \r\n", ""); patched_txt := f_str_replace(patched_txt, "Exten: \r\n", ""); patched_txt := f_str_replace(patched_txt, "Extension: \r\n", ""); + patched_txt := f_str_replace(patched_txt, "Hint: \r\n", ""); /* "AppData" field sometimes has a value containing separator ": ", which makes * TEXT dec not happy. Workaround it for now by removing the whole field line: -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36854?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: I50fb51fab3755e4d63f3a0bc06d90b42814231d2 Gerrit-Change-Number: 36854 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(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

1 year, 1 month

1
0
0 0

[S] Change in simtrace2[master]: RFC: card emu config: add feature sim polarity

by lynxis lazus

Attention is currently required from: laforge. lynxis lazus has posted comments on this change. ( https://gerrit.osmocom.org/c/simtrace2/+/36859?usp=email ) Change subject: RFC: card emu config: add feature sim polarity ...................................................................... Patch Set 1: Verified-1 (1 comment) Patchset: PS1: Only to review the protocol approach. -- To view, visit https://gerrit.osmocom.org/c/simtrace2/+/36859?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: simtrace2 Gerrit-Branch: master Gerrit-Change-Id: I36a8acb74ad7616205af68b1626b45ab79d7cb02 Gerrit-Change-Number: 36859 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Fri, 17 May 2024 12:27:39 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 year, 1 month

1
0
0 0

[S] Change in docker-playground[master]: debian-*: install pylint from debian repos

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36860?usp=email ) Change subject: debian-*: install pylint from debian repos ...................................................................... debian-*: install pylint from debian repos Get pylint from the debian repositories to be on the same version until we use the next debian version, instead of always installing the latest pylint version from pip. Change-Id: I721db1c18db43cc7a2d38177e9846c43dbdd0ae3 --- M debian-bookworm-build/Dockerfile M debian-bullseye-jenkins/Dockerfile M debian-buster-jenkins/Dockerfile 3 files changed, 16 insertions(+), 3 deletions(-) Approvals: fixeria: Looks good to me, approved osmith: Verified Jenkins Builder: Verified lynxis lazus: Looks good to me, but someone else must approve diff --git a/debian-bookworm-build/Dockerfile b/debian-bookworm-build/Dockerfile index d847bf1..edca2d7 100644 --- a/debian-bookworm-build/Dockerfile +++ b/debian-bookworm-build/Dockerfile @@ -142,6 +142,7 @@ patchelf \ picolibc-arm-none-eabi \ pkg-config \ + pylint \ python3 \ python3-gi \ python3-mako \ @@ -179,7 +180,6 @@ 'git+https://github.com/osmocom/sphinx-argparse@master#egg=sphinx-argparse' \ 'git+https://github.com/podshumok/python-smpplib.git' \ 'pydbus' \ - 'pylint' \ 'pysispm' \ 'sphinx' \ 'sphinxcontrib-napoleon' \ diff --git a/debian-bullseye-jenkins/Dockerfile b/debian-bullseye-jenkins/Dockerfile index 3c186cc..5132c0a 100644 --- a/debian-bullseye-jenkins/Dockerfile +++ b/debian-bullseye-jenkins/Dockerfile @@ -107,6 +107,7 @@ patchelf \ picolibc-arm-none-eabi \ pkg-config \ + pylint \ python-setuptools \ python2.7 \ python3 \ @@ -148,7 +149,6 @@ 'git+https://github.com/osmocom/sphinx-argparse@master#egg=sphinx-argparse' \ 'git+https://github.com/podshumok/python-smpplib.git' \ 'pydbus' \ - 'pylint' \ 'pysispm' \ 'sphinx' \ 'sphinxcontrib-napoleon' \ diff --git a/debian-buster-jenkins/Dockerfile b/debian-buster-jenkins/Dockerfile index cf443f0..baa5931 100644 --- a/debian-buster-jenkins/Dockerfile +++ b/debian-buster-jenkins/Dockerfile @@ -104,6 +104,7 @@ osc \ patchelf \ pkg-config \ + pylint \ python-pip \ python-setuptools \ python-yaml \ @@ -147,7 +148,6 @@ 'git+https://github.com/osmocom/sphinx-argparse@master#egg=sphinx-argparse' \ 'git+https://github.com/podshumok/python-smpplib.git' \ 'pydbus' \ - 'pylint' \ 'pysispm' \ 'sphinx' \ 'sphinxcontrib-napoleon' \ -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/36860?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I721db1c18db43cc7a2d38177e9846c43dbdd0ae3 Gerrit-Change-Number: 36860 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: merged

1 year, 1 month

1
0
0 0

[S] Change in docker-playground[master]: debian-*: install pylint from debian repos

by fixeria

Attention is currently required from: osmith. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36860?usp=email ) Change subject: debian-*: install pylint from debian repos ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/36860?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I721db1c18db43cc7a2d38177e9846c43dbdd0ae3 Gerrit-Change-Number: 36860 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Fri, 17 May 2024 10:41:56 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 year, 1 month

1
0
0 0

[S] Change in docker-playground[master]: debian-*: install pylint from debian repos

by lynxis lazus

Attention is currently required from: osmith. lynxis lazus has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36860?usp=email ) Change subject: debian-*: install pylint from debian repos ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/36860?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I721db1c18db43cc7a2d38177e9846c43dbdd0ae3 Gerrit-Change-Number: 36860 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Fri, 17 May 2024 10:02:44 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 year, 1 month

1
0
0 0

[S] Change in docker-playground[master]: debian-*: install pylint from debian repos

by osmith

osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36860?usp=email ) Change subject: debian-*: install pylint from debian repos ...................................................................... Patch Set 1: Verified+1 (1 comment) Patchset: PS1: tested that it works -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/36860?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I721db1c18db43cc7a2d38177e9846c43dbdd0ae3 Gerrit-Change-Number: 36860 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-CC: Jenkins Builder Gerrit-Comment-Date: Fri, 17 May 2024 09:56:50 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 year, 1 month

1
0
0 0

[S] Change in docker-playground[master]: debian-*: install pylint from debian repos

by osmith

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/docker-playground/+/36860?usp=email ) Change subject: debian-*: install pylint from debian repos ...................................................................... debian-*: install pylint from debian repos Get pylint from the debian repositories to be on the same version until we use the next debian version, instead of always installing the latest pylint version from pip. Change-Id: I721db1c18db43cc7a2d38177e9846c43dbdd0ae3 --- M debian-bookworm-build/Dockerfile M debian-bullseye-jenkins/Dockerfile M debian-buster-jenkins/Dockerfile 3 files changed, 16 insertions(+), 3 deletions(-) git pull ssh://gerrit.osmocom.org:29418/docker-playground refs/changes/60/36860/1 diff --git a/debian-bookworm-build/Dockerfile b/debian-bookworm-build/Dockerfile index d847bf1..edca2d7 100644 --- a/debian-bookworm-build/Dockerfile +++ b/debian-bookworm-build/Dockerfile @@ -142,6 +142,7 @@ patchelf \ picolibc-arm-none-eabi \ pkg-config \ + pylint \ python3 \ python3-gi \ python3-mako \ @@ -179,7 +180,6 @@ 'git+https://github.com/osmocom/sphinx-argparse@master#egg=sphinx-argparse' \ 'git+https://github.com/podshumok/python-smpplib.git' \ 'pydbus' \ - 'pylint' \ 'pysispm' \ 'sphinx' \ 'sphinxcontrib-napoleon' \ diff --git a/debian-bullseye-jenkins/Dockerfile b/debian-bullseye-jenkins/Dockerfile index 3c186cc..5132c0a 100644 --- a/debian-bullseye-jenkins/Dockerfile +++ b/debian-bullseye-jenkins/Dockerfile @@ -107,6 +107,7 @@ patchelf \ picolibc-arm-none-eabi \ pkg-config \ + pylint \ python-setuptools \ python2.7 \ python3 \ @@ -148,7 +149,6 @@ 'git+https://github.com/osmocom/sphinx-argparse@master#egg=sphinx-argparse' \ 'git+https://github.com/podshumok/python-smpplib.git' \ 'pydbus' \ - 'pylint' \ 'pysispm' \ 'sphinx' \ 'sphinxcontrib-napoleon' \ diff --git a/debian-buster-jenkins/Dockerfile b/debian-buster-jenkins/Dockerfile index cf443f0..baa5931 100644 --- a/debian-buster-jenkins/Dockerfile +++ b/debian-buster-jenkins/Dockerfile @@ -104,6 +104,7 @@ osc \ patchelf \ pkg-config \ + pylint \ python-pip \ python-setuptools \ python-yaml \ @@ -147,7 +148,6 @@ 'git+https://github.com/osmocom/sphinx-argparse@master#egg=sphinx-argparse' \ 'git+https://github.com/podshumok/python-smpplib.git' \ 'pydbus' \ - 'pylint' \ 'pysispm' \ 'sphinx' \ 'sphinxcontrib-napoleon' \ -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/36860?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I721db1c18db43cc7a2d38177e9846c43dbdd0ae3 Gerrit-Change-Number: 36860 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newchange

1 year, 1 month

1
0
0 0

[S] Change in simtrace2[master]: RFC: card emu config: add feature sim polarity

by lynxis lazus

lynxis lazus has uploaded this change for review. ( https://gerrit.osmocom.org/c/simtrace2/+/36859?usp=email ) Change subject: RFC: card emu config: add feature sim polarity ...................................................................... RFC: card emu config: add feature sim polarity Change-Id: I36a8acb74ad7616205af68b1626b45ab79d7cb02 --- M firmware/libcommon/include/simtrace_prot.h 1 file changed, 15 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/simtrace2 refs/changes/59/36859/1 diff --git a/firmware/libcommon/include/simtrace_prot.h b/firmware/libcommon/include/simtrace_prot.h index d1eb89b..209a5d1 100644 --- a/firmware/libcommon/include/simtrace_prot.h +++ b/firmware/libcommon/include/simtrace_prot.h @@ -260,6 +260,7 @@ /* enable/disable the generation of DO_STATUS on IRQ endpoint */ #define CEMU_FEAT_F_STATUS_IRQ 0x00000001 +#define CEMU_FEAT_F_POLARITY 0x00000002 /* SIMTRACE_MSGT_BD_CEMU_CONFIG */ struct cardemu_usb_msg_config { @@ -267,6 +268,11 @@ uint32_t features; /* the selected slot number (if an external mux is present) */ uint8_t slot_mux_nr; + /* sim present. Bit 1 is valid bit. Bit 0 gpio value when sim present + * When sim is present, set sim_present gpio to high -> 0x03 + * When sim is present, set sim_present gpio to low -> 0x02 + */ + uint8_t sim_polarity; } __attribute__ ((packed)); /*********************************************************************** -- To view, visit https://gerrit.osmocom.org/c/simtrace2/+/36859?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: simtrace2 Gerrit-Branch: master Gerrit-Change-Id: I36a8acb74ad7616205af68b1626b45ab79d7cb02 Gerrit-Change-Number: 36859 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-MessageType: newchange

1 year, 1 month

1
0
0 0

2025

2024

2023

2022

gerrit-log May 2024