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

Throughput calculation wrong by factor 2x on /system/nodes ? #1199

Closed
henrikjohansen opened this issue Jun 1, 2015 · 5 comments
Closed

Throughput calculation wrong by factor 2x on /system/nodes ? #1199

henrikjohansen opened this issue Jun 1, 2015 · 5 comments
Assignees
Labels
bug
Milestone

Comments

@henrikjohansen
Copy link

@henrikjohansen henrikjohansen commented Jun 1, 2015

This one is weird ... the throughput calculation jumps to 2x the value when navigating to /system/nodes?

Screenshot from our test cluster ... the upper topmenu is from /search .. the lower is from /system/nodes (using overlaying windows) :

screen shot 2015-06-01 at 13 02 13

@kroepke
Copy link
Member

@kroepke kroepke commented Jun 1, 2015

Thanks, will take a look!
On Jun 1, 2015 1:09 PM, "Henrik Johansen" notifications@github.com wrote:

This one is weird ... the throughput calculation jumps to 2x the value
when navigating to /system/nodes?

Screenshot from our test cluster ... the upper topmenu is from /search ..
the lower is from /system/nodes (using overlaying windows) :

[image: screen shot 2015-06-01 at 13 02 13]
https://cloud.githubusercontent.com/assets/1473308/7911531/2f2569b2-085f-11e5-9459-4e82cdaad0af.png


Reply to this email directly or view it on GitHub
#1199.

@henrikjohansen
Copy link
Author

@henrikjohansen henrikjohansen commented Jun 1, 2015

I checked all other pages btw - /system/nodes is the only place where this happens ....

@kroepke kroepke added the bug label Jun 1, 2015
@kroepke kroepke added this to the 1.1.0 milestone Jun 1, 2015
@kroepke kroepke self-assigned this Jun 1, 2015
@kroepke
Copy link
Member

@kroepke kroepke commented Jun 1, 2015

Will be in the next RC. Thanks for the update, that helped a lot :)

@dennisoelkers
Copy link
Member

@dennisoelkers dennisoelkers commented Jun 1, 2015

Is this issue solved? If yes, can it be closed?

@kroepke
Copy link
Member

@kroepke kroepke commented Jun 1, 2015

It hasn't been merged into master yet, that's why the ticket is still open.

kroepke added a commit to graylog-labs/graylog2-web-interface that referenced this issue Jun 2, 2015
…ame metric on the same page

if one global subscription and one per-node subscription was present on the same page, metrics were reported twice

fixes Graylog2/graylog2-server#1199
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants