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). -- https://code.launchpad.net/~raharper/cloud-init/+git/cloud-init/+merge/304700 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