September 2022 - gerrit-log - lists.osmocom.org

Change in osmo-ttcn3-hacks[master]: hnbgw: test PS RAB Ass both with and without PFCP

by laforge

Attention is currently required from: osmith, neels. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29231 ) Change subject: hnbgw: test PS RAB Ass both with and without PFCP ...................................................................... Patch Set 4: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29231 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: I511e758807e0512c18f3f9e0a8c4699b9a3f5992 Gerrit-Change-Number: 29231 Gerrit-PatchSet: 4 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Comment-Date: Wed, 14 Sep 2022 09:34:33 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

2 years, 9 months

1
0
0 0

Change in docker-playground[master]: ttcn3-bts: Add osmux configurations

by osmith

Attention is currently required from: fixeria, pespin. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/29338 ) Change subject: ttcn3-bts: Add osmux configurations ...................................................................... Patch Set 2: (1 comment) File ttcn3-bts-test/jenkins.sh: https://gerrit.osmocom.org/c/docker-playground/+/29338/comment/4f739433_e10… PS2, Line 148: image_suffix_is_latest > I am not 100% sure, but this is not going to work for Centos jobs, which may have IMAGE_SUFFIX like […] good point, this can be used instead: if ! image_suffix_is_master; then -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/29338 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I185b77eb4fa05501f098d89b80b66168349175bb Gerrit-Change-Number: 29338 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Wed, 14 Sep 2022 09:04:57 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: comment

2 years, 9 months

1
0
0 0

Change in osmo-ttcn3-hacks[master]: BSC_Tests: fix TC_assignment_osmux_cn: use c_mr_conf_5_90

by fixeria

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29345 ) Change subject: BSC_Tests: fix TC_assignment_osmux_cn: use c_mr_conf_5_90 ...................................................................... BSC_Tests: fix TC_assignment_osmux_cn: use c_mr_conf_5_90 Currently this testcase fails: RSL MR CONFIG IE does not match expectation. Expected: { other := { len := 2, payload := '2804'O } } Change-Id: Icc5381eccb924803b1117b46d2b4c47cee6dabd7 --- M bsc/BSC_Tests.ttcn 1 file changed, 1 insertion(+), 9 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/45/29345/1 diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn index b040274..63c4572 100644 --- a/bsc/BSC_Tests.ttcn +++ b/bsc/BSC_Tests.ttcn @@ -5253,18 +5253,10 @@ var TestHdlrParams pars := f_gen_test_hdlr_pars(); var MSC_ConnHdlr vc_conn; - /* See note above */ - var RSL_IE_Body mr_conf := { - other := { - len := 2, - payload := '2804'O - } - }; - pars.ass_codec_list := valueof(ts_BSSMAP_IE_CodecList({ts_CodecAMR_H})); pars.ass_codec_list.codecElements[0].s0_7 := '00000100'B; /* 5,90k */ pars.ass_codec_list.codecElements[0].s8_15 := '00000111'B; - pars.expect_mr_conf_ie := mr_conf; + pars.expect_mr_conf_ie := c_mr_conf_5_90; pars.use_osmux_cn := true; g_osmux_enabled_cn := true; -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29345 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: Icc5381eccb924803b1117b46d2b4c47cee6dabd7 Gerrit-Change-Number: 29345 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 9 months

1
0
0 0

Change in osmo-ttcn3-hacks[master]: BSC_Tests: cosmetic: s/mr_conf_amr_5_90/c_mr_conf_5_90/

