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' 

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).

Your team cloud init development team is requested to review the proposed merge 
of ~raharper/cloud-init:snapuser-create into cloud-init:master.

Mailing list: https://launchpad.net/~cloud-init-dev
Post to     : cloud-init-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~cloud-init-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to