pespin has uploaded this change for review. (
https://gerrit.osmocom.org/c/libosmo-sigtran/+/38426?usp=email )
Change subject: ss7 vty: Limit route priority range to 1-9
......................................................................
ss7 vty: Limit route priority range to 1-9
This matches route priority ranges used by other vendors.
Change-Id: I36ceee5d7cb1b2287c4a2d7ad67f478843300a0b
---
M src/osmo_ss7_vty.c
M tests/vty/osmo_stp_test.vty
2 files changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/26/38426/1
diff --git a/src/osmo_ss7_vty.c b/src/osmo_ss7_vty.c
index 356090c..e3142c7 100644
--- a/src/osmo_ss7_vty.c
+++ b/src/osmo_ss7_vty.c
@@ -74,7 +74,7 @@
QOS_CLASS_RANGE_HELP_STR \
"Default QoS Class (0)\n"
-#define ROUTE_PRIO_RANGE_STR "<0-4294967295>"
+#define ROUTE_PRIO_RANGE_STR "<1-9>"
#define ROUTE_PRIO_RANGE_HELP_STR "Priority\n"
#define ROUTE_PRIO_VAR_STR "(" ROUTE_PRIO_RANGE_STR "|default)"
#define ROUTE_PRIO_VAR_HELP_STR \
diff --git a/tests/vty/osmo_stp_test.vty b/tests/vty/osmo_stp_test.vty
index 7967216..cb24560 100644
--- a/tests/vty/osmo_stp_test.vty
+++ b/tests/vty/osmo_stp_test.vty
@@ -570,7 +570,7 @@
OsmoSTP(config-cs7)# route-table system
OsmoSTP(config-cs7-rt)# list
...
- update route POINT_CODE MASK linkset LS_NAME [priority]
[(<0-4294967295>|default)] [qos-class] [(<0-7>|default)]
+ update route POINT_CODE MASK linkset LS_NAME [priority] [(<1-9>|default)]
[qos-class] [(<0-7>|default)]
remove route POINT_CODE MASK
OsmoSTP(config-cs7-rt)# update route 3.2.1 7.255.7 linkset my-ass
OsmoSTP(config-cs7-rt)# show running-config
--
To view, visit
https://gerrit.osmocom.org/c/libosmo-sigtran/+/38426?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I36ceee5d7cb1b2287c4a2d7ad67f478843300a0b
Gerrit-Change-Number: 38426
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>