[Asterisk-code-review] Add core Prometheus support to Asterisk (...asterisk[master])

[George Joseph]

George Joseph has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/11337 )

Change subject: Add core Prometheus support to Asterisk
......................................................................


Patch Set 7:

(1 comment)

https://gerrit.asterisk.org/#/c/11337/7/tests/test_res_prometheus.c 
File tests/test_res_prometheus.c:

https://gerrit.asterisk.org/#/c/11337/7/tests/test_res_prometheus.c@1 
PS7, Line 1: /*
The test leaks.  Ran it 10 times...
Direct leak of 920 byte(s) in 10 object(s) allocated from:
    #0 0x7f861ee2c110  (/lib64/liblsan.so.0+0x10110)
    #1 0x46064c in __ast_repl_calloc /usr/src/asterisk/asterisk/main/astmm.c:1558
    #2 0x460873 in __ast_calloc /usr/src/asterisk/asterisk/main/astmm.c:1628
    #3 0x462130 in internal_ao2_alloc /usr/src/asterisk/asterisk/main/astobj2.c:694
    #4 0x462484 in __ao2_alloc /usr/src/asterisk/asterisk/main/astobj2.c:768
    #5 0x62521f in auth_create /usr/src/asterisk/asterisk/main/http.c:1555
    #6 0x62550e in ast_http_get_auth /usr/src/asterisk/asterisk/main/http.c:1625
    #7 0x7f861a6c44ca in http_callback /usr/src/asterisk/asterisk/res/res_prometheus.c:570
    #8 0x624e07 in handle_uri /usr/src/asterisk/asterisk/main/http.c:1482
    #9 0x625f59 in httpd_process_request /usr/src/asterisk/asterisk/main/http.c:1923
    #10 0x626247 in httpd_helper_thread /usr/src/asterisk/asterisk/main/http.c:1986
    #11 0x5c8091 in handle_tcptls_connection /usr/src/asterisk/asterisk/main/tcptls.c:241
    #12 0x5dca61 in dummy_start /usr/src/asterisk/asterisk/main/utils.c:1249
    #13 0x7f861e2ad5a1 in start_thread (/lib64/libpthread.so.0+0x85a1)



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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I891433a272c92fd11c705a2c36d65479a415ec42
Gerrit-Change-Number: 11337
Gerrit-PatchSet: 7
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-Comment-Date: Mon, 20 May 2019 16:12:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190520/0c3b3249/attachment.html>