Code review comment for ~raharper/curtin:feature/enable-storage-vmtest-on-centos

Revision history for this message
Ryan Harper (raharper) wrote :

On Thu, Aug 9, 2018 at 2:30 PM Scott Moser <email address hidden> wrote:
>
> Review: Approve
>
> I think i'm pretty much fine with this at this point.
> mega-branch, but we can take and address any issues one by one.
>
> Assuming the following are happy, I approve:
> a.) rharper

+1

> b.) vmtest

I'll kick off a full run on diglett; this allows me to "hack" in an
updated curtin-hooks.py for the centos images
However, we shouldn't land this until we get the MAAS image branch
approved and landed.

> c.) c-i bot
>
> --
> https://code.launchpad.net/~raharper/curtin/+git/curtin/+merge/349075
> You are the owner of ~raharper/curtin:feature/enable-storage-vmtest-on-centos.

« Back to merge proposal