-
Notifications
You must be signed in to change notification settings - Fork 373
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
[enhancement] Windows support #79
Comments
Hi @jkroepke, I'm not totally against it, but I have a few doubts... Here are a few questions to start the discussion:
If anyone is also using Windows hosts, feel free to jump in the discussion with your thoughts! |
Sure! The Kubernetes Control Plane does not support Windows. I have to run Linux Node (e.g. for CoreDNS, ArgoCD) and Windows Node Pools for Application (Customer runs dotNET Application) On Azure, a Kubernetes managed Services requires at-least one Linux Node and you can add additional Windows nodes. We are also quite common, that customers wants to use Standard infrastructure components like Redis, Solr or elasticsearch which are not running on Windows node. In general, we avoid Windows nodes as much as possible. Hybrid OS clusters are common, because mono-Windows cluster can't be exists.
Cluster, Namespace, Node, Pod
We have namespace which contains windows and linux containers. My personal opinion is that one dashboard provides a better user expericence. Since kube-state-metrics provides request/limits for pods on any OS, Windows Pods are already included Pod CPU/Memory Request/Limits panels, but excluded on the usage panel. For separate dashboards, Requests/Limits from Windows Pods should be excluded then. Example Panel: Real (linux only), Requests/Limits (any OS)
No, except Windows is not providing metrics, some panels will be Linux specific. But I do not plan to add new panels.
Yes, because out company has an upstream first culture. If other users have a bug, we may also have the bug, too. Of course, I will have. The good thing is that the dashboard are providing only basic metrics. I don't plan to engineer the queries on my own. kubernetes-mixin provides some rules for windows which I will use as base. for the panel. I won't add a dependency against the kubernetes-mixin recording rules. Since I expect some large changes with #15, I plan to start after the cluster variable work is finished. |
Hi @jkroepke, |
I would prefer to split the Windows integration into different, smaller PRs. At minimum a PR for each dashboard. Let me know, when I can start with the cluster dashboard. |
@jkroepke The other issues/PRs are done, so you can start working on this whenever you want. |
Hi @jkroepke, |
The combind solution (one query for Windows and OS) results into very complex and un-understandable queries. Due complexity, external users may avoid to contribute improvement or they may destroy the Windows support by accident |
Agree. |
Maybe, no idea yet. I may have to ask at Grafana slack. |
Also, could be nice to add Windows support knowledge in the docs (README.md) for the other users. |
Hi @jkroepke, Do you plan to add Windows support on the other dashboards? |
Hi @dotdc, in general, yes. But time is a bit limited and I had to take a focus on windows_exporter. Once I'm done, I will continue. |
That's fine, let me know. |
Describe the enhancement you'd like
I have some cluster with Windows nodes enabled. I would like to ask if I can add windows support or do you think it out of context here?
Unlike
kubernetes-mixin
, which have separate dashboard, I would like to add the Windows queries into the existing one. Thats possible by using queries with OR, e.g.:Additional context
Since I'm running multiple OS hybrid clusters, I would like to add PRs for windows pods here. I'm not expecting that the maintainers here provide support for Windows. Before start to work here, I would like to know if its getting accepted?
The text was updated successfully, but these errors were encountered: