iedemam has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmo-abis/+/28024 )


Change subject: stats: add site_id scope to ipa-rsl tcp stat names
......................................................................

stats: add site_id scope to ipa-rsl tcp stat names

Multiple IPA units can have the same bts_id but scoping by their
site_ids will make them unique. This also clarifies the "bts"
number being communicated. It is not the bts configuration index
in osmo-bsc.cfg, it is the bts id specified in the vty line:

bts X
 ipa unit-id SITE BTS

Change-Id: I3b44319fb4bc6a812800001c58dfe1a664645b43
---
M src/input/ipaccess.c
1 file changed, 2 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmo-abis refs/changes/24/28024/1

diff --git a/src/input/ipaccess.c b/src/input/ipaccess.c
index 8dc1008..9bee3f9 100644
--- a/src/input/ipaccess.c
+++ b/src/input/ipaccess.c
@@ -343,8 +343,8 @@
                                     "could not register FD\n");
                                goto err;
                        }
-                       snprintf(tcp_stat_name, sizeof(tcp_stat_name), 
"bts.%u.ipa-rsl.%u",
-                               unit_data.bts_id, unit_data.trx_id);
+                       snprintf(tcp_stat_name, sizeof(tcp_stat_name), 
"site.%u.bts.%u.ipa-rsl.%u",
+                               unit_data.site_id, unit_data.bts_id, 
unit_data.trx_id);
                        osmo_stats_tcp_osmo_fd_register(newbfd, tcp_stat_name);

                        e1i_ts = ipaccess_line_ts(newbfd, new_line);

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

Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I3b44319fb4bc6a812800001c58dfe1a664645b43
Gerrit-Change-Number: 28024
Gerrit-PatchSet: 1
Gerrit-Owner: iedemam <[email protected]>
Gerrit-MessageType: newchange

Reply via email to