qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] test-qga failures on Fedora 24
@ 2016-07-06 21:08 John Snow
  2016-07-06 21:40 ` Marc-André Lureau
  0 siblings, 1 reply; 7+ messages in thread
From: John Snow @ 2016-07-06 21:08 UTC (permalink / raw)
  To: qemu-devel; +Cc: Marc-André Lureau, Michael Roth

I upgraded to Fedora 24 last Friday before the long (American) weekend.
Probably a bad idea.

Regardless, test-qga has developed a new symptom on current development
HEAD that I don't know what to make of:

https://paste.fedoraproject.org/388356/14678205/

Here's the meat of it:

  LINK  tests/test-qga
GTESTER tests/test-qga
**
ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:266:test_qga_get_fsinfo:
assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
such file or directory
GTester: last random seed: R02S9d55aa8939b9bd797279bd3f69d33466
**
ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:685:test_qga_blacklist:
assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
such file or directory
GTester: last random seed: R02Sb9bc461c2129c284f83033dca27827c6
/home/bos/jhuston/src/qemu/tests/Makefile.include:684: recipe for target
'check-tests/test-qga' failed
make: *** [check-tests/test-qga] Error 1


Any clues as to what's gone awry?

--js

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Qemu-devel] test-qga failures on Fedora 24
  2016-07-06 21:08 [Qemu-devel] test-qga failures on Fedora 24 John Snow
@ 2016-07-06 21:40 ` Marc-André Lureau
  2016-07-06 22:20   ` Michael Roth
  0 siblings, 1 reply; 7+ messages in thread
From: Marc-André Lureau @ 2016-07-06 21:40 UTC (permalink / raw)
  To: John Snow; +Cc: qemu-devel, Marc-André Lureau, Michael Roth

Hi

----- Original Message -----
> I upgraded to Fedora 24 last Friday before the long (American) weekend.
> Probably a bad idea.
> 
> Regardless, test-qga has developed a new symptom on current development
> HEAD that I don't know what to make of:
> 
> https://paste.fedoraproject.org/388356/14678205/
> 
> Here's the meat of it:
> 
>   LINK  tests/test-qga
> GTESTER tests/test-qga
> **
> ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:266:test_qga_get_fsinfo:
> assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
> such file or directory
> GTester: last random seed: R02S9d55aa8939b9bd797279bd3f69d33466
> **
> ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:685:test_qga_blacklist:
> assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
> such file or directory
> GTester: last random seed: R02Sb9bc461c2129c284f83033dca27827c6
> /home/bos/jhuston/src/qemu/tests/Makefile.include:684: recipe for target
> 'check-tests/test-qga' failed
> make: *** [check-tests/test-qga] Error 1
> 
> 
> Any clues as to what's gone awry?
> 


build_guest_fsinfo_for_device() failed, it's probably worth to investigate why the agent treats this as a fatal error, this would likely happen in a guest too. Could you try to reproduce in a VM or give clues on your setup to reproduce it?

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Qemu-devel] test-qga failures on Fedora 24
  2016-07-06 21:40 ` Marc-André Lureau
@ 2016-07-06 22:20   ` Michael Roth
  2016-07-06 22:41     ` John Snow
  0 siblings, 1 reply; 7+ messages in thread
From: Michael Roth @ 2016-07-06 22:20 UTC (permalink / raw)
  To: Marc-André Lureau, John Snow
  Cc: qemu-devel, " Marc-André Lureau"

Quoting Marc-André Lureau (2016-07-06 16:40:52)
> Hi
> 
> ----- Original Message -----
> > I upgraded to Fedora 24 last Friday before the long (American) weekend.
> > Probably a bad idea.
> > 
> > Regardless, test-qga has developed a new symptom on current development
> > HEAD that I don't know what to make of:
> > 
> > https://paste.fedoraproject.org/388356/14678205/
> > 
> > Here's the meat of it:
> > 
> >   LINK  tests/test-qga
> > GTESTER tests/test-qga
> > **
> > ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:266:test_qga_get_fsinfo:
> > assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
> > such file or directory
> > GTester: last random seed: R02S9d55aa8939b9bd797279bd3f69d33466
> > **
> > ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:685:test_qga_blacklist:
> > assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
> > such file or directory
> > GTester: last random seed: R02Sb9bc461c2129c284f83033dca27827c6
> > /home/bos/jhuston/src/qemu/tests/Makefile.include:684: recipe for target
> > 'check-tests/test-qga' failed
> > make: *** [check-tests/test-qga] Error 1
> > 
> > 
> > Any clues as to what's gone awry?
> > 
> 
> 
> build_guest_fsinfo_for_device() failed, it's probably worth to investigate why the agent treats this as a fatal error, this would likely happen in a guest too. Could you try to reproduce in a VM or give clues on your setup to reproduce it?
> 

