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

Double send measurements #107

Closed
gpad opened this issue Feb 15, 2018 · 3 comments
Closed

Double send measurements #107

gpad opened this issue Feb 15, 2018 · 3 comments

Comments

@gpad
Copy link

gpad commented Feb 15, 2018

See PR #106

I found that the measurements are sent 2 times with the same timestamp. To find this bug I configured some predefined measurements, I sent it via statsd subscriber and I set the level of logger to debug. In this way I found that the measurements are sent 2 times on the same seconds (I set the interval to 5 seconds):

2018-02-14 19:04:40.626 [debug] <0.1462.0>@exometer_report_statsd:exometer_report:68 Report metric [<<"rezolve~core_1@elixir-node_1~local_dev">>,".","erlang",".","memory",".","total"] = 75257336
2018-02-14 19:04:40.629 [debug] <0.1462.0>@exometer_report_statsd:exometer_report:68 Report metric [<<"rezolve~core_1@elixir-node_1~local_dev">>,".","erlang",".","memory",".","total"] = 75248920
2018-02-14 19:04:45.631 [debug] <0.1462.0>@exometer_report_statsd:exometer_report:68 Report metric [<<"rezolve~core_1@elixir-node_1~local_dev">>,".","erlang",".","memory",".","total"] = 75306448
2018-02-14 19:04:45.636 [debug] <0.1462.0>@exometer_report_statsd:exometer_report:68 Report metric [<<"rezolve~core_1@elixir-node_1~local_dev">>,".","erlang",".","memory",".","total"] = 75328744
2018-02-14 19:04:50.637 [debug] <0.1462.0>@exometer_report_statsd:exometer_report:68 Report metric [<<"rezolve~core_1@elixir-node_1~local_dev">>,".","erlang",".","memory",".","total"] = 75451056
2018-02-14 19:04:50.639 [debug] <0.1462.0>@exometer_report_statsd:exometer_report:68 Report m
@sammy007
Copy link

sammy007 commented Feb 18, 2018

Same here:

{exometer, [
    {predefined, [
      {[node, height], gauge, []},
      {[rps], spiral, [{time_span, 1000}]}
    ]},
    {report, [
      {reporters, [
        {exometer_report_lager, [{type_map, [{'_', integer}]}]}
      ]},
      {subscribers, [
        {select, {[{{[node, height], gauge, '_'}, [], ['$_']}], exometer_report_lager, value, 5000}},
        {select, {[{{[rps], spiral, '_'}, [], ['$_']}], exometer_report_lager, [count, one], 5000}}
      ]}
    ]}
  ]}
10:50:11.434 [notice] exometer_report_lager node.height: value=1024
10:50:11.435 [notice] exometer_report_lager node.height: value=1025
10:50:11.444 [notice] exometer_report_lager rps: count=0
10:50:11.444 [notice] exometer_report_lager rps: one=0
10:50:11.446 [notice] exometer_report_lager rps: count=0
10:50:11.446 [notice] exometer_report_lager rps: one=0

@uwiger
Copy link
Member

uwiger commented Feb 18, 2018

Have you checked again with the latest version?

@sammy007
Copy link

I see, fixed in master. Thank you.

@tolbrino tolbrino closed this as completed Apr 9, 2018
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

No branches or pull requests

4 participants