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

Elasticsearch data mount is chowned after container start #37219

Merged
merged 1 commit into from
Nov 22, 2016
Merged

Elasticsearch data mount is chowned after container start #37219

merged 1 commit into from
Nov 22, 2016

Conversation

crassirostris
Copy link

@crassirostris crassirostris commented Nov 21, 2016 •
edited by k8s-oncall

Fix #37030

@piosz

This change is Reviewable

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Nov 21, 2016
@k8s-github-robot k8s-github-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. release-note-label-needed labels Nov 21, 2016
@piosz piosz assigned piosz and unassigned eparis Nov 21, 2016
@piosz piosz added release-note-none Denotes a PR that doesn't merit a release note. and removed release-note-label-needed labels Nov 21, 2016
@piosz
Copy link
Member

piosz commented Nov 21, 2016

LGTM

@piosz piosz added lgtm "Looks good to me", indicates that a PR is ready to be merged. and removed lgtm "Looks good to me", indicates that a PR is ready to be merged. labels Nov 21, 2016
@piosz
Copy link
Member

piosz commented Nov 21, 2016

cc @xcompass @yifan-gu

@piosz
Copy link
Member

piosz commented Nov 21, 2016

Removed lgtm label because it needs to be tested in CoreOS env.

@k8s-ci-robot
Copy link
Contributor

Jenkins GCE e2e failed for commit 1c63e06. Full PR test history.

The magic incantation to run this job again is @k8s-bot cvm gce e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@crassirostris
Copy link
Author

@k8s-bot cvm gce e2e test this

@crassirostris crassirostris mentioned this pull request Nov 22, 2016
Merged
@crassirostris
Copy link
Author

@piosz Confirmed in #37030 (comment)

Can proceed with this PR and after it's merged, #37283

@piosz piosz added this to the v1.5 milestone Nov 22, 2016
@piosz piosz added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 22, 2016
@piosz
Copy link
Member

piosz commented Nov 22, 2016

Yet another bug fix

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit fd6a805 into kubernetes:master Nov 22, 2016
k8s-github-robot pushed a commit that referenced this pull request Nov 22, 2016
Merge pull request #37283 from Crassirostris/fluentd-es-image-version…
b3a89cc 
…-bump

Automatic merge from submit-queue

Bump fluentd-es image version

New image version, containing changes from #37123 and #37219

Should be merged only after those two

@piosz
@xcompass
Copy link

Yeh! Thanks everyone for this prompt fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@piosz piosz

Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Milestone
v1.5
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@crassirostris @piosz @k8s-ci-robot @k8s-github-robot @xcompass @eparis