The code looks fine, but I'm slightly confused by the commit message. Can you explain why we're using assert modulename? Possible mention in the code as a comment as well. -- https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/325274 Your team cloud-init commiters is requested to review the proposed merge of ~powersj/cloud-init:style-fix 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