by fixeria

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29343 ) Change subject: BSC_Tests: cosmetic: s/mr_conf_amr_5_90/c_mr_conf_5_90/ ...................................................................... BSC_Tests: cosmetic: s/mr_conf_amr_5_90/c_mr_conf_5_90/ * This is a global constant, so prefix with 'c_' for consistency. * The RSL_IE_MultirateCfg is all about AMR, remove redundant 'amr'. Change-Id: I72405b7139fcfd0dbe90afc469ed4db71b43a87c --- M bsc/BSC_Tests.ttcn 1 file changed, 5 insertions(+), 5 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/43/29343/1 diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn index c68807d..a3a764e 100644 --- a/bsc/BSC_Tests.ttcn +++ b/bsc/BSC_Tests.ttcn @@ -111,7 +111,7 @@ /* BTS3 */ { trx_num := 1, tsc := 4 } } -private const RSL_IE_Body mr_conf_amr_5_90 := +private const RSL_IE_Body c_mr_conf_5_90 := valueof(RSL_IE_Body:{multirate_cfg := ts_RSL_MultirateCfg(true, 0, '00000100'B /* 5,90k */)}); /* per-BTS state which we keep */ @@ -4532,7 +4532,7 @@ pars.ass_codec_list := valueof(ts_BSSMAP_IE_CodecList({ts_CodecAMR_F})); pars.ass_codec_list.codecElements[0].s0_7 := '00000100'B; /* 5,90k */ pars.ass_codec_list.codecElements[0].s8_15 := '01010111'B; - pars.expect_mr_conf_ie := mr_conf_amr_5_90; + pars.expect_mr_conf_ie := c_mr_conf_5_90; f_init(1, true); f_sleep(1.0); @@ -4560,7 +4560,7 @@ pars.ass_codec_list := valueof(ts_BSSMAP_IE_CodecList({ts_CodecAMR_H})); pars.ass_codec_list.codecElements[0].s0_7 := '00000100'B; /* 5,90k */ pars.ass_codec_list.codecElements[0].s8_15 := '00000111'B; - pars.expect_mr_conf_ie := mr_conf_amr_5_90; + pars.expect_mr_conf_ie := c_mr_conf_5_90; f_init(1, true); f_sleep(1.0); @@ -5287,7 +5287,7 @@ pars.ass_codec_list := valueof(ts_BSSMAP_IE_CodecList({ts_CodecAMR_H})); pars.ass_codec_list.codecElements[0].s0_7 := '00000100'B; /* 5,90k */ pars.ass_codec_list.codecElements[0].s8_15 := '00000111'B; - pars.expect_mr_conf_ie := mr_conf_amr_5_90; + pars.expect_mr_conf_ie := c_mr_conf_5_90; pars.use_osmux_bts := true; g_osmux_enabled_bts := true; @@ -5310,7 +5310,7 @@ pars.ass_codec_list := valueof(ts_BSSMAP_IE_CodecList({ts_CodecAMR_H})); pars.ass_codec_list.codecElements[0].s0_7 := '00000100'B; /* 5,90k */ pars.ass_codec_list.codecElements[0].s8_15 := '00000111'B; - pars.expect_mr_conf_ie := mr_conf_amr_5_90; + pars.expect_mr_conf_ie := c_mr_conf_5_90; pars.use_osmux_cn := true; pars.use_osmux_bts := true; -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29343 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: I72405b7139fcfd0dbe90afc469ed4db71b43a87c Gerrit-Change-Number: 29343 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 9 months

1
0
0 0

Change in osmo-ttcn3-hacks[master]: BSC_Tests: use c_mr_conf_5_90 in TC_assignment_codec_amr_startmode_cruft

by fixeria

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29344 ) Change subject: BSC_Tests: use c_mr_conf_5_90 in TC_assignment_codec_amr_startmode_cruft ...................................................................... BSC_Tests: use c_mr_conf_5_90 in TC_assignment_codec_amr_startmode_cruft This way it's easier to see that the only difference from other tests using c_mr_conf_5_90 is the ICMI field set to false. Change-Id: If7b491fa55a9366520c2d665a700c5badb187fae --- M bsc/BSC_Tests.ttcn 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/44/29344/1 diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn index a3a764e..b040274 100644 --- a/bsc/BSC_Tests.ttcn +++ b/bsc/BSC_Tests.ttcn @@ -4612,8 +4612,8 @@ pars.ass_codec_list := valueof(ts_BSSMAP_IE_CodecList({ts_CodecAMR_F})); pars.ass_codec_list.codecElements[0].s0_7 := '00000100'B; /* 5,90k */ pars.ass_codec_list.codecElements[0].s8_15 := '01010111'B; - /* expect ICMI=0, smod=00: */ - pars.expect_mr_conf_ie := valueof(RSL_IE_Body:{multirate_cfg := ts_RSL_MultirateCfg(false, 0, '00000100'B /* 5,90k */)}); + pars.expect_mr_conf_ie := c_mr_conf_5_90; + pars.expect_mr_conf_ie.multirate_cfg.icmi := false; /* expect ICMI=0, smod=00: */ f_init(1, true); f_sleep(1.0); -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29344 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: If7b491fa55a9366520c2d665a700c5badb187fae Gerrit-Change-Number: 29344 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 9 months

1
0
0 0

Change in osmo-ttcn3-hacks[master]: BTS_Tests[_VAMOS]: use default param values for ts_RSL_MultirateCfg

