Code review comment for lp:~julian-edwards/juju-core/maas-uuid-file-prefix

Revision history for this message
Julian Edwards (julian-edwards) wrote :

On Wednesday 16 Oct 2013 05:51:10 you wrote:
> Seems to be missing a test for having no environment-uuid at all? Otherwise
> LGTM (modulo what it's building on...).

That's what TesttprefixWithPrivateNamespaceIgnoresEmptyUUID is for.

Thanks for the review.

« Back to merge proposal