-
Notifications
You must be signed in to change notification settings - Fork 600
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 multi-cluster support for windows #824
Conversation
@povilasv did you need assistance here for review? I have multiple windows cluster and I would appreciate the merge here. |
Can you folks send a screenshot of before / after? I have no windows clusters so hard to review |
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.
Some nits, because I used windows-exporter twice in the same cluster and I had recording rules issues.
👍 Let's apply the @jkroepke changes |
Co-authored-by: Jan-Otto Kröpke <github@jkroepke.de>
Fix multi-cluster support for windows.