Attention is currently required from: fixeria, osmith.

pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/38375?usp=email )

Change subject: ss7 vty: Fix insert order of routes with higher priority ......................................................................

Patch Set 3:

(4 comments)

File src/osmo_ss7.c:

https://gerrit.osmocom.org/c/libosmo-sigtran/+/38375/comment/9dd43fb6_8ce854... : PS3, Line 664: insertd

inserted

Done

https://gerrit.osmocom.org/c/libosmo-sigtran/+/38375/comment/8bc371a9_66b09d... : PS3, Line 736: return -ENODEV;

other error returns above log a message. […]

I prefer keeping it this way since the typology of return is different here. The others are really unexpected state problems which deserve logging. This one here, on the other hand, is just the lookup failing and makes more sense that the caller logs whatever it pleases. Otherwise for instance if some VTY user types in some asp with a typo we end up with a error in log which really doesn't have any kind of problem during the normal operation of the program.

https://gerrit.osmocom.org/c/libosmo-sigtran/+/38375/comment/706a655f_eca277... : PS3, Line 857: LOGSS7(rtbl->inst, LOGL_ERROR, "Refusing to create duplicate route: "

maybe be more specific here: "Refusing to create route with existing linkset name"

Done

File src/osmo_ss7_vty.c:

https://gerrit.osmocom.org/c/libosmo-sigtran/+/38375/comment/56e15297_690564... : PS3, Line 401: find

set?

The real only expected cause of this failing here is that the linkset cannot be found, so it's far more informative as it is now.