pespin has submitted this change. (
https://gerrit.osmocom.org/c/libosmo-sigtran/+/42027?usp=email )
Change subject: IPSP: Allow vty-configuration of IPSP role "ASPs"
......................................................................
IPSP: Allow vty-configuration of IPSP role "ASPs"
We should now have everything in place to support IPSP-SE, let's
accept enabling IPSP role via the VTY.
Change-Id: I0f0fa881471b87a4bb82a0211c6fb6e4c50b48ec
---
M src/ss7_asp_vty.c
1 file changed, 2 insertions(+), 5 deletions(-)
Approvals:
Jenkins Builder: Verified
osmith: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
laforge: Looks good to me, but someone else must approve
diff --git a/src/ss7_asp_vty.c b/src/ss7_asp_vty.c
index d54eb01..b7af9c9 100644
--- a/src/ss7_asp_vty.c
+++ b/src/ss7_asp_vty.c
@@ -414,15 +414,12 @@
{
struct osmo_ss7_asp *asp = vty->index;
- if (!strcmp(argv[0], "ipsp")) {
- vty_out(vty, "IPSP role isn't supported yet%s", VTY_NEWLINE);
- return CMD_WARNING;
- }
-
if (!strcmp(argv[0], "sg"))
asp->cfg.role = OSMO_SS7_ASP_ROLE_SG;
else if (!strcmp(argv[0], "asp"))
asp->cfg.role = OSMO_SS7_ASP_ROLE_ASP;
+ else if (!strcmp(argv[0], "ipsp"))
+ asp->cfg.role = OSMO_SS7_ASP_ROLE_IPSP;
else
OSMO_ASSERT(0);
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/42027?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I0f0fa881471b87a4bb82a0211c6fb6e4c50b48ec
Gerrit-Change-Number: 42027
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
