laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/35872?usp=email )
Change subject: pick up all *.vty in EXTRA_DIST
......................................................................
pick up all *.vty in EXTRA_DIST
(Same as osmo-bsc I47c9011b5e0e2886d221e34e6aa281d1dd0495c7)
*.vty tests are picked up by the Makefile.am by means of a wildcard --
they are run when they are there. So when you forget to add it to
EXTRA_DIST, it will be run in your local build tree, but it will be
silently omitted from a distribution tar, and nothing will complain
about it gone missing.
Instead, also use a *.vty wildcard in EXTRA_DIST. So any *.vty test
added to the git source will both be run *and* included in distribution
tars implicitly.
So far, test_neighbor_ident.vty was missing from the distribution.
Change-Id: Id28e020fc59b83d1b4cd0e5b72314a46bea62259
---
M tests/Makefile.am
1 file changed, 24 insertions(+), 1 deletion(-)
Approvals:
osmith: Looks good to me, but someone else must approve
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 839e53b..7ab3c37 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -38,7 +38,7 @@
vty_test_runner.py \
ctrl_test_runner.py \
smpp_test_runner.py \
- test_nodes.vty \
+ $(srcdir)/*.vty \
$(NULL)
TESTSUITE = $(srcdir)/testsuite
--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/35872?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Id28e020fc59b83d1b4cd0e5b72314a46bea62259
Gerrit-Change-Number: 35872
Gerrit-PatchSet: 2
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/35958?usp=email )
Change subject: config: Do not set up VRF
......................................................................
config: Do not set up VRF
gtp_u_kmod won't configure the VRF if the node is not present.
We don't want to use it for now, so disable it.
Change-Id: Ia47a85e2d51e56368cc9940ef0e6faa41afe81e9
---
M config/sys.config
1 file changed, 17 insertions(+), 5 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/erlang/osmo-epdg refs/changes/58/35958/1
diff --git a/config/sys.config b/config/sys.config
index 6119e02..6183daf 100755
--- a/config/sys.config
+++ b/config/sys.config
@@ -51,11 +51,11 @@
%% Allow binding to an IP address that is nonlocal or does not (yet) exist (IP_FREEBIND):
freebind,
% Create gtp tundev with role SGSN:
- {role, sgsn},
- {vrf, [{routes, [{{10, 180, 0, 0}, 16}]}%%,
- %%{netdev, "upstream"}
- ]
- }
+ {role, sgsn}%,
+ %{vrf, [{routes, [{{10, 180, 0, 0}, 16}]}%%,
+ % %%{netdev, "upstream"}
+ % ]
+ %}
]
}]
}
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/35958?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: erlang/osmo-epdg
Gerrit-Branch: master
Gerrit-Change-Id: Ia47a85e2d51e56368cc9940ef0e6faa41afe81e9
Gerrit-Change-Number: 35958
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newchange