Skip to content
Unstable

Changes

Summary

  1. {bsc-nat,selftest,sysinfo}/gen_links.sh: use bash, not sh (details)
  2. gen_links.sh.inc: 'set -e' to abort on error (details)
  3. testenv: testsuite: use rsync with --archive (details)
  4. testenv/README: consistency fix (details)
  5. testenv: podman.exec_cmd: better error if stopped (details)
  6. testenv/README: deduplicate clean= description (details)
  7. testenv/README: document current working dir (details)
  8. testenv: clean_run_scripts: skip if podman stopped (details)
  9. testenv: copy=: support copying directories too (details)
  10. ggsn: sync GGSN_Tests.cfg with docker-playground (details)
  11. ggsn: add testenv.cfgs for osmo-ggsn (details)
  12. ggsn: add testenv.cfg for open5gs (details)
  13. pcap-client: initial testenv.cfg (details)
Commit 5344866f793fd6bfc8d0f3f7c18a823893c16d71 by Vadim Yanitskiy
{bsc-nat,selftest,sysinfo}/gen_links.sh: use bash, not sh

This fixes the following error:

  ./gen_links.sh: 40: shopt: not found

which is printed when compiling the above-mentioned testsuites.

Change-Id: I994e241efb73f6f8b78803cf4ecdfb057b210a3b
The file was modifiedsysinfo/gen_links.sh
The file was modifiedbsc-nat/gen_links.sh
The file was modifiedselftest/gen_links.sh
Commit a42c49f1c8110357cf643013ded2745f5d5ece6b by Vadim Yanitskiy
gen_links.sh.inc: 'set -e' to abort on error

We should not tolerate errors in '*/gen_links.sh' files.

Change-Id: Id9367e4c45c84dcbf1c308581360be73242c0d38
The file was modifiedgen_links.sh.inc
Commit 452a888e4d109dee6559f5130c8000aa09c0fd60 by Oliver Smith
testenv: testsuite: use rsync with --archive

When running in podman, the source files from the testsuite get copied
to a temporary directory to build the testsuites out-of-tree (avoiding
conflicts with possibly incompatible binary objects that may exist from
previously building the testsuites on the host).

This also copies additional scripts for preparation / clean up that may
be used in testenv.cfg. Use the --archive flag with rsync to ensure
that the executability is the same. I could have also used
--executability, but --archive contains two other existing flags and
more flags which may help us from running into unexpected situations
such as this one.

Without this patch, there was a bug when:
* first creating a shell script but not making it executable
* running testenv with podman (where rsync runs and creates the file
  initially without executable permissions)
* making the script executable
* running testenv with podman again, rsync will not adjust the
  permissions for the copy of the file
* user wonders why there is a "sh: 1: script.sh: Permission denied"
  error

Change-Id: Ia9737bd8ba22a555615862bf5e14dbdf0dea19b4
The file was modified_testenv/testenv/testsuite.py
Commit 8e244c004f76c7e5b371ad83c8666442e628741a by Oliver Smith
testenv/README: consistency fix

Add = after the vty_* keys too, like the others.

Change-Id: Ic9828b67f5a81498d0e951dd71b80b36cdd36173
The file was modified_testenv/README.md
Commit 50ae2377c5b423b0836a76791780b19495e69477 by Oliver Smith
testenv: podman.exec_cmd: better error if stopped

Testenv may try to run a comand in podman after the container was
stopped, if there is a bug in the shutdown logic. Give a meaningful
error in that case, instead of failing later on with a cryptic error in
subprocess.run() because None was passed inside cmd (for the container
name) instead of a string.

Change-Id: I29ebe200258a9a9ffb127ec8fa1260b7e2124c38
The file was modified_testenv/testenv/podman.py
Commit dc5b2865c0a0d984a5696d73e2dcda2fc762acd9 by Oliver Smith
testenv/README: deduplicate clean= description

Change-Id: I12dc9adb508a409a6156f9682c9db5684f551b06
The file was modified_testenv/README.md
Commit 426191061deae634f5218a3938e42ba239c63a92 by Oliver Smith
testenv/README: document current working dir

Change-Id: I218507bcbec94a36976fbf57d12ffd083fd2acfb
The file was modified_testenv/README.md
Commit 3d2968521c5125ee88fd39f7f3d44a888f39406c by Oliver Smith
testenv: clean_run_scripts: skip if podman stopped

Do not attempt to run the clean_run_scripts, if podman is being used and
the container has already been shutdown.

Change-Id: I3cc05aabd97b73b65c3089e8806f7a9b32e5975c
The file was modified_testenv/testenv/testdir.py
The file was modified_testenv/README.md
Commit a247aa34831cf0a5580a8e24a0702956392bf032 by Oliver Smith
testenv: copy=: support copying directories too

In preparation for adding the inital testenv.cfgs for ggsn, allow
copying full directories with copy= too. This will make the ggsn
testenv.cfg files easier to maintain.

Change-Id: I8e680fbd93516030614c015d7c3b5bdb153ec487
The file was modified_testenv/testenv/testdir.py
The file was modified_testenv/README.md
Commit ca44f4f3558f0cd7f904bedf8c80ae16cf4a2b9a by Oliver Smith
ggsn: sync GGSN_Tests.cfg with docker-playground

Add parameters that are missing from the config.

Change-Id: I16a8cbd77f52a1ca5c52c045b8d0932e681cc380
The file was modifiedggsn_tests/GGSN_Tests.cfg
Commit fcaa06144464e0ee7315aefa179b70bcfbef0260 by Oliver Smith
ggsn: add testenv.cfgs for osmo-ggsn

Change-Id: I941b859261c1fbf27360673f14f8a79f0cf6f509
The file was addedggsn_tests/osmo-ggsn/osmo-ggsn-v4v6_only.confmerge
The file was removedggsn_tests/osmo-ggsn.cfg
The file was addedggsn_tests/osmo-ggsn/osmo-ggsn.src.cfg
The file was addedggsn_tests/osmo-ggsn/testenv.sh
The file was addedggsn_tests/testenv_osmo_ggsn_v6_only.cfg
The file was addedggsn_tests/testenv_osmo_ggsn_v4_only.cfg
The file was addedggsn_tests/testenv_osmo_ggsn_all.cfg
The file was addedggsn_tests/osmo-ggsn/osmo-ggsn-v4_only.confmerge
The file was addedggsn_tests/osmo-ggsn/osmo-ggsn-v6_only.confmerge
The file was addedggsn_tests/testenv_osmo_ggsn_v4v6_only.cfg
The file was addedggsn_tests/osmo-ggsn/osmo-ggsn-all.confmerge
Commit 74fc0707498ac562c17aaa04ffa317ec4f4a58b6 by Oliver Smith
ggsn: add testenv.cfg for open5gs

Change-Id: If11c0fcba84122d2398cb50208f161b9a3961df6
The file was addedggsn_tests/testenv_open5gs.cfg
The file was addedggsn_tests/open5gs/freediameter.conf
The file was modifiedggsn_tests/GGSN_Tests.cfg
The file was addedggsn_tests/open5gs/open5gs-upf.yaml
The file was addedggsn_tests/open5gs/testenv.sh
The file was addedggsn_tests/open5gs/open5gs-smf.yaml
Commit 4cb3d71bf9ce7488ce4c39537e71febd02379e16 by Oliver Smith
pcap-client: initial testenv.cfg

Change-Id: I44479dfa19390e325e42fa3416cf93260f74aea6
The file was addedpcap-client/testenv.cfg