-
Notifications
You must be signed in to change notification settings - Fork 672
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
Add new counter LDAP Client Sessions #1278
Add new counter LDAP Client Sessions #1278
Conversation
First time contributing here, so please let me know what else is required. Thanks |
@breed808 is there anything else required here? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for submitting this! I've left a couple comments, and the CI has picked up an error.
Have a look and let me know if you have any questions.
Signed-off-by: Mark Dordoy <mdordoy@palantir.com>
…antir.com> Signed-off-by: Mark Dordoy <mdordoy@palantir.com>
Signed-off-by: Mark Dordoy <mdordoy@palantir.com>
Signed-off-by: Mark Dordoy <mdordoy@palantir.com>
Signed-off-by: Mark Dordoy <mdordoy@palantir.com>
@breed808 updated based on comments. Think i've fixed linting, but will need workflows to run to confirm. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Nice, thank you @breed808 Whats the process for this to make it to the next version of the grafana agent? |
This change will be made available in the next windows_exporter release. If the Grafana Agent is pinned to a particular windows_exporter version, the agent would need to update the dependency. I'll aim to have a new release for the exporter done in the next week. |
Thanks @breed808 Looks like the agent needs an update, PR here: grafana/agent#5088 |
A very handy metric to observe. If you hit your hard limit (5000 by default) connections start to drop off. Its a good indicator to know there is a bad LDAP Client connecting to your directory