- gerrit-log - lists.osmocom.org

[M] Change in pysim[master]: [5/6] personalization: refactor AlgorithmID, K, Opc

by neels

Attention is currently required from: laforge. neels has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/pysim/+/39745?usp=email ) Change subject: [5/6] personalization: refactor AlgorithmID, K, Opc ...................................................................... Patch Set 4: (1 comment) File pySim/esim/saip/personalization.py: https://gerrit.osmocom.org/c/pysim/+/39745/comment/d2c44a7d_af409c41?usp=em… : PS3, Line 551: allow_len = int(128/8) # length in bytes (from BinaryParam) > just look at the pysim-shell code for setting the values on a sysmoISIM card: […] I'm sorry, I do not comprehend this code snippet at all... Can you decipher for me, I just need to know: what valid lengths can any K have for TUAK (and any other algo)? (matching a K length to a chosen algo will have to be done by a later integrity check, because there currently is no relation between separate ConfigurableParameters) We could also merge this patch now, because it supports Milenage, and fix TUAK later...? -- To view, visit https://gerrit.osmocom.org/c/pysim/+/39745?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I6296fdcfd5d2ed313c4aade57ff43cc362375848 Gerrit-Change-Number: 39745 Gerrit-PatchSet: 4 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Sat, 19 Apr 2025 22:13:23 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de> Comment-In-Reply-To: laforge <laforge(a)osmocom.org>

2 months, 3 weeks

1
0
0 0

[M] Change in pysim[master]: [4/6] personalization: refactor Pin, Adm

by neels

Attention is currently required from: laforge. neels has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/pysim/+/39744?usp=email ) Change subject: [4/6] personalization: refactor Pin, Adm ...................................................................... Patch Set 4: (1 comment) File pySim/esim/saip/personalization.py: https://gerrit.osmocom.org/c/pysim/+/39744/comment/6ef7717e_f3b9e05f?usp=em… : PS3, Line 408: return (pe for pe in l if pe.type == wanted_type) > * I think from an ISO7816 and UICC point of view it's sadly not correct to limit to MF. […] This patch is trying to not modify prior behavior. The aim is to remove code dup between class Pin and class AppPin. - Prior code happens to limit to MF for PIN1. - Prior API happens to produce a mix of ProfileElementList and plain py lists. If these prior conditions are wrong, I would like to fix them outside of this patch. I created issue https://osmocom.org/issues/6770 for the MF question. and https://osmocom.org/issues/6771 for the lists question. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/39744?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I54aef10b6d4309398d4b779a3740a7d706d68603 Gerrit-Change-Number: 39744 Gerrit-PatchSet: 4 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Sat, 19 Apr 2025 21:57:13 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de> Comment-In-Reply-To: laforge <laforge(a)osmocom.org>

2 months, 3 weeks

1
0
0 0

[M] Change in pysim[master]: [2/6] personalization: refactor ConfigurableParameter, Iccid, Imsi

by neels

Attention is currently required from: dexter, fixeria, laforge. neels has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/pysim/+/39742?usp=email ) Change subject: [2/6] personalization: refactor ConfigurableParameter, Iccid, Imsi ...................................................................... Patch Set 7: (3 comments) File pySim/esim/saip/personalization.py: https://gerrit.osmocom.org/c/pysim/+/39742/comment/2c294888_fba24441?usp=em… : PS6, Line 117: If self.value is None, first call self.validate() to generate a sanitized self.value from self.input_value. > I see how this creates a misunderstanding: I always write API docs in the "imperative mood", which i […] Done File pySim/esim/saip/personalization.py: https://gerrit.osmocom.org/c/pysim/+/39742/comment/0538f7c2_358f139f?usp=em… : PS7, Line 67: = > shouldn't that be a double-equals (also below)? to me it looks like an assignment rather than a com […] wow, a log string in a printf in example code in an api comment ... =) but ok https://gerrit.osmocom.org/c/pysim/+/39742/comment/d5946cd5_28545b40?usp=em… : PS7, Line 100: allow_types = (str, int, ) > Does the comma with empty element imply that None is a vaid input value? In that case, I'd prefer a […] no, the trailing comma does not add a third item. It's just that i like trailing commas for easier editing and use them where i can. Also I have this behavior from: a single-entry list *does* work as my_list = [foo], but tuple braces don't. This makes a single-entry tuple: my_tuple = (23, ) another_tuple = tuple((23, )) This not: not_my_tuple = (23) # an int also_not = tuple(23) # TypeError -- To view, visit https://gerrit.osmocom.org/c/pysim/+/39742?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I6522be4c463e34897ca9bff2309b3706a88b3ce8 Gerrit-Change-Number: 39742 Gerrit-PatchSet: 7 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Sat, 19 Apr 2025 21:17:47 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de> Comment-In-Reply-To: laforge <laforge(a)osmocom.org> Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>

