fixeria submitted this change.

View Change

Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified
s1gw: improve conn established/closed logging

Change-Id: Iacda25ac9b34fc55217e2681e434c68eef2d3445
---
M s1gw/S1AP_Server.ttcn
1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/s1gw/S1AP_Server.ttcn b/s1gw/S1AP_Server.ttcn
index df91eaa..6d5d254 100644
--- a/s1gw/S1AP_Server.ttcn
+++ b/s1gw/S1AP_Server.ttcn
@@ -398,16 +398,18 @@

/* connection opened/closed events */
[] S1AP.receive(PortEvent:{connOpened := ?}) -> value pev {
- log("eNB connection (id=", pev.connOpened.connId, ", ",
- pev.connOpened.remName, ":", pev.connOpened.remPort, ") ",
- "established");
- f_conn_add(pev.connOpened.connId);
+ var ConnectionOpenedEvent ev := pev.connOpened;
+ log("eNB connection (id=", ev.connId, ", ",
+ "r=", ev.remName, ":", ev.remPort, "<->",
+ "l=", ev.locName, ":", ev.locPort, ") established");
+ f_conn_add(ev.connId);
}
[] S1AP.receive(PortEvent:{connClosed := ?}) -> value pev {
- log("eNB connection (id=", pev.connClosed.connId, ", ",
- pev.connClosed.remName, ":", pev.connClosed.remPort, ") ",
- "closed");
- f_conn_del(pev.connClosed.connId);
+ var ConnectionOpenedEvent ev := pev.connClosed;
+ log("eNB connection (id=", ev.connId, ", ",
+ "r=", ev.remName, ":", ev.remPort, "<->",
+ "l=", ev.locName, ":", ev.locPort, ") closed");
+ f_conn_del(ev.connId);
}

/* SCTP events we don't care about */

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

Gerrit-MessageType: merged
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Iacda25ac9b34fc55217e2681e434c68eef2d3445
Gerrit-Change-Number: 42351
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>