The code assumes (non-virtual) block devices have a symlink at
/sys/dev/block/<major>:<minor>, maybe this isn't the case in Fedora
anymore?

The alternative is that realpath("/sys/dev/block/8:17", NULL) is what's actually
failing. That call assumes the first argument is a symlink, maybe that's
not the case on FC24?

Don't have an fc24 handy, but can check tomorrow.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Qemu-devel] test-qga failures on Fedora 24
  2016-07-06 22:20   ` Michael Roth
@ 2016-07-06 22:41     ` John Snow
  2016-07-06 23:02       ` Michael Roth
  0 siblings, 1 reply; 7+ messages in thread
From: John Snow @ 2016-07-06 22:41 UTC (permalink / raw)
  To: Michael Roth, Marc-André Lureau; +Cc: Marc-André Lureau, qemu-devel



On 07/06/2016 06:20 PM, Michael Roth wrote:
> Quoting Marc-André Lureau (2016-07-06 16:40:52)
>> Hi
>>
>> ----- Original Message -----
>>> I upgraded to Fedora 24 last Friday before the long (American) weekend.
>>> Probably a bad idea.
>>>
>>> Regardless, test-qga has developed a new symptom on current development
>>> HEAD that I don't know what to make of:
>>>
>>> https://paste.fedoraproject.org/388356/14678205/
>>>
>>> Here's the meat of it:
>>>
>>>   LINK  tests/test-qga
>>> GTESTER tests/test-qga
>>> **
>>> ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:266:test_qga_get_fsinfo:
>>> assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
>>> such file or directory
>>> GTester: last random seed: R02S9d55aa8939b9bd797279bd3f69d33466
>>> **
>>> ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:685:test_qga_blacklist:
>>> assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
>>> such file or directory
>>> GTester: last random seed: R02Sb9bc461c2129c284f83033dca27827c6
>>> /home/bos/jhuston/src/qemu/tests/Makefile.include:684: recipe for target
>>> 'check-tests/test-qga' failed
>>> make: *** [check-tests/test-qga] Error 1
>>>
>>>
>>> Any clues as to what's gone awry?
>>>
>>
>>
>> build_guest_fsinfo_for_device() failed, it's probably worth to investigate why the agent treats this as a fatal error, this would likely happen in a guest too. Could you try to reproduce in a VM or give clues on your setup to reproduce it?
>>
> 
> The code assumes (non-virtual) block devices have a symlink at
> /sys/dev/block/<major>:<minor>, maybe this isn't the case in Fedora
> anymore?
> 
> The alternative is that realpath("/sys/dev/block/8:17", NULL) is what's actually
> failing. That call assumes the first argument is a symlink, maybe that's
> not the case on FC24?
> 
> Don't have an fc24 handy, but can check tomorrow.
> 
> 

Good news: The problem went away.

It was likely due to an external XFS drive I have that perhaps didn't
properly register as ejected. I unmounted it and re-mounted it and now
everything's fine.

I'm not sure where it got 8:17, though. I guess mtab was stale?

Well, I learned a thing about test-qga today: If your local system is
weird it will explode and get angry at you for your trouble.

--js

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Qemu-devel] test-qga failures on Fedora 24
  2016-07-06 22:41     ` John Snow
@ 2016-07-06 23:02       ` Michael Roth
  2016-12-07 11:48         ` Peter Maydell
  0 siblings, 1 reply; 7+ messages in thread
From: Michael Roth @ 2016-07-06 23:02 UTC (permalink / raw)
  To: John Snow, Marc-André Lureau; +Cc: Marc-André Lureau, qemu-devel

