master/coturn
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix duplicate prometheus metric report (#1079)

Browse Source 
Prometheus would aggregate across all labels (when not specifying)
In this scenario `turn_total_allocations` does not need separate label
"type=all" as such in this case the result would be double the actual
amount
This commit is contained in:
Pavel Punsky 2022-11-06 11:13:48 -08:00 committed by GitHub
parent 5a28394200
commit da2ae173c4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/apps/relay/prom_server.c
View File

@ -131,14 +131,12 @@ void prom_set_finished_traffic(const char* realm, const char* user, unsigned lon
void prom_inc_allocation(SOCKET_TYPE type) {
if (turn_params.prometheus == 1){
prom_gauge_inc(turn_total_allocations, (const char*[]) {"all"});
prom_gauge_inc(turn_total_allocations, (const char*[]) {socket_type_name(type)});
}
}
void prom_dec_allocation(SOCKET_TYPE type) {
if (turn_params.prometheus == 1){
prom_gauge_dec(turn_total_allocations, (const char*[]) {"all"});
prom_gauge_dec(turn_total_allocations, (const char*[]) {socket_type_name(type)});
}
}