Skip to content
Unstable

Changes

Summary

  1. testenv: make copy= more intuitive (details)
  2. remsim: Add initial testenv (details)
  3. RAW_NS: SNS Add: handle IPv4/IPv6 element in SNS Ack (details)
  4. RAW_NS: SNS Changeweight: remove IPv4/IPv6 element in SNS Ack (details)
  5. PCUIF: fix usage of bts number in PCUIF_TXT_IND (details)
Commit 3f48fa0ffbdea022e3138bba8c89243e82f42a05 by Pau Espin Pedrol
testenv: make copy= more intuitive

Currently copy= in testenv.cfg creates copies of files under the full
source file name under the target directory: copy=dir/file.cfg creates
dir/file.cfg (like "cp -a --parents"). This is not very intuitive,
change it to create a "file.cfg" instead without subdirectory. With
this change, it behaves the same as "cp -a".

Suggested-by: Pau Espin Pedrol <pespin@sysmocom.de>
Change-Id: Id9e4ea066d0a0e72ccd83e4f6f887f382bf61e46
The file was modifiedggsn_tests/open5gs/open5gs-smf.yaml
The file was modified_testenv/README.md
The file was modifiedggsn_tests/testenv_open5gs.cfg
The file was modified_testenv/testenv/testdir.py
Commit 798c9a1dedaf43f9a0f3c4c3b967fb898bf7e170 by Pau Espin Pedrol
remsim: Add initial testenv

"server" testsuite is working as goos as in docker-playground.git
"bankd" testsuite is currently failing due to bankd exiting early after
starting.
"client" is not currently working/running in docker-playground, initial
config copied here for completeness.

This commit hence allows already quickly running "server" testsuite.

Change-Id: Ic69a04342a4cdfced90cf8ffbe7e5cdede8d4b8e
The file was addedremsim/client/REMSIM_Tests.cfg
The file was addedremsim/server/REMSIM_Tests.cfg
The file was addedremsim/testenv_client.cfg
The file was removedremsim/REMSIM_Tests.cfg
The file was addedremsim/testenv_bankd.cfg
The file was addedremsim/testenv_server.cfg
The file was addedremsim/bankd/REMSIM_Tests.cfg
The file was addedremsim/bankd/bankd_pcsc_slots.csv
Commit 7d747cc3ff7b2dc93ef9e6833935b43b94b1c009 by lynxis
RAW_NS: SNS Add: handle IPv4/IPv6 element in SNS Ack

When adding a new NS-VC via SNS Add, the SNS Ack should
only contains the IEs "List of IP4/6 elements" if the cause code is
Unknown IP endpoint.

However the libosmocore/ns2 code isn't following this behavior yet.

Change-Id: I18c659e43db79400c1b428ed745267a5e6e59102
The file was modifiedlibrary/RAW_NS.ttcnpp
Commit 5443d56283dd0fb814aca2040fab71e00497b4a3 by lynxis
RAW_NS: SNS Changeweight: remove IPv4/IPv6 element in SNS Ack

When changweights of a NS-VC via SNS, the SNS Ack should
only contains the IEs "List of IP4/6 elements" if the cause code is
Unknown IP endpoint.

However the libosmocore/ns2 code isn't following this behavior yet.

Change-Id: Ibf00cbab0c6691cdb72fe592ee30e122dd8f4783
The file was modifiedlibrary/RAW_NS.ttcnpp
Commit 5ad6e895f8f9d7b94645b3b37794866f2956176e by pmaier@sysmocom.de
PCUIF: fix usage of bts number in PCUIF_TXT_IND

When we receive the PCU_VERSION using tr_PCUIF_TXT_IND we must ignore the
included BTS number because the PCU_VERSION is not addressed to a specific
BTS. When we send a PCU_VERSION using ts_PCUIF_TXT_IND, we should always
use the bts number 0 to be consistent (the BSC/BTS will ignore this number
anyway).

Let's fix the usage of tr_PCUIF_TXT_IND and put comments, to make clear why
the above applies.

Change-Id: I93de261fc77806b2f817e0d30cb1b0d377ed0dbb
related: OS#6507
The file was modifiedpcu/PCUIF_Components.ttcn
The file was modifiedbts/BTS_Tests_OML.ttcn
The file was modifiedbts/BTS_Tests.ttcn
The file was modifiedpcu/PCU_Tests_NS.ttcn