Skip to content
This repository has been archived by the owner on Mar 8, 2023. It is now read-only.

Add support for "show info" #180

Merged
merged 1 commit into from
May 24, 2020
Merged

Add support for "show info" #180

merged 1 commit into from
May 24, 2020

Conversation

SuperQ
Copy link
Member

@SuperQ SuperQ commented May 19, 2020

Add a new haproxy_version_info metric by running the "show info" command
on the UNIX socket.

This feature is not supported in HTTP mode. It will require some
refactoring of the HTTP stats method to handle this.

Signed-off-by: Ben Kochie superq@gmail.com

haproxy_exporter.go Outdated Show resolved Hide resolved
Copy link
Member

@grobie grobie left a comment

Choose a reason for hiding this comment

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

Thanks!

haproxy_exporter.go Outdated Show resolved Hide resolved
haproxy_exporter.go Outdated Show resolved Hide resolved
haproxy_exporter.go Show resolved Hide resolved
haproxy_exporter.go Outdated Show resolved Hide resolved
@SuperQ SuperQ force-pushed the superq/haproxy_info branch 5 times, most recently from ed244dc to 2cd1ddb Compare May 21, 2020 12:19
Copy link
Member

@grobie grobie left a comment

Choose a reason for hiding this comment

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

Thanks, looks good. One more question on the error handing.

haproxy_exporter.go Outdated Show resolved Hide resolved
haproxy_exporter.go Outdated Show resolved Hide resolved
@SuperQ SuperQ force-pushed the superq/haproxy_info branch 2 times, most recently from 4f9faf1 to d914a17 Compare May 23, 2020 14:04
Add a new haproxy_version_info metric by running the "show info" command
on the UNIX socket.

This feature is not supported in HTTP mode. It will require some
refactoring of the HTTP stats method to handle this.

Signed-off-by: Ben Kochie <superq@gmail.com>
Copy link
Member

@grobie grobie left a comment

Choose a reason for hiding this comment

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

Thanks!

@grobie grobie merged commit db9370a into master May 24, 2020
@SuperQ SuperQ mentioned this pull request Jun 17, 2020
grobie added a commit that referenced this pull request Jun 21, 2020
* [CHANGE] Switch logging to go-kit #171
* [CHANGE] Fix metric types #182
* [CHANGE] Fix unit of time metric #183
* [FEATURE] Add filtering on server status #160
* [ENHANCEMENT] Add compression and server selection metrics #154
* [ENHANCEMENT] Add client/server abort metrics #167
* [ENHANCEMENT] Add version info metric (when using UNIX sockets) #180

Note: This release fixes the metric types of counters and renames the following metrics:

* `haproxy_exporter_csv_parse_failures` -> `haproxy_exporter_csv_parse_failures_total`
* `haproxy_exporter_total_scrapes` -> `haproxy_exporter_scrapes_total`
* `haproxy_server_check_duration_milliseconds` -> `haproxy_server_check_duration_seconds`

Signed-off-by: Tobias Schmidt <tobidt@gmail.com>
grobie added a commit that referenced this pull request Jun 21, 2020
* [CHANGE] Switch logging to go-kit #171
* [CHANGE] Fix metric types #182
* [CHANGE] Fix unit of time metric #183
* [FEATURE] Add filtering on server status #160
* [ENHANCEMENT] Add compression and server selection metrics #154
* [ENHANCEMENT] Add client/server abort metrics #167
* [ENHANCEMENT] Add version info metric (when using UNIX sockets) #180

Note: This release fixes the metric types of counters and renames the following metrics:

* `haproxy_exporter_csv_parse_failures` -> `haproxy_exporter_csv_parse_failures_total`
* `haproxy_exporter_total_scrapes` -> `haproxy_exporter_scrapes_total`
* `haproxy_server_check_duration_milliseconds` -> `haproxy_server_check_duration_seconds`

Signed-off-by: Tobias Schmidt <tobidt@gmail.com>

Co-authored-by: Tobias Schmidt <tobidt@gmail.com>
@SuperQ SuperQ deleted the superq/haproxy_info branch January 18, 2021 15:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants