-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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.
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
UPSTREAM: 45286: When pods are terminated we should detach the volume #14191
UPSTREAM: 45286: When pods are terminated we should detach the volume #14191
Conversation
[test][merge] |
Are there ansible changes planned to set this annotation? |
@smarterclayton there are no ansible change necessary to set this annotation. The annotation solely exists to make sure if at all kubelet was started with We are going to deprecated that flag. kubernetes/kubernetes#45615 |
please don't set the annotation. ever. period. no. |
[merge] |
Two unrelated flakes [test]
…On Tue, May 16, 2017 at 12:05 PM, OpenShift Bot ***@***.***> wrote:
continuous-integration/openshift-jenkins/test FAILURE (
https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/1488/)
(Base Commit: b96ef9a
<b96ef9a>
)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#14191 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABG_p5QQpUKmUSpPRiVlgGHuFTYWEeHjks5r6cllgaJpZM4NbJVd>
.
|
[test] #10773 |
Evaluated for origin test up to 64acaee |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/1509/) (Base Commit: 3949203) |
[merge] flake on oc get frontend pods in 60s
…On Thu, May 18, 2017 at 8:37 AM, OpenShift Bot ***@***.***> wrote:
continuous-integration/openshift-jenkins/merge FAILURE (
https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin/690/)
(Base Commit: fb92178
<fb92178>
)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#14191 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABG_p7bQEUUVwhnkV0frvEL6UPkzrwldks5r7DuWgaJpZM4NbJVd>
.
|
Evaluated for origin merge up to 64acaee |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin/707/) (Base Commit: 2ceece1) (Image: devenv-rhel7_6242) |
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1446788