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

Haproxy: add missing session limit #1937

Merged
merged 1 commit into from
Dec 13, 2016
Merged

Conversation

krzysztof-indyk
Copy link
Contributor

Required for all PRs:

  • CHANGELOG.md updated (we recommend not updating this until the PR has been approved by a maintainer)
  • Sign CLA (if not already signed)

Add collecting session limit from haproxy stats.

@sparrc sparrc added this to the 1.2.0 milestone Oct 24, 2016
@krzysztof-indyk
Copy link
Contributor Author

@sparrc what can I do to merge this?

We use multiple haproxy instances with different limits and this fix helps us a lot, to properly create dashboard and set alarms.

@sparrc
Copy link
Contributor

sparrc commented Nov 28, 2016

this is slated for 1.2, we will do our best to get it merged before the release

@nmische
Copy link

nmische commented Nov 30, 2016

Just adding a 👍 to this. We would also like to see session limit so we can calculate session utilization (scur / slim * 100).

@sparrc sparrc merged commit 8e3fbaa into influxdata:master Dec 13, 2016
njwhite pushed a commit to njwhite/telegraf that referenced this pull request Jan 31, 2017
maxunt pushed a commit that referenced this pull request Jun 26, 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

Successfully merging this pull request may close these issues.

3 participants