-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
Automated cherry pick of #15621 - initial empty SET PodUpdate #17947
Automated cherry pick of #15621 - initial empty SET PodUpdate #17947
Conversation
Labelling this PR as size/M |
@k8s-bot unit test this |
GCE e2e test build/test passed for commit 6269f12. |
@smarterclayton it's not a flake. Looking into it. |
6269f12
to
9e1e11f
Compare
GCE e2e build/test failed for commit 9e1e11fad7bf8b2809e23b77026422aa32fc32d5. |
@k8s-bot test this |
GCE e2e test build/test passed for commit 9e1e11fad7bf8b2809e23b77026422aa32fc32d5. |
In PodConfigNotificationIncremental PodConfig mode, when no pods are available for a source, the Merge function correctly concluded that neither ADD, UPDATE nor REMOVE updates are to be sent to the kubelet. But as a consequence the kubelet will not mark that source as seen. This is usually not a problem for the apiserver source. But it is a problem for an empty "file" source, e.g. by passing an empty directory to the kubelet for static pods. Then the file source will never be seen and the kubelet will stay in its special not-all-source-seen mode.
9e1e11f
to
560c830
Compare
Squashed and rebased, hopefully to a sane version without broken unit tests. |
GCE e2e test build/test passed for commit 560c830. |
@smarterclayton PTAL |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e test build/test passed for commit 560c830. |
Automatic merge from submit-queue |
…5621-upstream-release-1.1 Auto commit by PR queue bot
…pick-of-#15621-upstream-release-1.1 Auto commit by PR queue bot
…pick-of-#15621-upstream-release-1.1 Auto commit by PR queue bot
Cherry pick of #15621 on release-1.1.
This is part of a set of cherry-picks to fix Mesosphere e2e smoke tests for the release branch: mesosphere/kubernetes-mesos#640