Code review comment for ~raharper/cloud-init:snapuser-create

Revision history for this message
Scott Moser (smoser) wrote :

over all, looks good.
you dont have to clean up the handle, but if you see easy way to do that that'd be nice.

is the snappy path now valid on non-snappy system ? (ubuntu server with 'snap' support).

lastly, please just review your commit message and make sure its up to date with all changes (it may well be, just think you made some changes since you wrote it).

« Back to merge proposal