Skip to content
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.

Sign up for GitHub

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

Jump to bottom

cAdvisor shouldn't be publicly exposed by default #321

Closed
luxas opened this issue Jun 26, 2017 · 1 comment · Fixed by kubernetes/kubernetes#48042
Closed

cAdvisor shouldn't be publicly exposed by default #321

luxas opened this issue Jun 26, 2017 · 1 comment · Fixed by kubernetes/kubernetes#48042
Assignees
@luxas
Labels
area/security priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release.
Milestone
v1.7

Comments

@luxas
Copy link
Member

luxas commented Jun 26, 2017

More information in: kubernetes/release#356 and kubernetes/kubernetes#48042
@luxas luxas self-assigned this Jun 26, 2017
@luxas luxas changed the title cAdvisor shouldn't be publicly exposed cAdvisor shouldn't be publicly exposed by default Jun 26, 2017
@luxas luxas added area/security kind/enhancement priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Jun 26, 2017
@luxas luxas added this to the v1.7 milestone Jun 26, 2017
This was referenced Jun 26, 2017
Merged
Merged
k8s-github-robot pushed a commit to kubernetes/kubernetes that referenced this issue Jun 27, 2017
Merge pull request #48042 from luxas/update_kubeadm_deb
51c43ed 
Automatic merge from submit-queue (batch tested with PRs 48139, 48042, 47645, 48054, 48003)

Reflect kubeadm-specific kubelet changes in the bazel debs

**What this PR does / why we need it**:

Sync this file with the one in kubernetes/release

cc @pipejakob @mikedanese @ixdy 

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #

Fixes: kubernetes/kubeadm#321

**Special notes for your reviewer**:

**Release note**:

```release-note
NONE
```
@luxas
Copy link
Member Author

luxas commented Jun 29, 2017 •
edited
Loading

Required changes have merged.
kubernetes/kubernetes: kubernetes/kubernetes#48042
kubernetes/release: kubernetes/release#356
Docs: kubernetes/website#4229
Release note: kubernetes/enhancements#320

@luxas luxas closed this as completed Jun 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@luxas luxas

Labels
area/security priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release.
Projects
None yet
Milestone
v1.7
Development

Successfully merging a pull request may close this issue.

Reflect kubeadm-specific kubelet changes in the bazel debs luxas/kubernetes
1 participant
@luxas