Skip to content
This repository has been archived by the owner on Dec 1, 2018. It is now read-only.

Use RSS instead of Usage when filtering out terminated containers #1807

Merged
merged 1 commit into from
Sep 13, 2017
Merged

Use RSS instead of Usage when filtering out terminated containers #1807

merged 1 commit into from
Sep 13, 2017

Conversation

piosz
Copy link
Contributor

@piosz piosz commented Sep 13, 2017

fix #1697

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Sep 13, 2017
@k8s-reviewable
Copy link

This change is Reviewable

@k8s-ci-robot k8s-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 13, 2017
@piosz
Copy link
Contributor Author

piosz commented Sep 13, 2017

cc @loburm @yujuhong

@yguo0905
Copy link

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 13, 2017
@yujuhong
Copy link

lgtm

@piosz piosz merged commit 0b96b6a into kubernetes-retired:master Sep 13, 2017
@piosz piosz deleted the mem-fix branch September 13, 2017 17:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Heapster should handle duplicated stats from kubelet gracefully
5 participants
@piosz @k8s-reviewable @yguo0905 @yujuhong @k8s-ci-robot