Change in osmo-ttcn3-hacks[master]: mgw: Drop unneeded mp_enable_conn_timeout_test

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Sun Jan 5 11:21:07 UTC 2020


laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/16727 )

Change subject: mgw: Drop unneeded mp_enable_conn_timeout_test
......................................................................

mgw: Drop unneeded mp_enable_conn_timeout_test

Latest osmo-mgw release is 1.7.0, so this param is not needed at all.
Furthermore, the config can be moved to .cfg.

Change-Id: I537c0f5fd6f9e18e111c773c0e42e5f1120ce2f4
---
M mgw/MGCP_Test.ttcn
M mgw/osmo-mgw.cfg
2 files changed, 2 insertions(+), 10 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/mgw/MGCP_Test.ttcn b/mgw/MGCP_Test.ttcn
index b46b178..77eed4b 100644
--- a/mgw/MGCP_Test.ttcn
+++ b/mgw/MGCP_Test.ttcn
@@ -69,10 +69,6 @@
 		charstring mp_remote_ip := "127.0.0.1";
 		PortNumber mp_local_rtp_port_base := 10000;
 		PortNumber mp_local_osmux_port := 1985;
-
-		/* Whether to enable conn-timeout tests. Can be dropped completely and enabled unconditionally once new
-		 * version of osmo-mgw is released (current version: 1.5.0) */
-		boolean mp_enable_conn_timeout_test := true;
 	}
 
 	private function f_vty_enable_osmux(boolean osmux_on) runs on dummy_CT {
@@ -95,9 +91,6 @@
 		f_vty_transceive(MGWVTY, "enable");
 
 		f_vty_enable_osmux(osmux_on);
-		if (mp_enable_conn_timeout_test) {
-			f_vty_config(MGWVTY, "mgcp", "conn-timeout 0");
-		}
 	}
 
 	private function f_rtpem_init(inout RTP_Emulation_CT comp_ref, integer i)
@@ -2199,8 +2192,6 @@
 		execute(TC_amr_oa_oa_rtp_conversion());
 		execute(TC_amr_bwe_bwe_rtp_conversion());
 
-		if (mp_enable_conn_timeout_test) {
-			execute(TC_conn_timeout());
-		}
+		execute(TC_conn_timeout());
 	}
 }
diff --git a/mgw/osmo-mgw.cfg b/mgw/osmo-mgw.cfg
index b27a50c..edc8190 100644
--- a/mgw/osmo-mgw.cfg
+++ b/mgw/osmo-mgw.cfg
@@ -31,6 +31,7 @@
   sdp audio-payload name GSM
   sdp audio-payload send-ptime
   sdp audio-payload send-name
+  conn-timeout 0
   loop 0
   number endpoints 31
   allow-transcoding

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/16727
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I537c0f5fd6f9e18e111c773c0e42e5f1120ce2f4
Gerrit-Change-Number: 16727
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200105/5ef63944/attachment.htm>


More information about the gerrit-log mailing list