-
Notifications
You must be signed in to change notification settings - Fork 2k
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
ReplicaSet unexpected type in v1.7.0-rc.0 #815
Comments
Thanks for reporting @brancz. I can push the fix for this. |
@brancz @tariq1890 Good catch and quick fix! 🎉 It would be a good idea to have a test to make sure the kube-state-metrics has 0 errors in the logs as part of the e2e test suite. Also erroring out and terminating the pod on any errors like this, so we (in CI) and the users (in their clusters) detect problems sooner and easier. I had an RBAC problem and while kube-state-metrics was up and running, certain resources were not working and while the logs were reporting not being able to list/watch those resource, I was expecting the pod to be terminated in that case and not just log that. WDYT? (Happy to tackle those if everyone agrees on those) |
No errors in logs is something we can do immediately I think (not blocking v1.7.0, but we can immediately get started working on that), go for it! :) Erroring out needs a bit more thought and major refactorings I think, I'm not even sure it's entirely possible with the way informers work today, but I'd be happy to see this! |
Closed by #816. @lilic feel free to create a separate ticket for what you mentioned in #815 (comment) |
Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
What happened:
Logs showed:
What you expected to happen:
Only non-error logs.
How to reproduce it (as minimally and precisely as possible):
Always with v1.7.0-rc.0
Anything else we need to know?:
n/a
Environment:
kubectl version
): v1.15.0 minikube@tariq1890 @lilic @mxinden @andyxning
The text was updated successfully, but these errors were encountered: