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 incorrect type conversion of average response time in Haproxy dashboards #8404

Merged
merged 1 commit into from
Sep 24, 2018

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented Sep 24, 2018

Average response time in haproxy dashboard was being converted from seconds to milliseconds, but it is already in milliseconds, so it was showing values 1000 times greater than the expected ones.

Reported in https://discuss.elastic.co/t/average-response-time-on-haproxy-dashboard/149391

@jsoriano jsoriano added module Metricbeat Metricbeat needs_backport PR is waiting to be backported to other branches. v7.0.0-alpha1 v6.4.2 labels Sep 24, 2018
@jsoriano jsoriano force-pushed the haproxy-dashboard-milliseconds branch from 7137560 to 6cd4bf6 Compare September 24, 2018 09:04
@jsoriano jsoriano changed the title Fix incorrect type conversion of average response type in Haproxy dashboards Fix incorrect type conversion of average response time in Haproxy dashboards Sep 24, 2018
@jsoriano jsoriano merged commit ede435c into elastic:master Sep 24, 2018
@jsoriano jsoriano deleted the haproxy-dashboard-milliseconds branch September 24, 2018 14:21
jsoriano added a commit to jsoriano/beats that referenced this pull request Sep 24, 2018
@jsoriano jsoriano added v6.5.0 and removed needs_backport PR is waiting to be backported to other branches. labels Sep 24, 2018
jsoriano added a commit to jsoriano/beats that referenced this pull request Sep 24, 2018
jsoriano added a commit that referenced this pull request Sep 25, 2018
jsoriano added a commit that referenced this pull request Oct 1, 2018
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants