chewbranca commented on PR #5602: URL: https://github.com/apache/couchdb/pull/5602#issuecomment-3255347111
Thanks for taking another look, @nickva! That's odd the reports aren't showing for you, the config settings you listed look okay, and `couch_srt_logger` has a config:listener to reload the matchers on changes to any of the logger config groups https://github.com/apache/couchdb/blob/couch-stats-resource-tracker-v3-rebase-main/src/couch_srt/src/couch_srt_logger.erl#L588-L598 so that's odd you're not seeing the changes picked up. The first four commands you did _should_ bootstrap the system into generating reports, eg these settings: ``` ([email protected])2> config:set("csrt", "enable", "true"). ok ([email protected])3> config:set("csrt", "enable_init_p", "true"). ok ([email protected])4> config:set("csrt", "enable_reporting", "true"). ok ([email protected])5> config:set("csrt_logger.matchers_enabled", "all_coordinators", "true"). ok ``` I just did a fresh clone and with the following diff I'm properly getting reports generated like: > [notice] 2025-09-04T19:43:09.288686Z [email protected] <0.11224.0> 381d115cad localhost:15984 127.0.0.1 adm GET /_all_dbs 200 ok 3 > [report] 2025-09-04T19:43:09.288790Z [email protected] <0.11274.0> -------- [csrt-pid-usage-lifetime db_open=5 get_kv_node=2 nonce="381d115cad" pid_ref="<0.11224.0>:#Ref<0.3500684338.1004273667.225747>" rows_read=2 started_at="2025-09-04T19:43:09.292z" type="coordinator-{chttpd_misc:handle_all_dbs_req}:GET:/_all_dbs" updated_at="2025-09-04T19:43:09.295z" username="adm"] ```diff (chewbranca)-(jobs:0)-(/tmp/couchdb) (! 15428)-> git diff diff --git a/rel/overlay/etc/default.ini b/rel/overlay/etc/default.ini index b2ffa87b7..3cfe9d1aa 100644 --- a/rel/overlay/etc/default.ini +++ b/rel/overlay/etc/default.ini @@ -1153,9 +1153,9 @@ url = {{nouveau_url}} ; Couch Stats Resource Tracker (CSRT) [csrt] -;enable = false -;enable_init_p = false -;enable_reporting = false +enable = true +enable_init_p = true +enable_reporting = true ;enable_rpc_reporting = false ; Truncate reports to not include zero values for counter fields. This is a @@ -1223,7 +1223,7 @@ url = {{nouveau_url}} ; CSRT default matchers - enablement configuration ; The default CSRT loggers can be individually enabled below [csrt_logger.matchers_enabled] -;all_coordinators = false +all_coordinators = true ;all_rpc_workers = false ;docs_read = false ;rows_read = false ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
