Include docker-cleanup rpm package by default, with appropriate settings.

[sehqlr]

As of right now, we have a rpm package for docker-cleanup, but it is not installed by default. We need to include the install as well as appropriate exclusion settings. For example, zookeeper keeps data in a volume that could be collected, and break things. (Thanks for pointing this out, @ryane). Testing needs to be done to make sure that this doesn't happen.

@stevendborrelli, is this something that we need to include in 0.5? What do you think?

[stevendborrelli]

Yes, it should be part of the docker role.

[eirslett]

The zookeeper case could probably be solved by moving zookeeper out of docker? #765

[stevendborrelli]

@eirslett we're planning on refactoring zk, but it won't make it for the 0.5 release.

[sehqlr]

I've tested my changes for excluding the zookeeper data volume from cleanup, and it works as expected. Are there other volumes that need to be excluded, or is this a go for a PR?

[stevendborrelli]

Yes. Update this ticket with the PR.

[sehqlr]

PR #861 is ready for testing

[stevendborrelli]

Closing this due to docker-gc installation.