Quoting John Snow (2016-07-06 17:41:32)
> 
> 
> On 07/06/2016 06:20 PM, Michael Roth wrote:
> > Quoting Marc-André Lureau (2016-07-06 16:40:52)
> >> Hi
> >>
> >> ----- Original Message -----
> >>> I upgraded to Fedora 24 last Friday before the long (American) weekend.
> >>> Probably a bad idea.
> >>>
> >>> Regardless, test-qga has developed a new symptom on current development
> >>> HEAD that I don't know what to make of:
> >>>
> >>> https://paste.fedoraproject.org/388356/14678205/
> >>>
> >>> Here's the meat of it:
> >>>
> >>>   LINK  tests/test-qga
> >>> GTESTER tests/test-qga
> >>> **
> >>> ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:266:test_qga_get_fsinfo:
> >>> assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
> >>> such file or directory
> >>> GTester: last random seed: R02S9d55aa8939b9bd797279bd3f69d33466
> >>> **
> >>> ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:685:test_qga_blacklist:
> >>> assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
> >>> such file or directory
> >>> GTester: last random seed: R02Sb9bc461c2129c284f83033dca27827c6
> >>> /home/bos/jhuston/src/qemu/tests/Makefile.include:684: recipe for target
> >>> 'check-tests/test-qga' failed
> >>> make: *** [check-tests/test-qga] Error 1
> >>>
> >>>
> >>> Any clues as to what's gone awry?
> >>>
> >>
> >>
> >> build_guest_fsinfo_for_device() failed, it's probably worth to investigate why the agent treats this as a fatal error, this would likely happen in a guest too. Could you try to reproduce in a VM or give clues on your setup to reproduce it?
> >>
> > 
> > The code assumes (non-virtual) block devices have a symlink at
> > /sys/dev/block/<major>:<minor>, maybe this isn't the case in Fedora
> > anymore?
> > 
> > The alternative is that realpath("/sys/dev/block/8:17", NULL) is what's actually
> > failing. That call assumes the first argument is a symlink, maybe that's
> > not the case on FC24?
> > 
> > Don't have an fc24 handy, but can check tomorrow.
> > 
> > 
> 
> Good news: The problem went away.
> 
> It was likely due to an external XFS drive I have that perhaps didn't
> properly register as ejected. I unmounted it and re-mounted it and now
> everything's fine.
> 
> I'm not sure where it got 8:17, though. I guess mtab was stale?

Yah, that would explain it. The values are pulled from mtab via
getmntent()

> 
> Well, I learned a thing about test-qga today: If your local system is
> weird it will explode and get angry at you for your trouble.

Yes, one of it's less-appreciated features :) This specific situation
should be accounted for though. I'll send a patch.

> 
> --js
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Qemu-devel] test-qga failures on Fedora 24
  2016-07-06 23:02       ` Michael Roth
@ 2016-12-07 11:48         ` Peter Maydell
  2016-12-08  7:47           ` Marc-André Lureau
  0 siblings, 1 reply; 7+ messages in thread
From: Peter Maydell @ 2016-12-07 11:48 UTC (permalink / raw)
  To: Michael Roth
  Cc: John Snow, Marc-André Lureau, Marc-André Lureau,
	qemu-devel

On 7 July 2016 at 00:02, Michael Roth <mdroth@linux.vnet.ibm.com> wrote:
> Quoting John Snow (2016-07-06 17:41:32)
>>
>>
>> On 07/06/2016 06:20 PM, Michael Roth wrote:
>> > Quoting Marc-André Lureau (2016-07-06 16:40:52)
>> >>> Here's the meat of it:
>> >>>
>> >>>   LINK  tests/test-qga
>> >>> GTESTER tests/test-qga
>> >>> **
>> >>> ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:266:test_qga_get_fsinfo:
>> >>> assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
>> >>> such file or directory
>> >>> GTester: last random seed: R02S9d55aa8939b9bd797279bd3f69d33466
>> >>> **
>> >>> ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:685:test_qga_blacklist:
>> >>> assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
>> >>> such file or directory
>> >>> GTester: last random seed: R02Sb9bc461c2129c284f83033dca27827c6
>> >>> /home/bos/jhuston/src/qemu/tests/Makefile.include:684: recipe for target
>> >>> 'check-tests/test-qga' failed
>> >>> make: *** [check-tests/test-qga] Error 1
>> >>>
>> >>>
>> >>> Any clues as to what's gone awry?

>> Good news: The problem went away.
>>
>> It was likely due to an external XFS drive I have that perhaps didn't
>> properly register as ejected. I unmounted it and re-mounted it and now
>> everything's fine.
>>
>> I'm not sure where it got 8:17, though. I guess mtab was stale?
>
> Yah, that would explain it. The values are pulled from mtab via
> getmntent()

I just ran into this today too (in my case due to an external
USB mass storage device that was powered-down without being
unmounted). It's kind of annoying that the test fails: why should
it care at all about what block devices you happen to have
on your system, whether they're mounted or busted or otherwise ?

thanks
-- PMM

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Qemu-devel] test-qga failures on Fedora 24
  2016-12-07 11:48         ` Peter Maydell
