Skip to content
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

Fix [amoc, users, size] metric #4

Merged
merged 1 commit into from
Aug 29, 2024
Merged

Fix [amoc, users, size] metric #4

merged 1 commit into from
Aug 29, 2024

Conversation

NelsonVides
Copy link
Collaborator

@NelsonVides NelsonVides commented Aug 29, 2024

It was reading an ets table that is in fact an implementation detail.
Now it uses the correctly exposed callback to read.

$ rebar3 shell
===> Verifying dependencies...
===> Analyzing applications...
===> Compiling amoc_arsenal
Erlang/OTP 27 [erts-15.0.1] [source] [64-bit] [smp:12:12] [ds:12:12:10] [async-threads:1] [jit]

Eshell V15.0.1 (press Ctrl+G to abort, type help(). for help)
=WARNING REPORT==== 29-Aug-2024::17:38:43.547597 ===
Reporter=exometer_report_graphite not_enabled
=WARNING REPORT==== 29-Aug-2024::17:38:43.547645 ===
Reporter=exometer_report_graphite not_enabled
=WARNING REPORT==== 29-Aug-2024::17:38:43.547662 ===
Reporter=exometer_report_graphite not_enabled
===> Booted compiler
===> Booted telemetry
===> Booted amoc
===> Booted exometer_core
===> Booted exometer_report_graphite
===> Booted jsx
===> Booted rfc3339
===> Booted jesse
===> Booted cowlib
===> Booted ranch
===> Booted cowboy
===> Booted amoc_rest
===> Booted syntax_tools
===> Booted edoc
===> Booted xmerl
===> Booted docsh
===> Booted amoc_arsenal
===> Booted runtime_tools
1> exometer:get_value([amoc, users]).
{ok,[{size,0}]}

It was reading an ets table that is in fact an implementation detail.
Now it uses the correctly exposed callback to read.

```
$ rebar3 shell
===> Verifying dependencies...
===> Analyzing applications...
===> Compiling amoc_arsenal
Erlang/OTP 27 [erts-15.0.1] [source] [64-bit] [smp:12:12] [ds:12:12:10] [async-threads:1] [jit]

Eshell V15.0.1 (press Ctrl+G to abort, type help(). for help)
=WARNING REPORT==== 29-Aug-2024::17:38:43.547597 ===
Reporter=exometer_report_graphite not_enabled
=WARNING REPORT==== 29-Aug-2024::17:38:43.547645 ===
Reporter=exometer_report_graphite not_enabled
=WARNING REPORT==== 29-Aug-2024::17:38:43.547662 ===
Reporter=exometer_report_graphite not_enabled
===> Booted compiler
===> Booted telemetry
===> Booted amoc
===> Booted exometer_core
===> Booted exometer_report_graphite
===> Booted jsx
===> Booted rfc3339
===> Booted jesse
===> Booted cowlib
===> Booted ranch
===> Booted cowboy
===> Booted amoc_rest
===> Booted syntax_tools
===> Booted edoc
===> Booted xmerl
===> Booted docsh
===> Booted amoc_arsenal
===> Booted runtime_tools
1> exometer:get_value([amoc, users]).
{ok,[{size,0}]}
```
Copy link
Member

@chrzaszcz chrzaszcz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

@chrzaszcz chrzaszcz merged commit b6dc37c into main Aug 29, 2024
6 checks passed
@chrzaszcz chrzaszcz deleted the fix_users_size_metric branch August 29, 2024 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants