Comment 2 for bug 1597754

FYI, this is fixed by my change If2fcc97928607189446e067886f70786bcc1cb3f,
which moves around lots of stuff in that function anyway. However, that's
buried in the middle of a big series. If you want to fix it anyway please
go ahead and I'll sort the merge conflict when it comes around.

On Mon, Jul 4, 2016 at 9:28 AM, Eli Qiao <email address hidden> wrote:

> I will take a look at this.
>
> ** Changed in: nova
> Assignee: (unassigned) => Eli Qiao (taget-9)
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1597754
>
> Title:
> Unable to boot instance using UML
>
> Status in OpenStack Compute (nova):
> New
>
> Bug description:
> CAVEAT: This is from code inspection only.
>
> Change I931421ea moved the following snippet of code:
>
> if CONF.libvirt.virt_type == 'uml':
> libvirt_utils.chown(image('disk').path, 'root')
>
> from the bottom of _create_image to the top. The problem is, the new
> location is before the creation of the root disk. This means that on
> initial creation we will run libvirt_utils.chown on a path which
> hasn't been created yet, which will cause an exception.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/nova/+bug/1597754/+subscriptions
>

--
Matthew Booth
Red Hat Engineering, Virtualisation Team

Phone: +442070094448 (UK)