-
Notifications
You must be signed in to change notification settings - Fork 40k
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 the broken kubemark-5 suite #25997
Conversation
You guys have identified the root cause of the breakage? I will also revert the support from my PR #25927 |
We are trying to root cause it now. We have a repro case. On Fri, May 20, 2016 at 3:25 PM, Andy Zheng notifications@github.com
|
We haven't decided if we should merge this or try for a fix yet. @mhrgoog and @mikedanese have a repro and are looking at it right now. Other things are blocking the queue at the moment so we don't have to decide right away. |
Labelling this PR as size/XL |
GCE e2e build/test passed for commit 107d68d. |
So mike and I poked at this and I think we found the issue. After sshing to We looked at the following files. $ cat /var/log/controller.log k8s.io/kubernetes/pkg/controller/node.(_NodeController).filterOutServiceRange(0xc820144000) ... The line in question is: nodecontroller.go:342 if !nc.clusterCIDR.Contains(nc.serviceCIDR.IP.Mask(nc.clusterCIDR.Mask)) && Mike guessed that nc.serviceCIDR or nc.clusterCIDRwas a nil pointer. Then we read controllers_flags $ cat controllers_flags Here he saw that SERVICE_CLUSTER_IP_RANGE is not set. Mike figured out that this was due to a flag that is now required to be set Once we set the flag we re-ran test/kubemark/run-e2e-tests.sh and now it is mrubin On Fri, May 20, 2016 at 3:33 PM, Daniel Smith notifications@github.com
|
This is a rollback for PR #25984 and #19242.
The work here is typed by mhrgoog and directed by his overlords @lavalamp @mikedanese.