@ 2016-12-08  7:47           ` Marc-André Lureau
  0 siblings, 0 replies; 7+ messages in thread
From: Marc-André Lureau @ 2016-12-08  7:47 UTC (permalink / raw)
  To: Peter Maydell; +Cc: Michael Roth, John Snow, Marc-André Lureau, qemu-devel

Hi

----- Original Message -----
> On 7 July 2016 at 00:02, Michael Roth <mdroth@linux.vnet.ibm.com> wrote:
> > Quoting John Snow (2016-07-06 17:41:32)
> >>
> >>
> >> On 07/06/2016 06:20 PM, Michael Roth wrote:
> >> > Quoting Marc-André Lureau (2016-07-06 16:40:52)
> >> >>> Here's the meat of it:
> >> >>>
> >> >>>   LINK  tests/test-qga
> >> >>> GTESTER tests/test-qga
> >> >>> **
> >> >>> ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:266:test_qga_get_fsinfo:
> >> >>> assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
> >> >>> such file or directory
> >> >>> GTester: last random seed: R02S9d55aa8939b9bd797279bd3f69d33466
> >> >>> **
> >> >>> ERROR:/home/bos/jhuston/src/qemu/tests/test-qga.c:685:test_qga_blacklist:
> >> >>> assertion failed ret: GenericError realpath("/sys/dev/block/8:17"): No
> >> >>> such file or directory
> >> >>> GTester: last random seed: R02Sb9bc461c2129c284f83033dca27827c6
> >> >>> /home/bos/jhuston/src/qemu/tests/Makefile.include:684: recipe for
> >> >>> target
> >> >>> 'check-tests/test-qga' failed
> >> >>> make: *** [check-tests/test-qga] Error 1
> >> >>>
> >> >>>
> >> >>> Any clues as to what's gone awry?
> 
> >> Good news: The problem went away.
> >>
> >> It was likely due to an external XFS drive I have that perhaps didn't
> >> properly register as ejected. I unmounted it and re-mounted it and now
> >> everything's fine.
> >>
> >> I'm not sure where it got 8:17, though. I guess mtab was stale?
> >
> > Yah, that would explain it. The values are pulled from mtab via
> > getmntent()
> 
> I just ran into this today too (in my case due to an external
> USB mass storage device that was powered-down without being
> unmounted). It's kind of annoying that the test fails: why should
> it care at all about what block devices you happen to have
> on your system, whether they're mounted or busted or otherwise ?
> 

Since guest-get-fsinfo isn't always succeeding, I propose to either handle errors gracefully and/or to move it with other tests under QGA_TEST_SIDE_EFFECTING.

QGA_TEST_SIDE_EFFECTING is supposed to succeed under a test VM/container, it's probably best to just move the test there.

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2016-12-08  7:47 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-06 21:08 [Qemu-devel] test-qga failures on Fedora 24 John Snow
2016-07-06 21:40 ` Marc-André Lureau
2016-07-06 22:20   ` Michael Roth
2016-07-06 22:41     ` John Snow
2016-07-06 23:02       ` Michael Roth
2016-12-07 11:48         ` Peter Maydell
2016-12-08  7:47           ` Marc-André Lureau

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).