Code review comment for lp:~zulcss/openstack-charm-testing/lxd-qa

Revision history for this message
Ryan Beisner (1chb1n) wrote :

Looking good! Thanks for the rearrangement.

It's ok to remove the isolated dir since the source dir is doing lxd from source with all other services from distro/cloud archive.

Please see inline comments on 'bundles/lxd/source/next.yaml' and apply those changes to both source bundles. The goal there is to remove irrelevant and unneeded legacy config data from the bundle.

We should do the same on lxd/next.yaml and lxd/default.yaml, but that doesn't necessarily have to accompany this MP.

review: Needs Fixing

« Back to merge proposal