8000 avoid self-killing on kubernetes by grosser · Pull Request #66 · alexei-led/pumba · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

avoid self-killing on kubernetes #66

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

Merged
merged 1 commit into from
Dec 28, 2017

Conversation

grosser
Copy link
Contributor
@grosser grosser commented Dec 18, 2017

solves #62

@Dieterbe

@grosser
Copy link
Contributor Author
grosser commented Dec 18, 2017

did not work :(

@Dieterbe
Copy link
Contributor

why are you mentioning me ? i haven't had a problem like this. i used to use my both my own docker image as well as the official one without this problem.

@grosser
Copy link
Contributor Author
grosser commented Dec 18, 2017

mentioned you for code-review :)

so strange that your own image worked for you ... I though this fix would work ...

I got it running with the label but it still kills itself :/

                "com.gaiaadm.pumba": "true",

@Dieterbe
Copy link
Contributor

to be clear i've just used these images in a local dockre-compose stack, not k8s

@grosser
Copy link
Contributor Author
grosser commented Dec 18, 2017

ah, that might be the difference ... I'll keep digging, no worries :D

@grosser grosser changed the title add instructions on how to avoid self-killing avoid self-killing on kubernetes Dec 19, 2017
@grosser
Copy link
Contributor Author
grosser commented Dec 19, 2017

found it ... good to merge ?

@alexei-led alexei-led merged commit 20d70db into alexei-led:master Dec 28, 2017
@grosser grosser deleted the grosser/labels branch December 28, 2017 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0