-
Notifications
You must be signed in to change notification settings - Fork 38.9k
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
Remove periodSeconds, unavailable in 1.1 #18257
Conversation
@zmerlynn can I put this directly into 1.1, since HEAD onwards can still use the field? I can make the corresponding change in HEAD and cherry pick if you prefer (or just revert the previous cherry-pick, tests should pass since we had 2 greens inbetween). |
Labelling this PR as size/XS |
@k8s-bot unit test this please |
GCE e2e build/test failed for commit f4ce3d2. |
@k8s-bot test this please |
e2e failure looked unrelated |
GCE e2e test build/test passed for commit f4ce3d2. |
@k8s-bot unit test this please |
LGTM |
Green, merging. |
Remove periodSeconds, unavailable in 1.1
Remove periodSeconds, unavailable in 1.1
Remove periodSeconds, unavailable in 1.1
Didn't realize "periodSeconds" is from: 1e88a68, which is not in 1.1. So currently the gke-e2e cluster doesn't even start the loadbalancer controller.