Code review comment for lp:~bac/launchpad/lxc_cleanup

Revision history for this message
Robert Collins (lifeless) wrote :

Hi, there is already a script to delete containers; I think you should
run that script rather than inventing our own - its an abstraction
barrier. (E.g. this script should just identify the lxc containers to
remove and invoke lxc-destroy for each lxc container it no longer
wants.).

Also, this should be in lp-dev-utils as it is not part of LP itself.

Thanks,
Rob

« Back to merge proposal