[PATCH] osmo-msc[master]: fix make distcheck with python tests

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/.

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Thu Jul 20 17:38:18 UTC 2017


fix make distcheck with python tests

- bscs.config needed by the vty tests was not picked up as a dist file, because
  its suffix is not 'cfg'. Rename it, also indicating its relation to the vty
  tests: bscs-for-vty-tests.cfg. Apply this rename in vty_test_runner.py.
- Remove restart counters after external tests, otherwise distcheck complains
  about uncleaned files.
- Add contrib/ipa.py to EXTRA_DIST, hence add a Makefile.am to contrib/.
  Otherwise the python tests cannot find that dependency.

Change-Id: I42b55cb1125099afc3a8e3f87c0e398426b2e2a9
---
M Makefile.am
M configure.ac
D doc/examples/osmo-bsc_nat/bscs.config
M doc/examples/osmo-bsc_nat/osmo-bsc_nat.cfg
M tests/Makefile.am
M tests/vty_test_runner.py
6 files changed, 5 insertions(+), 16 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/18/3318/2

diff --git a/Makefile.am b/Makefile.am
index 31a54ea..4a1f65d 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -13,6 +13,7 @@
 	include \
 	src \
 	tests \
+	contrib \
 	$(NULL)
 
 pkgconfigdir = $(libdir)/pkgconfig
diff --git a/configure.ac b/configure.ac
index ff51ccf..adae598 100644
--- a/configure.ac
+++ b/configure.ac
@@ -263,4 +263,5 @@
     tests/nanobts_omlattr/Makefile
     doc/Makefile
     doc/examples/Makefile
+    contrib/Makefile
     Makefile)
diff --git a/doc/examples/osmo-bsc_nat/bscs.config b/doc/examples/osmo-bsc_nat/bscs.config
deleted file mode 100644
index 176debe..0000000
--- a/doc/examples/osmo-bsc_nat/bscs.config
+++ /dev/null
@@ -1,13 +0,0 @@
-nat
- bsc 0
-  token lol
-  location_area_code 1234
-  description bsc
-  max-endpoints 32
-  paging forbidden 0
- bsc 1
-  token wat
-  location_area_code 5678
-  description bsc
-  max-endpoints 32
-  paging forbidden 0
diff --git a/doc/examples/osmo-bsc_nat/osmo-bsc_nat.cfg b/doc/examples/osmo-bsc_nat/osmo-bsc_nat.cfg
index 6b48e97..efbb64d 100644
--- a/doc/examples/osmo-bsc_nat/osmo-bsc_nat.cfg
+++ b/doc/examples/osmo-bsc_nat/osmo-bsc_nat.cfg
@@ -62,5 +62,5 @@
  timeout ping 20
  timeout pong 5
  ip-dscp 0
- bscs-config-file bscs.config
+ bscs-config-file bscs-for-vty-tests.cfg
  access-list bla imsi-allow ^11$
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 9cbc1c1..72161ee 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -87,7 +87,7 @@
 if BUILD_SMPP
 	$(PYTHON) $(srcdir)/smpp_test_runner.py -w $(abs_top_builddir) -v
 endif
-	rm -f $(top_builddir)/hlr.sqlite3
+	rm -f $(top_builddir)/hlr.sqlite3 $(top_builddir)/gsn_restart $(top_builddir)/gtphub_restart_count
 else
 python-tests: $(BUILT_SOURCES)
 	echo "Not running python-based tests (determined at configure-time)"
diff --git a/tests/vty_test_runner.py b/tests/vty_test_runner.py
index 92775d5..7d84e21 100644
--- a/tests/vty_test_runner.py
+++ b/tests/vty_test_runner.py
@@ -1220,7 +1220,7 @@
 def nat_bsc_reload(x):
     x.vty.command("configure terminal")
     x.vty.command("nat")
-    x.vty.command("bscs-config-file bscs.config")
+    x.vty.command("bscs-config-file bscs-for-vty-tests.cfg")
     x.vty.command("end")
 
 def nat_msc_ip(x, ip, port):

-- 
To view, visit https://gerrit.osmocom.org/3318
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I42b55cb1125099afc3a8e3f87c0e398426b2e2a9
Gerrit-PatchSet: 2
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>



More information about the gerrit-log mailing list