Attention is currently required from: pespin.
msuraev has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/32277 )
Change subject: ctrl: drop deprecated neighbor resolution interface
......................................................................
Patch Set 8:
(1 comment)
File src/osmo-bsc/neighbor_ident_ctrl.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/32277/comment/d92b42fd_b24d4ad7
PS8, Line 553: /* This and the following: Add/Remove a CGI-PS as neighbor */
> why are you dropping this? This code is independent from the inteface you are removing.
You mean parse_cgi_ps()? It's only used by deprecated ctrl commands which are removed by this patch.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/32277
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I93e25856eede4b2f5aa36d8a16714b43d37b4fd0
Gerrit-Change-Number: 32277
Gerrit-PatchSet: 8
Gerrit-Owner: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 05 May 2023 06:44:12 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment
neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/32623 )
Change subject: actually run vty-tests: VTY transcript tests
......................................................................
Patch Set 1:
(1 comment)
This change is ready for review.
Patchset:
PS1:
well ok, even a successful "enable" does test *something*.
Also for CN pooling being implemented there will be a lot of new VTY transcript tests.
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/32623
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: Ie9fc8f29b84e1dffea000fea11e0afe285c1df39
Gerrit-Change-Number: 32623
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Comment-Date: Fri, 05 May 2023 00:38:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
neels has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/32623 )
Change subject: actually run vty-tests: VTY transcript tests
......................................................................
actually run vty-tests: VTY transcript tests
There have been VTY transcript tests around for a long time, but we are
not running them.
Change-Id: Ie9fc8f29b84e1dffea000fea11e0afe285c1df39
---
M tests/Makefile.am
1 file changed, 13 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-hnbgw refs/changes/23/32623/1
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 806bfb1..26c01b6 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -36,9 +36,7 @@
if ENABLE_EXT_TESTS
python-tests:
- echo ""
-# TODO: Enable once we have a VTY/CTRL interface:
-# $(MAKE) vty-test
+ $(MAKE) vty-test
# osmotestvty.py -p $(abs_top_srcdir) -w $(abs_top_builddir) -v
# osmotestconfig.py -p $(abs_top_srcdir) -w $(abs_top_builddir) -v
# $(srcdir)/ctrl_test_runner.py -w $(abs_top_builddir) -v
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/32623
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: Ie9fc8f29b84e1dffea000fea11e0afe285c1df39
Gerrit-Change-Number: 32623
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-MessageType: newchange
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/32620 )
Change subject: move main() to separate file
......................................................................
Patch Set 1:
(2 comments)
File src/osmo-hnbgw/osmo_hnbgw_main.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-6790):
https://gerrit.osmocom.org/c/osmo-hnbgw/+/32620/comment/dbf034d3_70e8be05
PS1, Line 65: static void print_usage()
Bad function definition - void print_usage() should probably be void print_usage(void)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-6790):
https://gerrit.osmocom.org/c/osmo-hnbgw/+/32620/comment/d5db598a_32042d20
PS1, Line 70: static void print_help()
Bad function definition - void print_help() should probably be void print_help(void)
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/32620
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I5f26a6b68f0d380617d73371f40f3b9055cac362
Gerrit-Change-Number: 32620
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 04 May 2023 23:22:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/32618 )
Change subject: simplify: one g_hnbgw as global state and root ctx
......................................................................
Patch Set 1:
(1 comment)
File src/osmo-hnbgw/hnbgw_vty.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-6788):
https://gerrit.osmocom.org/c/osmo-hnbgw/+/32618/comment/eb6adbe1_11a656b7
PS1, Line 294: talloc_free((void*)g_hnbgw->config.iuh_local_ip);
"(foo*)" should be "(foo *)"
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/32618
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I3d54a5bb30c16a990c6def2a0ed207f60a95ef5d
Gerrit-Change-Number: 32618
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 04 May 2023 23:22:50 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment