osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/14601 )
Change subject: vty: allow 0 as conn-timeout to disable it ...................................................................... Patch Set 3: Code-Review+2 (1 comment) Thanks for all the reviews! > Is there any 'mgcp_vty_reference.xml'? If yes, please also update it. I'll do that in a follow-up patch. Giving +2, because of >2x +1 (see https://osmocom.org/projects/cellular-infrastructure/wiki/Gerrit#Voting-Rules-for-merging-a-patch-to-master ). https://gerrit.osmocom.org/#/c/14601/1/src/libosmo-mgcp/mgcp_vty.c File src/libosmo-mgcp/mgcp_vty.c: https://gerrit.osmocom.org/#/c/14601/1/src/libosmo-mgcp/mgcp_vty.c@1392 PS1, Line 1392: DEFUN(cfg_mgcp_no_conn_timeout, > We should agree on one way to do it, because "no e1_line 0 keepalive" is > still what libosmo-abis wan […] I would also like to have this consistent, but unfortunately we have both versions already. Here are two examples of <0-...: osmo-hlr/src/hlr_vty.c: "ncss-guard-timeout <0-255>", osmo-msc/src/libmsc/msc_vty.c: "mncc guard-timeout <0-255>", So I'll go with the <0-... version here instead of changing it again. Maybe it makes sense to propose a standard way of doing it (on the ML) and then deprecating the other one? -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/14601 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I7dee7dafaaf4bb93fd692ea06b52b9e012beac6d Gerrit-Change-Number: 14601 Gerrit-PatchSet: 3 Gerrit-Owner: osmith <[email protected]> Gerrit-Reviewer: Hoernchen <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <[email protected]> Gerrit-Reviewer: laforge <[email protected]> Gerrit-Reviewer: osmith <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-Comment-Date: Thu, 27 Jun 2019 07:23:39 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: pespin <[email protected]> Comment-In-Reply-To: osmith <[email protected]> Comment-In-Reply-To: Hoernchen <[email protected]> Gerrit-MessageType: comment
