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.

Sign up for GitHub

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

Jump to bottom

haproxy.stat.check.health.last set as long, should be a string type #4407

Closed
dariusj18 opened this issue May 27, 2017 · 0 comments
Closed

haproxy.stat.check.health.last set as long, should be a string type #4407

dariusj18 opened this issue May 27, 2017 · 0 comments
Labels
bug Metricbeat Metricbeat

Comments

@dariusj18
Copy link
Contributor

I see entries like "HTTP status check returned code <200>" in the field haproxy.stat.check.health.last

I don't know if it should be "string", "text", or "keyword"

I was told the change could be made in this file

https://github.com/elastic/beats/blob/master/metricbeat/module/haproxy/stat/_meta/fields.yml#L301
@dariusj18 dariusj18 mentioned this issue May 27, 2017
Closed
@tsg tsg added bug Metricbeat Metricbeat labels May 29, 2017
tsg pushed a commit to tsg/beats that referenced this issue May 29, 2017
Fix type for HAProxy health.last field
a853d1b 
Fixes elastic#4407. Also adds docs for two fields where docs were missing.
@tsg tsg mentioned this issue May 29, 2017
Merged
exekias pushed a commit that referenced this issue May 29, 2017
@tsg @exekias
Fix type for HAProxy health.last field (#4410)
a2ea586 
Fixes #4407. Also adds docs for two fields where docs were missing.
tsg added a commit to tsg/beats that referenced this issue May 31, 2017
@tsg
Fix type for HAProxy health.last field (elastic#4410)
3b3110e 
Fixes elastic#4407. Also adds docs for two fields where docs were missing.
(cherry picked from commit a2ea586)
@tsg tsg mentioned this issue May 31, 2017
Merged
monicasarbu pushed a commit that referenced this issue Jun 2, 2017
@tsg @monicasarbu
Fix type for HAProxy health.last field (#4410) (#4425)
5fd28cd 
Fixes #4407. Also adds docs for two fields where docs were missing.
(cherry picked from commit a2ea586)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Metricbeat Metricbeat
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dariusj18 @tsg