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

fix docs&&misc about delete pod_namespace #1661

Conversation

andyxning
Copy link
Contributor

This PR is followed by #1659.

I have forgot to grep pod_namespace usage instead of LabelPodNamespace. This PR will clean the usage about pod_namespace.

/ping @kubernetes/heapster-maintainers @piosz @DirectXMan12

This also change code with ES, so @AlmogBaku PTAL.

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

This change is Reviewable

Copy link
Contributor

@DirectXMan12 DirectXMan12 left a comment

Choose a reason for hiding this comment

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

seems reasonable to me 👍

Just want to make sure the other affected sinks are on board.

@AlmogBaku
Copy link
Contributor

AlmogBaku commented May 25, 2017 via email

@andyxning
Copy link
Contributor Author

@AlmogBaku pod_namespace has been commented as deprecated for more than two years. It has been deleted in #1659. This PR will clean all the references about pod_namespace. :)

@andyxning
Copy link
Contributor Author

Friendly ping @piosz @DirectXMan12

@andyxning
Copy link
Contributor Author

@piosz @DirectXMan12 PTAL. :)

Copy link
Contributor

@DirectXMan12 DirectXMan12 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@DirectXMan12 DirectXMan12 merged commit ec52db6 into kubernetes-retired:master May 31, 2017
@andyxning andyxning deleted the fix_docs_and_misc_about_delete_pod_namespace branch May 31, 2017 22:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants