-
Notifications
You must be signed in to change notification settings - Fork 1
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 metrics-server to MDE #4
Comments
we need to do a bit of work internally to output the correct cert to use here before we get this merged up. Kubadm apparently does not use the same cert. see issue here : kubernetes/kubeadm#1223 |
Starting to think that, given how few people set this up securely or even have their kubelets configred such that a secure install is possible, including this in MDE would add prerequisite steps that are unrealistic for the vast majority of users. Maybe if revelry really wants this, we should just drop our own HelmRelease into our own flux repos instead of having MDE do it. |
Based on what I've been told, and following the convo re: possible solutions, I agree. We can/should drop this from MDE and let it be a manual install if needed until the security issue(s) are resolved. Let's assign @bit-herder to close this out or make an argument for including it, but with some changes. |
Thats fine for now. Closing. |
Background:
kubectl top pods
orkubectl top nodes
without metrics-serverA/C:
The text was updated successfully, but these errors were encountered: