Re: [OE-core] [PATCH] qemurunner.py: print output from runqemu/qemu-system in stop()
On Thu, 2021-08-26 at 13:45 +0200, Alexander Kanavin wrote: > This is done when starting up qemu has failed, but is not done > when qemu started ok, but fails later in QMP communication. > > Output from runqemu does contain valuable information to find out > why, so rather than fix all the QMP fails to include it, let's just > print it in stop(). > > Signed-off-by: Alexander Kanavin > --- > meta/lib/oeqa/utils/qemurunner.py | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/meta/lib/oeqa/utils/qemurunner.py > b/meta/lib/oeqa/utils/qemurunner.py > index 5c9d2b24a3..204ad8b918 100644 > --- a/meta/lib/oeqa/utils/qemurunner.py > +++ b/meta/lib/oeqa/utils/qemurunner.py > @@ -535,6 +535,8 @@ class QemuRunner: > if self.runqemu.poll() is None: > self.logger.debug("Sending SIGKILL to runqemu") > os.killpg(os.getpgid(self.runqemu.pid), signal.SIGKILL) > +if not self.runqemu.stdout.closed: > +self.logger.info("Output from runqemu:\n%s" % > self.getOutput(self.runqemu.stdout)) > self.runqemu.stdin.close() > self.runqemu.stdout.close() > self.runqemu_exited = True This isn't quite so simple I'm afraid: https://autobuilder.yoctoproject.org/typhoon/#/builders/97/builds/3210/steps/16/logs/stdio Cheers, Richard -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155352): https://lists.openembedded.org/g/openembedded-core/message/155352 Mute This Topic: https://lists.openembedded.org/mt/85148684/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH] qemurunner.py: print output from runqemu/qemu-system in stop()
This is done when starting up qemu has failed, but is not done when qemu started ok, but fails later in QMP communication. Output from runqemu does contain valuable information to find out why, so rather than fix all the QMP fails to include it, let's just print it in stop(). Signed-off-by: Alexander Kanavin --- meta/lib/oeqa/utils/qemurunner.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/meta/lib/oeqa/utils/qemurunner.py b/meta/lib/oeqa/utils/qemurunner.py index 5c9d2b24a3..204ad8b918 100644 --- a/meta/lib/oeqa/utils/qemurunner.py +++ b/meta/lib/oeqa/utils/qemurunner.py @@ -535,6 +535,8 @@ class QemuRunner: if self.runqemu.poll() is None: self.logger.debug("Sending SIGKILL to runqemu") os.killpg(os.getpgid(self.runqemu.pid), signal.SIGKILL) +if not self.runqemu.stdout.closed: +self.logger.info("Output from runqemu:\n%s" % self.getOutput(self.runqemu.stdout)) self.runqemu.stdin.close() self.runqemu.stdout.close() self.runqemu_exited = True -- 2.31.1 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155349): https://lists.openembedded.org/g/openembedded-core/message/155349 Mute This Topic: https://lists.openembedded.org/mt/85148684/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH] qemurunner.py: print output from runqemu/qemu-system in stop()
Please tweak, I will separately confirm that the original issue (virgl headless fail on Ubuntu 20.04) would be still including this directly in ab output. Replying from a sushi restaurant :) Alex On Thu 26. Aug 2021 at 13.01, Richard Purdie < richard.pur...@linuxfoundation.org> wrote: > On Thu, 2021-08-26 at 12:45 +0200, Alexander Kanavin wrote: > > It’s not a real warning, just that the print itself is issued as a > warning. I > > think it just needs to be issued as info instead. > > Agreed, I realised that after replying to the patch. The current build is > going > rather orange! Will you send a v2 or should I tweak this one? > > Cheers, > > Richard > > -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155347): https://lists.openembedded.org/g/openembedded-core/message/155347 Mute This Topic: https://lists.openembedded.org/mt/85148684/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH] qemurunner.py: print output from runqemu/qemu-system in stop()
On Thu, 2021-08-26 at 12:45 +0200, Alexander Kanavin wrote: > It’s not a real warning, just that the print itself is issued as a warning. I > think it just needs to be issued as info instead. Agreed, I realised that after replying to the patch. The current build is going rather orange! Will you send a v2 or should I tweak this one? Cheers, Richard -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155346): https://lists.openembedded.org/g/openembedded-core/message/155346 Mute This Topic: https://lists.openembedded.org/mt/85148684/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH] qemurunner.py: print output from runqemu/qemu-system in stop()
It’s not a real warning, just that the print itself is issued as a warning. I think it just needs to be issued as info instead. Alex On Thu 26. Aug 2021 at 11.42, Richard Purdie < richard.pur...@linuxfoundation.org> wrote: > On Thu, 2021-08-26 at 00:35 +0200, Alexander Kanavin wrote: > > This is done when starting up qemu has failed, but is not done > > when qemu started ok, but fails later in QMP communication. > > > > Output from runqemu does contain valuable information to find out > > why, so rather than fix all the QMP fails to include it, let's just > > print it in stop(). > > > > Signed-off-by: Alexander Kanavin > > --- > > meta/lib/oeqa/utils/qemurunner.py | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/meta/lib/oeqa/utils/qemurunner.py > b/meta/lib/oeqa/utils/qemurunner.py > > index 5c9d2b24a3..e099c6e2a7 100644 > > --- a/meta/lib/oeqa/utils/qemurunner.py > > +++ b/meta/lib/oeqa/utils/qemurunner.py > > @@ -535,6 +535,8 @@ class QemuRunner: > > if self.runqemu.poll() is None: > > self.logger.debug("Sending SIGKILL to runqemu") > > os.killpg(os.getpgid(self.runqemu.pid), signal.SIGKILL) > > +if not self.runqemu.stdout.closed: > > +self.logger.warning("Output from runqemu:\n%s" % > self.getOutput(self.runqemu.stdout)) > > self.runqemu.stdin.close() > > self.runqemu.stdout.close() > > self.runqemu_exited = True > > I like the idea of this but it is showing warnings on the autobuilder: > > https://autobuilder.yoctoproject.org/typhoon/#/builders/61/builds/3877 > > So we may need to fix the issue it is exposing? > > Cheers, > > Richard > > -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155344): https://lists.openembedded.org/g/openembedded-core/message/155344 Mute This Topic: https://lists.openembedded.org/mt/85148684/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH] qemurunner.py: print output from runqemu/qemu-system in stop()
On Thu, 2021-08-26 at 00:35 +0200, Alexander Kanavin wrote: > This is done when starting up qemu has failed, but is not done > when qemu started ok, but fails later in QMP communication. > > Output from runqemu does contain valuable information to find out > why, so rather than fix all the QMP fails to include it, let's just > print it in stop(). > > Signed-off-by: Alexander Kanavin > --- > meta/lib/oeqa/utils/qemurunner.py | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/meta/lib/oeqa/utils/qemurunner.py > b/meta/lib/oeqa/utils/qemurunner.py > index 5c9d2b24a3..e099c6e2a7 100644 > --- a/meta/lib/oeqa/utils/qemurunner.py > +++ b/meta/lib/oeqa/utils/qemurunner.py > @@ -535,6 +535,8 @@ class QemuRunner: > if self.runqemu.poll() is None: > self.logger.debug("Sending SIGKILL to runqemu") > os.killpg(os.getpgid(self.runqemu.pid), signal.SIGKILL) > +if not self.runqemu.stdout.closed: > +self.logger.warning("Output from runqemu:\n%s" % > self.getOutput(self.runqemu.stdout)) > self.runqemu.stdin.close() > self.runqemu.stdout.close() > self.runqemu_exited = True I like the idea of this but it is showing warnings on the autobuilder: https://autobuilder.yoctoproject.org/typhoon/#/builders/61/builds/3877 So we may need to fix the issue it is exposing? Cheers, Richard -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155342): https://lists.openembedded.org/g/openembedded-core/message/155342 Mute This Topic: https://lists.openembedded.org/mt/85148684/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[OE-core] [PATCH] qemurunner.py: print output from runqemu/qemu-system in stop()
This is done when starting up qemu has failed, but is not done when qemu started ok, but fails later in QMP communication. Output from runqemu does contain valuable information to find out why, so rather than fix all the QMP fails to include it, let's just print it in stop(). Signed-off-by: Alexander Kanavin --- meta/lib/oeqa/utils/qemurunner.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/meta/lib/oeqa/utils/qemurunner.py b/meta/lib/oeqa/utils/qemurunner.py index 5c9d2b24a3..e099c6e2a7 100644 --- a/meta/lib/oeqa/utils/qemurunner.py +++ b/meta/lib/oeqa/utils/qemurunner.py @@ -535,6 +535,8 @@ class QemuRunner: if self.runqemu.poll() is None: self.logger.debug("Sending SIGKILL to runqemu") os.killpg(os.getpgid(self.runqemu.pid), signal.SIGKILL) +if not self.runqemu.stdout.closed: +self.logger.warning("Output from runqemu:\n%s" % self.getOutput(self.runqemu.stdout)) self.runqemu.stdin.close() self.runqemu.stdout.close() self.runqemu_exited = True -- 2.31.1 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155333): https://lists.openembedded.org/g/openembedded-core/message/155333 Mute This Topic: https://lists.openembedded.org/mt/85148684/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-