Skip to content
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

rkt: Avoid possible panic #3111

Merged
merged 1 commit into from
Aug 25, 2016
Merged

rkt: Avoid possible panic #3111

merged 1 commit into from
Aug 25, 2016

Conversation

@euank
Copy link
Member

@euank euank commented Aug 24, 2016

Fixes #3109

See the comment there for more details about this

@s-urbaniak s-urbaniak added this to the v1.14.0 milestone Aug 25, 2016
@s-urbaniak
Copy link
Contributor

s-urbaniak commented Aug 25, 2016

Semaphore doesn't let me rekick the build for some reason, but Jenkins is all green.

@s-urbaniak s-urbaniak merged commit 8d75491 into rkt:master Aug 25, 2016
@s-urbaniak
Copy link
Contributor

s-urbaniak commented Aug 25, 2016

Confirming running a while true; do echo "creating nginx"; kubectl create -f nginx.yaml; sleep 10; echo "deleting nginx"; kubectl delete -f nginx.yaml; sleep 10; done for the last 1.5hrs in the minikube buildroot image didn't mark the node being down 👍

@euank euank deleted the dodge-panic branch August 25, 2016 16:58
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

component/api server reviewed/lgtm

Projects

None yet

Milestone

v1.14.0

Development

Successfully merging this pull request may close these issues.

2 participants

@euank @s-urbaniak