Pau Espin Pedrol has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/11526 )

Change subject: default-suites.conf: Make sure we use fr codec with IPA dyn TS
......................................................................

default-suites.conf: Make sure we use fr codec with IPA dyn TS

IPA style dynamic timeslots (TCH/F_PDCH) support only TCH/F and thus
only full rate codecs are to be used.

On the other hand, OSMO style dynamic timeslots (TCH/F_TCH/H_PDCH) can
use both full rate and half rate, so no need to be restrictive there.

Change-Id: I0039ef60b323ed72cfe00d8fd9e9287e9c82d49f
---
M example/default-suites.conf
A example/scenarios/cfg-codec-fr-any.conf
2 files changed, 20 insertions(+), 13 deletions(-)

Approvals:
  Jenkins Builder: Verified
  dexter: Looks good to me, but someone else must approve
  Pau Espin Pedrol: Looks good to me, approved



diff --git a/example/default-suites.conf b/example/default-suites.conf
index c95bda8..c7f2377 100644
--- a/example/default-suites.conf
+++ b/example/default-suites.conf
@@ -7,13 +7,13 @@
 - voice:sysmo+mod-bts0-ts-tchf+cfg-codec-fr3
 - voice:sysmo+mod-bts0-ts-tchh+cfg-codec-hr1
 - voice:sysmo+mod-bts0-ts-tchh+cfg-codec-hr3
-- voice:sysmo+mod-bts0-dynts-ipa
+- voice:sysmo+mod-bts0-dynts-ipa+cfg-codec-fr-any
 - voice:sysmo+mod-bts0-dynts-osmo
 - gprs:sysmo
 - gprs:sysmo+mod-bts0-dynts-ipa
 - gprs:sysmo+mod-bts0-dynts-osmo
-- dynts:sysmo+mod-bts0-dynts67-ipa+cfg-codec-fr1
-- dynts:sysmo+mod-bts0-dynts67-osmo+cfg-codec-fr1
+- dynts:sysmo+mod-bts0-dynts67-ipa+cfg-codec-fr-any
+- dynts:sysmo+mod-bts0-dynts67-osmo

 - nitb_sms:trx-b200
 - sms:trx-b200
@@ -24,15 +24,15 @@
 - voice:trx-b200+mod-bts0-ts-tchf+cfg-codec-fr3
 - voice:trx-b200+mod-bts0-ts-tchh+cfg-codec-hr1
 - voice:trx-b200+mod-bts0-ts-tchh+cfg-codec-hr3
-- voice:trx-b200+mod-bts0-dynts-ipa
+- voice:trx-b200+mod-bts0-dynts-ipa+cfg-codec-fr-any
 - voice:trx-b200+mod-bts0-dynts-osmo
 - voice:trx-b200+mod-bts0-numtrx2+mod-bts0-chanallocdescend
 - gprs:trx-b200
 - gprs:trx-b200+mod-bts0-dynts-ipa
 - gprs:trx-b200+mod-bts0-dynts-osmo
 - gprs:trx-b200+mod-bts0-numtrx2+mod-bts0-chanallocdescend
-- dynts:trx-b200+mod-bts0-dynts67-ipa+cfg-codec-fr1
-- dynts:trx-b200+mod-bts0-dynts67-osmo+cfg-codec-fr1
+- dynts:trx-b200+mod-bts0-dynts67-ipa+cfg-codec-fr-any
+- dynts:trx-b200+mod-bts0-dynts67-osmo

 - nitb_sms:trx-sysmocell5000
 - sms:trx-sysmocell5000
@@ -43,13 +43,13 @@
 - voice:trx-sysmocell5000+mod-bts0-ts-tchf+cfg-codec-fr3
 - voice:trx-sysmocell5000+mod-bts0-ts-tchh+cfg-codec-hr1
 - voice:trx-sysmocell5000+mod-bts0-ts-tchh+cfg-codec-hr3
-- voice:trx-sysmocell5000+mod-bts0-dynts-ipa
+- voice:trx-sysmocell5000+mod-bts0-dynts-ipa+cfg-codec-fr-any
 - voice:trx-sysmocell5000+mod-bts0-dynts-osmo
 - gprs:trx-sysmocell5000
 - gprs:trx-sysmocell5000+mod-bts0-dynts-ipa
 - gprs:trx-sysmocell5000+mod-bts0-dynts-osmo
-- dynts:trx-sysmocell5000+mod-bts0-dynts67-ipa+cfg-codec-fr1
-- dynts:trx-sysmocell5000+mod-bts0-dynts67-osmo+cfg-codec-fr1
+- dynts:trx-sysmocell5000+mod-bts0-dynts67-ipa+cfg-codec-fr-any
+- dynts:trx-sysmocell5000+mod-bts0-dynts67-osmo

 - nitb_sms:nanobts
 - sms:nanobts+band-1900
@@ -59,10 +59,10 @@
 - voice:nanobts+band-1900+mod-bts0-ts-tchf+cfg-codec-fr2
 - voice:nanobts+band-1900+mod-bts0-ts-tchf+cfg-codec-fr3
 - voice:nanobts+band-1900+mod-bts0-ts-tchh+cfg-codec-hr3
-- voice:nanobts+band-1900+mod-bts0-dynts-ipa
+- voice:nanobts+band-1900+mod-bts0-dynts-ipa+cfg-codec-fr-any
 - gprs:nanobts+band-1900
 - gprs:nanobts+band-1900+mod-bts0-dynts-ipa
-- dynts:nanobts+band-1900+mod-bts0-dynts67-ipa+cfg-codec-fr1
+- dynts:nanobts+band-1900+mod-bts0-dynts67-ipa+cfg-codec-fr-any

 - sms:nanobts+band-900
 - nitb_ussd:nanobts+band-900
@@ -71,12 +71,12 @@
 - voice:nanobts+band-900+mod-bts0-ts-tchf+cfg-codec-fr2
 - voice:nanobts+band-900+mod-bts0-ts-tchf+cfg-codec-fr3
 - voice:nanobts+band-900+mod-bts0-ts-tchh+cfg-codec-hr3
-- voice:nanobts+band-900+mod-bts0-dynts-ipa
+- voice:nanobts+band-900+mod-bts0-dynts-ipa+cfg-codec-fr-any
 - voice:nanobts+band-900+mod-bts0-numtrx2+mod-bts0-chanallocdescend
 - gprs:nanobts+band-900
 - gprs:nanobts+band-900+mod-bts0-dynts-ipa
 - gprs:nanobts+band-900+mod-bts0-numtrx2+mod-bts0-chanallocdescend
-- dynts:nanobts+band-900+mod-bts0-dynts67-ipa+cfg-codec-fr1
+- dynts:nanobts+band-900+mod-bts0-dynts67-ipa+cfg-codec-fr-any

 - nitb_smpp
 - smpp
diff --git a/example/scenarios/cfg-codec-fr-any.conf 
b/example/scenarios/cfg-codec-fr-any.conf
new file mode 100644
index 0000000..9cab6cc
--- /dev/null
+++ b/example/scenarios/cfg-codec-fr-any.conf
@@ -0,0 +1,7 @@
+config:
+  bsc:
+    net:
+      codec_list:
+      - fr1
+      - fr2
+      - fr3

--
To view, visit https://gerrit.osmocom.org/11526
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I0039ef60b323ed72cfe00d8fd9e9287e9c82d49f
Gerrit-Change-Number: 11526
Gerrit-PatchSet: 1
Gerrit-Owner: Pau Espin Pedrol <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Pau Espin Pedrol <[email protected]>
Gerrit-Reviewer: dexter <[email protected]>

Reply via email to