-
Notifications
You must be signed in to change notification settings - Fork 428
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Metrics/init #4070
Metrics/init #4070
Conversation
9039104
to
c99bd86
Compare
This comment was marked as outdated.
This comment was marked as outdated.
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #4070 +/- ##
=======================================
Coverage 83.88% 83.88%
=======================================
Files 526 526
Lines 33172 33181 +9
=======================================
+ Hits 27826 27834 +8
- Misses 5346 5347 +1
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
First config, then metrics, then everything else, including dependent metrics at the end
c99bd86
to
8485ebb
Compare
small_tests_24 / small_tests / 8485ebb elasticsearch_and_cassandra_25 / elasticsearch_and_cassandra_mnesia / 8485ebb small_tests_25 / small_tests / 8485ebb small_tests_25_arm64 / small_tests / 8485ebb ldap_mnesia_24 / ldap_mnesia / 8485ebb dynamic_domains_pgsql_mnesia_24 / pgsql_mnesia / 8485ebb ldap_mnesia_25 / ldap_mnesia / 8485ebb dynamic_domains_mysql_redis_25 / mysql_redis / 8485ebb pgsql_mnesia_24 / pgsql_mnesia / 8485ebb mam_SUITE:rdbms_async_pool_muc_all:muc_configurable_archiveid:muc_no_elements{failed,
{mam_SUITE,end_per_testcase,
{'EXIT',
{{room_archive_size,0,[{times,200,1}],ok},
[{mongoose_helper,do_wait_until,2,
[{file,
"/home/circleci/project/big_tests/tests/mongoose_helper.erl"},
{line,357}]},
{mam_helper,wait_for_room_archive_size,3,
[{file,
"/home/circleci/project/big_tests/tests/mam_helper.erl"},
{line,789}]},
{mam_helper,clean_room_archive,1,
[{file,
"/home/circleci/project/big_tests/tests/mam_helper.erl"},
{line,733}]},
{mam_helper,destroy_room,1,
[{file,
"/home/circleci/project/big_tests/tests/mam_helper.erl"},
{line,726}]},
{mam_SUITE,end_per_testcase,2,
[{file,
"/home/circleci/project/big_tests/tests/mam_SUITE.erl"},
{line,962}]},
{test_server,do_end_per_testcase,4,
[{file,"test_server.erl"},{line,1627}]},
{test_server,run_test_case_eval1,6,
[{file,"test_server.erl"},{line,1335}]},
{test_server,run_test_case_eval,9,
[{file,"test_server.erl"},{line,1224}]}]}}}} dynamic_domains_pgsql_mnesia_25 / pgsql_mnesia / 8485ebb internal_mnesia_25 / internal_mnesia / 8485ebb mssql_mnesia_25 / odbc_mssql_mnesia / 8485ebb pgsql_mnesia_25 / pgsql_mnesia / 8485ebb mysql_redis_25 / mysql_redis / 8485ebb dynamic_domains_mssql_mnesia_25 / odbc_mssql_mnesia / 8485ebb |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👌
The goal is that the whole metrics system can be initialised early and without any dependencies, on hooks, session manager, or other subsystems. It should be such subsystems the ones that register the metrics they consider necessary, and metrics one of the first things to be initialised by the system.