neels submitted this change.

View Change


Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve fixeria: Looks good to me, approved
pfcp: fix missing vty_write of pfcp local-port

Change-Id: I180e67b8abdd3c0d91b30f1e7668b02f1a323809
---
M src/osmo-hnbgw/hnbgw_vty.c
M tests/pfcp_cfg.vty.with_pfcp
2 files changed, 12 insertions(+), 0 deletions(-)

diff --git a/src/osmo-hnbgw/hnbgw_vty.c b/src/osmo-hnbgw/hnbgw_vty.c
index 674832b..d401279 100644
--- a/src/osmo-hnbgw/hnbgw_vty.c
+++ b/src/osmo-hnbgw/hnbgw_vty.c
@@ -945,6 +945,8 @@
vty_out(vty, " pfcp%s", VTY_NEWLINE);
if (g_hnbgw->config.pfcp.local_addr)
vty_out(vty, " local-addr %s%s", g_hnbgw->config.pfcp.local_addr, VTY_NEWLINE);
+ if (g_hnbgw->config.pfcp.local_port)
+ vty_out(vty, " local-port %u%s", g_hnbgw->config.pfcp.local_port, VTY_NEWLINE);
if (g_hnbgw->config.pfcp.remote_addr)
vty_out(vty, " remote-addr %s%s", g_hnbgw->config.pfcp.remote_addr, VTY_NEWLINE);

diff --git a/tests/pfcp_cfg.vty.with_pfcp b/tests/pfcp_cfg.vty.with_pfcp
index a093ffb..44b922c 100644
--- a/tests/pfcp_cfg.vty.with_pfcp
+++ b/tests/pfcp_cfg.vty.with_pfcp
@@ -50,6 +50,7 @@
...
pfcp
local-addr 127.0.0.1
+ local-port 8805
remote-addr 127.0.0.2
msc 0
...

To view, visit change 35644. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I180e67b8abdd3c0d91b30f1e7668b02f1a323809
Gerrit-Change-Number: 35644
Gerrit-PatchSet: 2
Gerrit-Owner: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de>
Gerrit-MessageType: merged