by fixeria

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29341 ) Change subject: BTS_Tests[_VAMOS]: use default param values for ts_RSL_MultirateCfg ...................................................................... BTS_Tests[_VAMOS]: use default param values for ts_RSL_MultirateCfg There is no real need to pass the multi-rate parameters explicitly in each invocation of ts_RSL_MultirateCfg. The exact multi-rate configuration does not matter in scope of BTS specific tests. Change-Id: I24117af8c5b7b11fe4b8f24fca5279ad97b46fab --- M bts/BTS_Tests.ttcn M bts/BTS_Tests_VAMOS.ttcn 2 files changed, 5 insertions(+), 5 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/41/29341/1 diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn index e71aed9..fed45a9 100644 --- a/bts/BTS_Tests.ttcn +++ b/bts/BTS_Tests.ttcn @@ -8264,7 +8264,7 @@ /* TS1, TCH/F, V3 (AMR codec) */ pars := valueof(t_Pars(ts_RslChanNr_Bm(1), ts_RSL_ChanMode(RSL_CHRT_TCH_F, RSL_CMOD_SP_GSM3))); - pars.mr_conf := valueof(ts_RSL_MultirateCfg(false, 0, '00000100'B /* 5,90k */)); + pars.mr_conf := valueof(ts_RSL_MultirateCfg); vc_conn := f_start_handler(refers(f_TC_speech_rtp), pars); vc_conn.done; @@ -8283,7 +8283,7 @@ /* TS1, TCH/H0, V3 (AMR codec) */ pars := valueof(t_Pars(ts_RslChanNr_Lm(5, 0), ts_RSL_ChanMode(RSL_CHRT_TCH_H, RSL_CMOD_SP_GSM3))); - pars.mr_conf := valueof(ts_RSL_MultirateCfg(false, 0, '00000100'B /* 5,90k */)); + pars.mr_conf := valueof(ts_RSL_MultirateCfg); vc_conn := f_start_handler(refers(f_TC_speech_rtp), pars); vc_conn.done; @@ -8383,7 +8383,7 @@ /* TS5, TCH/H0, V3 (AMR codec) */ pars := valueof(t_Pars(ts_RslChanNr_Bm(1), ts_RSL_ChanMode(RSL_CHRT_TCH_F, RSL_CMOD_SP_GSM3))); - pars.mr_conf := valueof(ts_RSL_MultirateCfg(false, 0, '00000100'B /* 5,90k */)); + pars.mr_conf := valueof(ts_RSL_MultirateCfg); vc_conn := f_start_handler(refers(f_TC_speech_osmux), pars); vc_conn.done; @@ -8397,7 +8397,7 @@ /* TS5, TCH/H0, V3 (AMR codec) */ pars := valueof(t_Pars(ts_RslChanNr_Lm(5, 0), ts_RSL_ChanMode(RSL_CHRT_TCH_H, RSL_CMOD_SP_GSM3))); - pars.mr_conf := valueof(ts_RSL_MultirateCfg(false, 0, '00000100'B /* 5,90k */)); + pars.mr_conf := valueof(ts_RSL_MultirateCfg); vc_conn := f_start_handler(refers(f_TC_speech_osmux), pars); vc_conn.done; diff --git a/bts/BTS_Tests_VAMOS.ttcn b/bts/BTS_Tests_VAMOS.ttcn index 637a256..2d03411 100644 --- a/bts/BTS_Tests_VAMOS.ttcn +++ b/bts/BTS_Tests_VAMOS.ttcn @@ -175,7 +175,7 @@ pars[ch] := valueof(t_Pars(test[i][ch].chan_nr, test[i][ch].chan_mode)); if (test[i][ch].chan_mode.coding_alg_rate == RSL_CMOD_SP_GSM3) { - pars[ch].mr_conf := valueof(ts_RSL_MultirateCfg(false, 0, '00000100'B /* 5,90k */)); + pars[ch].mr_conf := valueof(ts_RSL_MultirateCfg); } vc_conn[ch] := f_start_handler(handler, pars[ch], l1ctl := false); } -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29341 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: I24117af8c5b7b11fe4b8f24fca5279ad97b46fab Gerrit-Change-Number: 29341 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 9 months

1
0
0 0

Change in osmo-ttcn3-hacks[master]: library/RSL_Types: s/amr_codec_modes/codec_modes/ in RSL_IE_MultirateCfg

by fixeria

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29342 ) Change subject: library/RSL_Types: s/amr_codec_modes/codec_modes/ in RSL_IE_MultirateCfg ...................................................................... library/RSL_Types: s/amr_codec_modes/codec_modes/ in RSL_IE_MultirateCfg The RSL MultiRate configuration IE is all about AMR (Adaptive Multi Rate) codec, so there is no need for 'amr_' prefix in field names. Change-Id: If63ee50e8681ad4e0a202f142f2fca2268d55079 --- M bts/BTS_Tests.ttcn M library/RSL_Types.ttcn 2 files changed, 6 insertions(+), 6 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/42/29342/1 diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn index fed45a9..c6ce115 100644 --- a/bts/BTS_Tests.ttcn +++ b/bts/BTS_Tests.ttcn @@ -2540,7 +2540,7 @@ case (RSL_CMOD_SP_GSM1) { tch_mode := L1CTL_CHAN_MODE_SPEECH_V1; } case (RSL_CMOD_SP_GSM2) { tch_mode := L1CTL_CHAN_MODE_SPEECH_V2; } case (RSL_CMOD_SP_GSM3) { tch_mode := L1CTL_CHAN_MODE_SPEECH_V3; - amr_codecs_bitmask := g_pars.mr_conf.amr_codec_modes; + amr_codecs_bitmask := g_pars.mr_conf.codec_modes; } case else { log("RSL channel mode := ", g_pars.chan_mode.coding_alg_rate, @@ -2568,7 +2568,7 @@ var integer n := 0; for (var integer i:= 7; i >= 0; i := i - 1) { - if (g_pars.mr_conf.amr_codec_modes[i] == '1'B) { + if (g_pars.mr_conf.codec_modes[i] == '1'B) { if (n == start_nth) { return 7 - i; } diff --git a/library/RSL_Types.ttcn b/library/RSL_Types.ttcn index a86be86..6681024 100644 --- a/library/RSL_Types.ttcn +++ b/library/RSL_Types.ttcn @@ -824,20 +824,20 @@ boolean icmi, BIT1 spare, uint2_t start_mode, - BIT8 amr_codec_modes, + BIT8 codec_modes, octetstring parameters } with { - variant (len) "LENGTHTO(mr_speech_ver,nscb,icmi,spare,start_mode,amr_codec_modes,parameters)" + variant (len) "LENGTHTO(mr_speech_ver,nscb,icmi,spare,start_mode,codec_modes,parameters)" }; template (value) RSL_IE_MultirateCfg ts_RSL_MultirateCfg(boolean icmi := true, uint2_t start_mode := 0, - BIT8 amr_codec_modes := '00000100'B /* 5,90k */) := { + BIT8 codec_modes := '00000100'B /* 5,90k */) := { len := 2, mr_speech_ver := 1, nscb := false, icmi := icmi, spare := '0'B, start_mode := start_mode, - amr_codec_modes := amr_codec_modes, + codec_modes := codec_modes, parameters := ''O } -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29342 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: If63ee50e8681ad4e0a202f142f2fca2268d55079 Gerrit-Change-Number: 29342 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 9 months

1
0
0 0

Change in osmo-bts[master]: Don't manually create pid file

by msuraev

msuraev has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/29303 ) Change subject: Don't manually create pid file ...................................................................... Don't manually create pid file Previously osmo-bts created /var/run/osmo-bts.pid which isn't used by anything and makes it hard to run as non-root user. Let's get rid of this pre-systemd relic. Related: OS#4107 Change-Id: I86bcaedbc8cb1297476ad741eaa45585fea3c380 --- M doc/manuals/chapters/architecture.adoc M doc/startup.txt M src/common/main.c 3 files changed, 0 insertions(+), 23 deletions(-) Approvals: laforge: Looks good to me, but someone else must approve fixeria: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/doc/manuals/chapters/architecture.adoc b/doc/manuals/chapters/architecture.adoc index 30eab7a..128ed59 100644 --- a/doc/manuals/chapters/architecture.adoc +++ b/doc/manuals/chapters/architecture.adoc @@ -89,7 +89,6 @@ | common | phy_links_open() | Iterate over list of configured PHY links | bts-specific | bts_model_phy_link_open() | Open each of the configured PHY links | bts-specific | bts_model_phy_link_close() | Close each of the configured PHY links -| common | write_pid_file() | Generate the pid file | common | osmo_daemonize() | Fork as daemon in background (if configured) | common | bts_main() | Run main loop until global variable quit >= 2 |=== diff --git a/doc/startup.txt b/doc/startup.txt index c60cc0a..816e76b 100644 --- a/doc/startup.txt +++ b/doc/startup.txt @@ -27,7 +27,6 @@ | common | abis_open() | Start of the A-bis connection to BSC | common | phy_links_open() | Iterate over list of configured PHY links | bts-specific | bts_model_phy_link_open() | Open each of the configured PHY links -| common | write_pid_file() | Generate the pid file | common | osmo_daemonize() | Fork as daemon in background (if configured) | common | bts_main() | Run main loop until global variable quit >= 2 | bts-specific | bts_model_oml_estab() | Called by core once OML link is established diff --git a/src/common/main.c b/src/common/main.c index d069812..954cc34 100644 --- a/src/common/main.c +++ b/src/common/main.c @@ -257,25 +257,6 @@ } } -static int write_pid_file(char *procname) -{ - FILE *outf; - char tmp[PATH_MAX+1]; - - snprintf(tmp, sizeof(tmp)-1, "/var/run/%s.pid", procname); - tmp[PATH_MAX-1] = '\0'; - - outf = fopen(tmp, "w"); - if (!outf) - return -1; - - fprintf(outf, "%d\n", getpid()); - - fclose(outf); - - return 0; -} - int bts_main(int argc, char **argv) { struct gsm_bts_trx *trx; @@ -358,8 +339,6 @@ } } - write_pid_file("osmo-bts"); - /* Accept a GSMTAP host from VTY config, but a commandline option overrides that. */ if (gsmtap_ip != NULL) { if (g_bts->gsmtap.remote_host != NULL) { -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/29303 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: I86bcaedbc8cb1297476ad741eaa45585fea3c380 Gerrit-Change-Number: 29303 Gerrit-PatchSet: 2 Gerrit-Owner: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

2 years, 9 months

1
0
0 0

Change in osmo-ttcn3-hacks[master]: hnbgw: test PS RAB Ass both with and without PFCP

by daniel

Attention is currently required from: osmith, neels. daniel has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29231 ) Change subject: hnbgw: test PS RAB Ass both with and without PFCP ...................................................................... Patch Set 4: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29231 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: I511e758807e0512c18f3f9e0a8c4699b9a3f5992 Gerrit-Change-Number: 29231 Gerrit-PatchSet: 4 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Comment-Date: Wed, 14 Sep 2022 08:21:50 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

2 years, 9 months

1
0
0 0

Change in osmo-ttcn3-hacks[master]: fixup: bsc: Add testscase & infra to validate Osmux support BTS<->BSC

by fixeria

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29340 ) Change subject: fixup: bsc: Add testscase & infra to validate Osmux support BTS<->BSC ...................................................................... fixup: bsc: Add testscase & infra to validate Osmux support BTS<->BSC The VTY command 'osmux (on|off|only)' was added to osmo-bsc.git recently, and thus is not available in -latest yet. Use it conditionally. Change-Id: Id501d3f4b5eb18b096d8baffcb5f38e583f7a3d8 Fixes: [1] I6e82eb9d995de988b812001e1c4cf6923509de66 --- M bsc/BSC_Tests.ttcn 1 file changed, 11 insertions(+), 7 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/40/29340/1 diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn index d4a7b20..c68807d 100644 --- a/bsc/BSC_Tests.ttcn +++ b/bsc/BSC_Tests.ttcn @@ -1203,7 +1203,9 @@ activate(as_Tguard()); f_init_vty("VirtMSC"); - f_vty_allow_osmux_cn(g_osmux_enabled_cn); + if (Misc_Helpers.f_osmo_repo_is("nightly")) { + f_vty_allow_osmux_cn(g_osmux_enabled_cn); + } var my_BooleanList allow_attach := { false, false, false }; f_init_statsd("VirtMSC", vc_STATSD, mp_test_ip, mp_bsc_statsd_port); @@ -1273,13 +1275,15 @@ f_wait_oml(bts_idx, "connected", 5.0); /* Set up BTS with VTY commands: */ - f_vty_enter_cfg_bts(BSCVTY, bts_idx); - if (g_osmux_enabled_bts) { - f_vty_transceive(BSCVTY, "osmux on"); - } else { - f_vty_transceive(BSCVTY, "osmux off"); + if (Misc_Helpers.f_osmo_repo_is("nightly")) { + f_vty_enter_cfg_bts(BSCVTY, bts_idx); + if (g_osmux_enabled_bts) { + f_vty_transceive(BSCVTY, "osmux on"); + } else { + f_vty_transceive(BSCVTY, "osmux off"); + } + f_vty_transceive(BSCVTY, "end"); } - f_vty_transceive(BSCVTY, "end"); } function f_init_bts_and_check_sysinfo(integer bts_idx := 0, -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29340 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: Id501d3f4b5eb18b096d8baffcb5f38e583f7a3d8 Gerrit-Change-Number: 29340 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange

2 years, 9 months

1
0
0 0

2025

2024

2023

2022

gerrit-log September 2022