Attention is currently required from: osmith, neels, pespin, fixeria.

msuraev has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/31513 )

Change subject: GSMTAP: allow configuring src IP for log messages
......................................................................


Patch Set 13:

(2 comments)

File src/core/logging_gsmtap.c:

https://gerrit.osmocom.org/c/libosmocore/+/31513/comment/a53b04e1_cdd8449d
PS12, Line 186:                 target->tgt_gsmtap.src_addr = 
talloc_strdup(target, src);
> Yes, the actionable suggestion is that you verify everything is fine since 
> your are submitting this  […]
See - https://gerrit.osmocom.org/c/libosmocore/+/31868


File utils/osmo-stat-dummy/osmo-stat-dummy.cfg:

https://gerrit.osmocom.org/c/libosmocore/+/31513/comment/1fc4d995_15ffd122
PS4, Line 1: log gsmtap 127.0.0.69 127.6.9.1
> Yes. […]
The purpose is described in the README - it's to test statsd and rate counters. 
Having any kind of logging setup does not affect it in any way.



--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31513
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I9000269ce5b3dce1e757271b7c42e77b68d38f25
Gerrit-Change-Number: 31513
Gerrit-PatchSet: 13
Gerrit-Owner: msuraev <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-CC: neels <[email protected]>
Gerrit-Attention: osmith <[email protected]>
Gerrit-Attention: neels <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Mon, 13 Mar 2023 15:44:27 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: neels <[email protected]>
Comment-In-Reply-To: pespin <[email protected]>
Comment-In-Reply-To: msuraev <[email protected]>
Gerrit-MessageType: comment

Reply via email to