2 months, 3 weeks

1
0
0 0

[M] Change in pysim[master]: [1/6] personalization: refactor: drop ClassVarMeta use

by neels

Attention is currently required from: fixeria, laforge. Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/pysim/+/39741?usp=email to look at the new patch set (#9). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: [1/6] personalization: refactor: drop ClassVarMeta use ...................................................................... [1/6] personalization: refactor: drop ClassVarMeta use Drop the ClassVarMeta/metaclass/ABCMeta stuff -- it doesn't seem to serve any purpose that is not similarly achieved with plain python inheritance. Upcoming patches will use normal inheritance a lot more. Note that most use of the ClassVarMeta was in the SdKey subclasses, and that these currently don't actually work. See the fix in patch I07dfc378705eba1318e9e8652796cbde106c6a52 . name: set a default name from the python class, as ClassVarMeta did. Also allow setting an explicit string as name instead, per subclass implementation (see I31f390d634e58c384589c50a33ca45d6f86d4e10). Related: SYS#6768 Change-Id: I60ea8fd11fb438ec90ddb08b17b658cbb789c051 --- M pySim/esim/saip/personalization.py 1 file changed, 149 insertions(+), 82 deletions(-) git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/41/39741/9 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/39741?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I60ea8fd11fb438ec90ddb08b17b658cbb789c051 Gerrit-Change-Number: 39741 Gerrit-PatchSet: 9 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>

2 months, 3 weeks

1
0
0 0

[M] Change in pysim[master]: [2/6] personalization: refactor ConfigurableParameter, Iccid, Imsi

by neels

Attention is currently required from: dexter, fixeria, laforge, neels. Hello Jenkins Builder, dexter, laforge, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/pysim/+/39742?usp=email to look at the new patch set (#8). The following approvals got outdated and were removed: Code-Review+1 by laforge, Verified+1 by Jenkins Builder Change subject: [2/6] personalization: refactor ConfigurableParameter, Iccid, Imsi ...................................................................... [2/6] personalization: refactor ConfigurableParameter, Iccid, Imsi Main points/rationales of the refactoring, details below: 1) common validation implementation 2) offer classmethods The new features are optional, and will be heavily used by batch personalization patches coming soon. Implement Iccid and Imsi to use the new way, with a common abstract DecimalParam implementation. So far leave the other parameter classes working as they always did, to follow suit in subsequent commits. Details: 1) common validation implementation: There are very common validation steps in the various parameter implementations. It is more convenient and much more readable to implement those once and set simple validation parameters per subclass. So there now is a validate_val() classmethod, which subclasses can use as-is to apply the validation parameters -- or subclasses can override their cls.validate_val() for specialized validation. (Those subclasses that this patch doesn't touch still override the self.validate() instance method. Hence they still work as before this patch, but don't use the new common features yet.) 2) offer stateless classmethods: It is useful for... - batch processing of multiple profiles (in upcoming patches) and - user input validation to be able to have classmethods that do what self.validate() and self.apply() do, but do not modify any self.* members. So far the paradigm was to create a class instance to keep state about the value. This remains available, but in addition we make available the paradigm of a singleton that is stateless (the classmethods). Using self.validate() and self.apply() still work the same as before this patch, i.e. via self.input_value and self.value -- but in addition, there are now classmethods that don't touch self.* members. Related: SYS#6768 Change-Id: I6522be4c463e34897ca9bff2309b3706a88b3ce8 --- M pySim/esim/saip/personalization.py 1 file changed, 180 insertions(+), 34 deletions(-) git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/42/39742/8 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/39742?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I6522be4c463e34897ca9bff2309b3706a88b3ce8 Gerrit-Change-Number: 39742 Gerrit-PatchSet: 8 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de>

2 months, 3 weeks

1
0
0 0

[M] Change in pysim[master]: [1/6] personalization: refactor: drop ClassVarMeta use

by neels

Attention is currently required from: fixeria, laforge. Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/pysim/+/39741?usp=email to look at the new patch set (#8). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: [1/6] personalization: refactor: drop ClassVarMeta use ...................................................................... [1/6] personalization: refactor: drop ClassVarMeta use Drop the ClassVarMeta/metaclass/ABCMeta stuff -- it doesn't seem to serve any purpose that is not similarly achieved with plain python inheritance. Upcoming patches will use normal inheritance a lot more. Note that most use of the ClassVarMeta was in the SdKey subclasses, and that these currently don't actually work. See the fix in patch I07dfc378705eba1318e9e8652796cbde106c6a52 . name: set a default name from the python class, as ClassVarMeta did. Also allow setting an explicit string as name instead, per subclass implementation (see I31f390d634e58c384589c50a33ca45d6f86d4e10). Related: SYS#6768 Change-Id: I60ea8fd11fb438ec90ddb08b17b658cbb789c051 --- M pySim/esim/saip/personalization.py 1 file changed, 149 insertions(+), 82 deletions(-) git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/41/39741/8 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/39741?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I60ea8fd11fb438ec90ddb08b17b658cbb789c051 Gerrit-Change-Number: 39741 Gerrit-PatchSet: 8 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>

2 months, 3 weeks

1
0
0 0

[M] Change in pysim[master]: [1/6] personalization: refactor: drop ClassVarMeta use

by neels

Attention is currently required from: fixeria, laforge. neels has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/pysim/+/39741?usp=email ) Change subject: [1/6] personalization: refactor: drop ClassVarMeta use ...................................................................... Patch Set 7: (1 comment) Patchset: PS6: > CSV matching: I have other use cases than you do. […] In this new patch set, the name and camel_to_snake() are back (they were lost by accident from branch rebasing). reasons in my decision against a meta class: - i don't understand what purpose a meta class serves. - fixeria's example shows that the shortest way to achieve class attributes is ~~~ class Foo: a = 1 b = 2 ~~~ I like this plain python better than kwargs juggling. - upcoming patches add a lot of class attributes, and i would like to consistently use one way. I chose the "normal python" way, so as a first step, this patch drops the meta class. - name: i would like to set explicit names like 'IMSI' instead of 'imsi', Hence i change to using an explicit 'name' attribute instead of getattr magic. When no explicit name is set, all still behaves as before this patch. I hope that I've made my point, so i am now, in a hopeful way, resolving this thread. Please -1 again if i should change it. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/39741?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I60ea8fd11fb438ec90ddb08b17b658cbb789c051 Gerrit-Change-Number: 39741 Gerrit-PatchSet: 7 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Sat, 19 Apr 2025 20:59:45 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de> Comment-In-Reply-To: laforge <laforge(a)osmocom.org> Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>

2 months, 3 weeks

1
0
0 0

[M] Change in pysim[master]: [1/6] personalization: refactor: drop ClassVarMeta use

by neels

Attention is currently required from: fixeria, laforge. neels has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/pysim/+/39741?usp=email ) Change subject: [1/6] personalization: refactor: drop ClassVarMeta use ...................................................................... Patch Set 7: (1 comment) File pySim/esim/saip/personalization.py: https://gerrit.osmocom.org/c/pysim/+/39741/comment/2f8fe753_c0f31009?usp=em… : PS7, Line 155: class SdKeyScp80_01(SdKey): > I'm still somewhat doubtful that this is an improvement. […] (merging with the other "thread") -- To view, visit https://gerrit.osmocom.org/c/pysim/+/39741?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I60ea8fd11fb438ec90ddb08b17b658cbb789c051 Gerrit-Change-Number: 39741 Gerrit-PatchSet: 7 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Sat, 19 Apr 2025 20:37:55 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <laforge(a)osmocom.org>

2 months, 3 weeks

1
0
0 0

[M] Change in ...osmo-s1gw[master]: Bump version: 0.2.0-61-g09ac383 → 0.3.0

by fixeria

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40091?usp=email ) Change subject: Bump version: 0.2.0-61-g09ac383 → 0.3.0 ...................................................................... Bump version: 0.2.0-61-g09ac383 → 0.3.0 Change-Id: Ibd805fb754ed210e07f2a7fea1a14f610e35e520 --- M debian/changelog 1 file changed, 66 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/debian/changelog b/debian/changelog index 35ec97f..90691c8 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,69 @@ +osmo-s1gw (0.3.0) unstable; urgency=medium + + * s1ap_proxy: merge process_pdu_safe() with process_pdu() + * s1ap_proxy: do not expose {en,de}code_pdu() API + * s1ap_proxy: turn this module into a gen_server + * s1ap_proxy: add E-RAB introspection API + * erab_fsm: rename erab_release_req/1 -> erab_release_cmd/1 + * erab_fsm: add erab_release/2 and erab_release_ind/1 + * s1ap_proxy: properly handle E-RAB RELEASE IND + * erab_fsm: add all-state event handler + * erab_fsm: add fetch_info/1 for state introspection + * sctp_server: add type hints + * sctp_server: cosmetic: use ?FUNCTION_NAME in send_data/2 + * sctp_server: pass MME addr/port to sctp_proxy as opaque data + * sctp_server: cosmetic: rename State -> S + * sctp_server: cosmetic: make #server_state last argument + * sctp_server: pass #server_state to client_{add,del}() + * sctp_server: make it abstract from sctp_proxy logic + * sctp_server: add fetch_conn_list/0 for state introspection + * sctp_proxy: add all-state event handler + * sctp_proxy: add fetch_info/1 for state introspection + * s1ap_proxy: use erab_uid() to log unique E-RAB ID + * s1ap_proxy: add a TODO about an optional IE + * s1ap_proxy: rework handle_ie/2 into handle_ie/3 + * s1ap_proxy: handle_ies(): reorder arguments + * s1ap_proxy: handle_ies(): pass IEI path to handle_ie() + * s1ap_proxy: handle_ies(): also handle #'ProtocolExtensionField' + * erab_fsm: store UID, expose it via fetch_info/1 + * erab_fsm: session_modify(): split PFCP PDU handling + * s1ap_proxy: cosmetic: fix wrong arity in comment + * s1ap_proxy: fix IEI path leak in handle_ies/4 + * erab_fsm: fix copy-paste in a log message + * erab_fsm: add more logging to clarify timeout events + * rebar.config: remove unneeded parse_transform + * erab_fsm: add pretty printer for F-TEID + * s1ap_proxy: handle UE CONTEXT RELEASE PDUs + * osmo_s1gw.app.src: set 'vsn' to 'semver' + * rebar.config: cosmetic: use spaces, not tabs + * rebar.config: relx: fix release name (must be an atom) + * rebar.config: relx: generate the version automatically + * {erab_fsm,sctp_proxy}: handle_event/4: fix return value + * {erab_fsm,sctp_proxy}: handle_event/4: reply to unknown calls + * erab_fsm: make erab_release_{cmd,ind}/1 non-blocking + * erab_fsm: handle RELEASE.{cmd,ind} in state erab_wait_setup_rsp + * erab_fsm: handle RELEASE.ind in state erab_wait_release_rsp + * erab_fsm: implement E-RAB MODIFY Req/Rsp procedure + * erab_fsm: implement E-RAB MODIFY Ind/Cnf procedure + * erab_fsm: pretty-print U2CM and U2AM during MODIFY Req/Ind + * s1ap_proxy: catch exceptions in handle_pdu/2 + * s1ap_proxy: handle E-RAB MODIFY REQ/RSP + * s1ap_proxy: handle E-RAB MODIFICATION IND/CNF + * s1ap_proxy: increment *_IN_PKT_* counters on exception + * sctp_server: invalidate handler's Pid on 'EXIT' + * sctp_{server,proxy}: log SCTP TSN, SID, SSN, and data len + * sctp_common: new module with common SCTP/S1AP API + * s1gw_metrics: use lists:filter/2 to create counters/gauges + * sctp_{client,server}: disable Nagle by setting sctp_nodelay + * s1gw_metrics: make exometer_report_statsd less verbose + * sctp_{client,server}: add and use sctp_common:parse_addr/1 + * sctp_{client,server}: accept arguments in form of a map + * sctp_{client,server}: allow passing additional sockopts + * sctp_{client,server}: make sctp_nodelay configurable + * sctp_{client,server}: set sndbuf/recbuf explicitly + + -- Vadim Yanitskiy <vyanitskiy(a)sysmocom.de> Fri, 18 Apr 2025 02:56:33 +0700 + osmo-s1gw (0.2.0) unstable; urgency=medium [ Vadim Yanitskiy ] -- To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40091?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: erlang/osmo-s1gw Gerrit-Branch: master Gerrit-Change-Id: Ibd805fb754ed210e07f2a7fea1a14f610e35e520 Gerrit-Change-Number: 40091 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>

2 months, 3 weeks

1
0
0 0

[S] Change in osmocom-bb[master]: trx_toolkit/burst_fwd: Switch BurstForwarder to cdef class

by laforge

Attention is currently required from: kirr. laforge has posted comments on this change by kirr. ( https://gerrit.osmocom.org/c/osmocom-bb/+/40065?usp=email ) Change subject: trx_toolkit/burst_fwd: Switch BurstForwarder to cdef class ...................................................................... Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/40065?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: I8e5fe20ff5a9a0fbbb8ce82a7776837de1219e13 Gerrit-Change-Number: 40065 Gerrit-PatchSet: 3 Gerrit-Owner: kirr <kirr(a)nexedi.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: kirr <kirr(a)nexedi.com> Gerrit-Comment-Date: Sat, 19 Apr 2025 16:53:25 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

2 months, 3 weeks

1
0
0 0

2025

2024

2023

2022

gerrit-log