Skip to content
This repository was archived by the owner on Apr 17, 2019. It is now read-only.

Conversation

@mwielgus
Copy link
Contributor

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Namespace?

@mwielgus mwielgus force-pushed the skip-kubernetes-system branch from 1f2fbdd to 272bf2e Compare May 24, 2016 10:35
@piosz
Copy link
Contributor

piosz commented May 24, 2016

LGTM

@piosz piosz added the lgtm Indicates that a PR is ready to be merged. label May 24, 2016
@mwielgus mwielgus merged commit 638a315 into kubernetes-retired:master May 24, 2016
@fgrzadkowski
Copy link
Contributor

I thought we agreed to wait with this PR. Let's discuss tomorrow.

-- sent from mobile device
24.05.2016 3:38 PM "Marcin Wielgus" notifications@github.com napisał(a):

Merged #1049 #1049.


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#1049 (comment)

mwielgus added a commit to kubernetes/autoscaler that referenced this pull request Apr 18, 2017
…-kubernetes-system

Cluster-autoscaler: do not delete nodes with kube-system, not-mirrored, not-daemon set pods
mwielgus added a commit to kubernetes/autoscaler that referenced this pull request Apr 18, 2017
…-kubernetes-system

Cluster-autoscaler: do not delete nodes with kube-system, not-mirrored, not-daemon set pods
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants