Skip to content
This repository was archived by the owner on Apr 19, 2021. It is now read-only.

Conversation

@rogeruiz
Copy link

@rogeruiz rogeruiz commented Aug 3, 2017

Seeing this from CloudWatch whenever we try to drain minions with pods on them that have local data.

error: pods with local storage (use --delete-local-data to override): some-pod, another-pod

@rogeruiz rogeruiz requested review from LinuxBozo and jmcarp August 3, 2017 13:17
@LinuxBozo LinuxBozo merged commit ced3f74 into master Aug 3, 2017
@jmcarp
Copy link

jmcarp commented Aug 3, 2017

Remind me why pods need local data?

@LinuxBozo
Copy link

They don't. That's the point. But, in the case they do have any (say, a redis cluster that hasn't actually been configured for persistent volumes yet), we want to make sure they get nuked properly on drain.

@rogeruiz
Copy link
Author

rogeruiz commented Aug 3, 2017

Taking a further look at this too, we use local data for the binaries we ship with 18fgsa/redis-sentinel-init and 18fgsa/redis-server-init. We mount the /opt directory for the initContainers and add the configuration and bootstrap scripts in there.

@rogeruiz rogeruiz deleted the delete-local-data-on-drain branch August 3, 2017 17:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants