March 2025 - gerrit-log - lists.osmocom.org

[XS] Change in osmo-pcu[master]: Drop function declaration for non longer existing function
by laforge 05 Mar '25

05 Mar '25

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/39686?usp=email ) Change subject: Drop function declaration for non longer existing function ...................................................................... Drop function declaration for non longer existing function Drop use of deprecated vty is_config_node() cb This callback was drepecated and is not ever called since libosmocore.git 70ce871532ab21955e0955d7e230eae65438f047 (release 1.3.0). See also libosmocore.git d31de237582f6fe3315d61bb9a488d4cda92654e. Change-Id: I999ac55fca21cf942876bc19c19f0eda2754501b Fixes: 1ec29c7324e69a6f294704fb8eeb093a33085aba --- M src/pcu_vty.h 1 file changed, 0 insertions(+), 3 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/src/pcu_vty.h b/src/pcu_vty.h index 45d3227..8178e11 100644 --- a/src/pcu_vty.h +++ b/src/pcu_vty.h @@ -9,9 +9,6 @@ PCU_NODE = _LAST_OSMOVTY_NODE + 1, }; -int pcu_vty_go_parent(struct vty *vty); -int pcu_vty_is_config_node(struct vty *vty, int node); - int pcu_vty_init(); extern struct vty_app_info pcu_vty_info; -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/39686?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: I999ac55fca21cf942876bc19c19f0eda2754501b Gerrit-Change-Number: 39686 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

1 0

[XS] Change in osmo-pcu[master]: Drop function declaration for non longer existing function
by laforge 05 Mar '25

05 Mar '25

Attention is currently required from: pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-pcu/+/39686?usp=email ) Change subject: Drop function declaration for non longer existing function ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/39686?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: I999ac55fca21cf942876bc19c19f0eda2754501b Gerrit-Change-Number: 39686 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Wed, 05 Mar 2025 14:55:31 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in osmo-sip-connector[master]: Drop use of deprecated vty is_config_node() cb
by laforge 05 Mar '25

05 Mar '25

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sip-connector/+/39688?usp=email ) Change subject: Drop use of deprecated vty is_config_node() cb ...................................................................... Drop use of deprecated vty is_config_node() cb This callback was drepecated and is not ever called since libosmocore.git 70ce871532ab21955e0955d7e230eae65438f047 (release 1.3.0). See also libosmocore.git d31de237582f6fe3315d61bb9a488d4cda92654e. Change-Id: I3809f9c0529004ff117e9e79ee81d30dca6e9bc4 --- M src/vty.c 1 file changed, 1 insertion(+), 8 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/vty.c b/src/vty.c index ea76d46..f7fa041 100644 --- a/src/vty.c +++ b/src/vty.c @@ -33,7 +33,6 @@ struct app_config g_app; static int mncc_vty_go_parent(struct vty *vty); -static int mncc_vty_is_config_node(struct vty *vty, int node); static struct cmd_node sip_node = { SIP_NODE, @@ -57,7 +56,6 @@ .name = "OsmoSIPcon", .version = PACKAGE_VERSION, .go_parent_cb = mncc_vty_go_parent, - .is_config_node = mncc_vty_is_config_node, .copyright = "GNU AGPLv3+\n", }; @@ -71,7 +69,7 @@ vty->index = NULL; break; default: - if (mncc_vty_is_config_node(vty, vty->node)) + if (vty->node >= SIP_NODE) vty->node = CONFIG_NODE; else vty->node = ENABLE_NODE; @@ -81,11 +79,6 @@ return vty->node; } -static int mncc_vty_is_config_node(struct vty *vty, int node) -{ - return node >= SIP_NODE; -} - static int config_write_sip(struct vty *vty) { vty_out(vty, "sip%s", VTY_NEWLINE); -- To view, visit https://gerrit.osmocom.org/c/osmo-sip-connector/+/39688?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sip-connector Gerrit-Branch: master Gerrit-Change-Id: I3809f9c0529004ff117e9e79ee81d30dca6e9bc4 Gerrit-Change-Number: 39688 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

1 0

[XS] Change in osmo-sip-connector[master]: Drop use of deprecated vty is_config_node() cb
by laforge 05 Mar '25

05 Mar '25

Attention is currently required from: pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-sip-connector/+/39688?usp=email ) Change subject: Drop use of deprecated vty is_config_node() cb ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-sip-connector/+/39688?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-sip-connector Gerrit-Branch: master Gerrit-Change-Id: I3809f9c0529004ff117e9e79ee81d30dca6e9bc4 Gerrit-Change-Number: 39688 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Wed, 05 Mar 2025 14:55:20 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in osmo-msc[master]: Drop use of deprecated vty is_config_node() cb
by laforge 05 Mar '25

05 Mar '25

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/39684?usp=email ) Change subject: Drop use of deprecated vty is_config_node() cb ...................................................................... Drop use of deprecated vty is_config_node() cb This callback was drepecated and is not ever called since libosmocore.git 70ce871532ab21955e0955d7e230eae65438f047 (release 1.3.0). See also libosmocore.git d31de237582f6fe3315d61bb9a488d4cda92654e. Change-Id: Id6620095af153781cde926458f54fed5bd51a9e7 --- M src/osmo-msc/msc_main.c 1 file changed, 0 insertions(+), 19 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/src/osmo-msc/msc_main.c b/src/osmo-msc/msc_main.c index 2238e2b..2435029 100644 --- a/src/osmo-msc/msc_main.c +++ b/src/osmo-msc/msc_main.c @@ -354,29 +354,10 @@ return vty->node; } -static int msc_vty_is_config_node(struct vty *vty, int node) -{ - /* Check if libosmo-sigtran declares the node in - * question as config node */ - if (osmo_ss7_is_config_node(vty, node)) - return 1; - - switch (node) { - /* add items that are not config */ - case SUBSCR_NODE: - case CONFIG_NODE: - return 0; - - default: - return 1; - } -} - static struct vty_app_info msc_vty_info = { .name = "OsmoMSC", .version = PACKAGE_VERSION, .go_parent_cb = msc_vty_go_parent, - .is_config_node = msc_vty_is_config_node, }; #define DEFAULT_M3UA_LOCAL_IP "localhost" -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/39684?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Id6620095af153781cde926458f54fed5bd51a9e7 Gerrit-Change-Number: 39684 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

