Code review comment for ~powersj/cloud-init:fix-integration-ntp

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

I am pretty sure that you should change the docstring for 'test_ntp_entires'. Ie, you're changing 'ntp_conf_pools' to 'ntp_conf_servers' inside a function with a doc string "Test config pools entries".

And while you're there, you might as well spell entries right in the function name.

review: Needs Fixing

« Back to merge proposal