1 0

[S] Change in osmo-msc[master]: Drop use of deprecated vty is_config_node() cb
by laforge 05 Mar '25

05 Mar '25

Attention is currently required from: pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-msc/+/39684?usp=email ) Change subject: Drop use of deprecated vty is_config_node() cb ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/39684?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Id6620095af153781cde926458f54fed5bd51a9e7 Gerrit-Change-Number: 39684 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Wed, 05 Mar 2025 14:55:12 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in osmo-ggsn[master]: Drop use of deprecated vty is_config_node() cb
by laforge 05 Mar '25

05 Mar '25

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ggsn/+/39685?usp=email ) Change subject: Drop use of deprecated vty is_config_node() cb ...................................................................... Drop use of deprecated vty is_config_node() cb This callback was drepecated and is not ever called since libosmocore.git 70ce871532ab21955e0955d7e230eae65438f047 (release 1.3.0). See also libosmocore.git d31de237582f6fe3315d61bb9a488d4cda92654e. Change-Id: Ief57c4ce98055e5063aafc53cb8330d3bd95d323 --- M ggsn/ggsn_vty.c 1 file changed, 0 insertions(+), 12 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/ggsn/ggsn_vty.c b/ggsn/ggsn_vty.c index b0f07fe..534b3d2 100644 --- a/ggsn/ggsn_vty.c +++ b/ggsn/ggsn_vty.c @@ -1124,17 +1124,6 @@ return 0; } -static int ggsn_vty_is_config_node(struct vty *vty, int node) -{ - switch (node) { - case GGSN_NODE: - case APN_NODE: - return 1; - default: - return 0; - } -} - static int ggsn_vty_go_parent(struct vty *vty) { switch (vty->node) { @@ -1174,5 +1163,4 @@ .version = PACKAGE_VERSION, .copyright = ggsn_copyright, .go_parent_cb = ggsn_vty_go_parent, - .is_config_node = ggsn_vty_is_config_node, }; -- To view, visit https://gerrit.osmocom.org/c/osmo-ggsn/+/39685?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ggsn Gerrit-Branch: master Gerrit-Change-Id: Ief57c4ce98055e5063aafc53cb8330d3bd95d323 Gerrit-Change-Number: 39685 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

1 0

[S] Change in osmo-ggsn[master]: Drop use of deprecated vty is_config_node() cb
by laforge 05 Mar '25

05 Mar '25

Attention is currently required from: pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ggsn/+/39685?usp=email ) Change subject: Drop use of deprecated vty is_config_node() cb ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ggsn/+/39685?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ggsn Gerrit-Branch: master Gerrit-Change-Id: Ief57c4ce98055e5063aafc53cb8330d3bd95d323 Gerrit-Change-Number: 39685 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Wed, 05 Mar 2025 14:55:01 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in osmo-mgw[master]: Drop use of deprecated vty is_config_node() cb
by laforge 05 Mar '25

05 Mar '25

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39683?usp=email ) Change subject: Drop use of deprecated vty is_config_node() cb ...................................................................... Drop use of deprecated vty is_config_node() cb This callback was drepecated and is not ever called since libosmocore.git 70ce871532ab21955e0955d7e230eae65438f047 (release 1.3.0). See also libosmocore.git d31de237582f6fe3315d61bb9a488d4cda92654e. Change-Id: Ifdecb35f329c243222bc5d77f0a6d394c9413684 --- M src/osmo-mgw/mgw_main.c 1 file changed, 1 insertion(+), 13 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/osmo-mgw/mgw_main.c b/src/osmo-mgw/mgw_main.c index 0b66184..96e424f 100644 --- a/src/osmo-mgw/mgw_main.c +++ b/src/osmo-mgw/mgw_main.c @@ -249,17 +249,6 @@ return 0; } -int mgcp_vty_is_config_node(struct vty *vty, int node) -{ - switch (node) { - case CONFIG_NODE: - return 0; - - default: - return 1; - } -} - int mgcp_vty_go_parent(struct vty *vty) { switch (vty->node) { @@ -269,7 +258,7 @@ break; case MGCP_NODE: default: - if (mgcp_vty_is_config_node(vty, vty->node)) + if (vty->node != CONFIG_NODE) vty->node = CONFIG_NODE; else vty->node = ENABLE_NODE; @@ -310,7 +299,6 @@ .name = "OsmoMGW", .version = PACKAGE_VERSION, .go_parent_cb = mgcp_vty_go_parent, - .is_config_node = mgcp_vty_is_config_node, }; static const struct log_info_cat log_categories[] = { -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39683?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: Ifdecb35f329c243222bc5d77f0a6d394c9413684 Gerrit-Change-Number: 39683 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

1 0

[S] Change in osmo-mgw[master]: Drop use of deprecated vty is_config_node() cb
by laforge 05 Mar '25

05 Mar '25

Attention is currently required from: pespin. laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39683?usp=email ) Change subject: Drop use of deprecated vty is_config_node() cb ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39683?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: Ifdecb35f329c243222bc5d77f0a6d394c9413684 Gerrit-Change-Number: 39683 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Wed, 05 Mar 2025 14:54:54 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0