diff mbox series

ptest-runner: Pull in parallel test fixes

Message ID 20230717143624.437293-1-richard.purdie@linuxfoundation.org
State New
Headers show
Series ptest-runner: Pull in parallel test fixes | expand

Commit Message

Richard Purdie July 17, 2023, 2:36 p.m. UTC
Pull in the commits:

    Change test timeout to be total elapsed time
    Report if child dies from a signal
    Recreate pipe for each test
    Revert "runner: Correctly handle running parallel tests"
    runner: Correctly handle running parallel tests

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Alexander Kanavin July 19, 2023, 8:26 p.m. UTC | #1
Unfortunately this seems to once again have regressed glib/codegen
ptest (the test writes large amounts to stdout):
https://bugzilla.yoctoproject.org/show_bug.cgi?id=15154

I'll send a revert.

Alex

On Mon, 17 Jul 2023 at 16:36, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
>
> Pull in the commits:
>
>     Change test timeout to be total elapsed time
>     Report if child dies from a signal
>     Recreate pipe for each test
>     Revert "runner: Correctly handle running parallel tests"
>     runner: Correctly handle running parallel tests
>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
>  meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> index 6f3104499f2..77e2b94f376 100644
> --- a/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> +++ b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> @@ -7,7 +7,7 @@ HOMEPAGE = "http://git.yoctoproject.org/cgit/cgit.cgi/ptest-runner2/about/"
>  LICENSE = "GPL-2.0-or-later"
>  LIC_FILES_CHKSUM = "file://LICENSE;md5=751419260aa954499f7abaabaa882bbe"
>
> -SRCREV = "8259375d306a8129f6c5d8528314496fc6ae1ca3"
> +SRCREV = "07d8a676aa962ecc5ec264ec33b0074adf2a8733"
>  PV .= "+git${SRCPV}"
>
>  SRC_URI = "git://git.yoctoproject.org/ptest-runner2;branch=master;protocol=https \
> --
> 2.39.2
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#184473): https://lists.openembedded.org/g/openembedded-core/message/184473
> Mute This Topic: https://lists.openembedded.org/mt/100195650/1686489
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kanavin@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Joshua Watt July 19, 2023, 8:32 p.m. UTC | #2
There is a newer commit that was merged to master just today; please
see if that fixes it.

On Wed, Jul 19, 2023 at 2:26 PM Alexander Kanavin
<alex.kanavin@gmail.com> wrote:
>
> Unfortunately this seems to once again have regressed glib/codegen
> ptest (the test writes large amounts to stdout):
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=15154
>
> I'll send a revert.
>
> Alex
>
> On Mon, 17 Jul 2023 at 16:36, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> >
> > Pull in the commits:
> >
> >     Change test timeout to be total elapsed time
> >     Report if child dies from a signal
> >     Recreate pipe for each test
> >     Revert "runner: Correctly handle running parallel tests"
> >     runner: Correctly handle running parallel tests
> >
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > ---
> >  meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> > index 6f3104499f2..77e2b94f376 100644
> > --- a/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> > +++ b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> > @@ -7,7 +7,7 @@ HOMEPAGE = "http://git.yoctoproject.org/cgit/cgit.cgi/ptest-runner2/about/"
> >  LICENSE = "GPL-2.0-or-later"
> >  LIC_FILES_CHKSUM = "file://LICENSE;md5=751419260aa954499f7abaabaa882bbe"
> >
> > -SRCREV = "8259375d306a8129f6c5d8528314496fc6ae1ca3"
> > +SRCREV = "07d8a676aa962ecc5ec264ec33b0074adf2a8733"
> >  PV .= "+git${SRCPV}"
> >
> >  SRC_URI = "git://git.yoctoproject.org/ptest-runner2;branch=master;protocol=https \
> > --
> > 2.39.2
> >
> >
> > -=-=-=-=-=-=-=-=-=-=-=-
> > Links: You receive all messages sent to this group.
> > View/Reply Online (#184473): https://lists.openembedded.org/g/openembedded-core/message/184473
> > Mute This Topic: https://lists.openembedded.org/mt/100195650/1686489
> > Group Owner: openembedded-core+owner@lists.openembedded.org
> > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kanavin@gmail.com]
> > -=-=-=-=-=-=-=-=-=-=-=-
> >
Alexander Kanavin July 19, 2023, 8:35 p.m. UTC | #3
Which newer commit, in which repo? I believe the latest SRCREV update
in oe-core master that landed 12 hours ago or so is what broke things.

Alex

On Wed, 19 Jul 2023 at 22:32, Joshua Watt <jpewhacker@gmail.com> wrote:
>
> There is a newer commit that was merged to master just today; please
> see if that fixes it.
>
> On Wed, Jul 19, 2023 at 2:26 PM Alexander Kanavin
> <alex.kanavin@gmail.com> wrote:
> >
> > Unfortunately this seems to once again have regressed glib/codegen
> > ptest (the test writes large amounts to stdout):
> > https://bugzilla.yoctoproject.org/show_bug.cgi?id=15154
> >
> > I'll send a revert.
> >
> > Alex
> >
> > On Mon, 17 Jul 2023 at 16:36, Richard Purdie
> > <richard.purdie@linuxfoundation.org> wrote:
> > >
> > > Pull in the commits:
> > >
> > >     Change test timeout to be total elapsed time
> > >     Report if child dies from a signal
> > >     Recreate pipe for each test
> > >     Revert "runner: Correctly handle running parallel tests"
> > >     runner: Correctly handle running parallel tests
> > >
> > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > > ---
> > >  meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> > > index 6f3104499f2..77e2b94f376 100644
> > > --- a/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> > > +++ b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> > > @@ -7,7 +7,7 @@ HOMEPAGE = "http://git.yoctoproject.org/cgit/cgit.cgi/ptest-runner2/about/"
> > >  LICENSE = "GPL-2.0-or-later"
> > >  LIC_FILES_CHKSUM = "file://LICENSE;md5=751419260aa954499f7abaabaa882bbe"
> > >
> > > -SRCREV = "8259375d306a8129f6c5d8528314496fc6ae1ca3"
> > > +SRCREV = "07d8a676aa962ecc5ec264ec33b0074adf2a8733"
> > >  PV .= "+git${SRCPV}"
> > >
> > >  SRC_URI = "git://git.yoctoproject.org/ptest-runner2;branch=master;protocol=https \
> > > --
> > > 2.39.2
> > >
> > >
> > > -=-=-=-=-=-=-=-=-=-=-=-
> > > Links: You receive all messages sent to this group.
> > > View/Reply Online (#184473): https://lists.openembedded.org/g/openembedded-core/message/184473
> > > Mute This Topic: https://lists.openembedded.org/mt/100195650/1686489
> > > Group Owner: openembedded-core+owner@lists.openembedded.org
> > > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kanavin@gmail.com]
> > > -=-=-=-=-=-=-=-=-=-=-=-
> > >
Joshua Watt July 19, 2023, 8:44 p.m. UTC | #4
Do you have a recent log of a failure? All the ones in the ticket
buildbot says are from a month ago (or, maybe that's just a quirk of
buildbot?)

On Wed, Jul 19, 2023 at 2:35 PM Alexander Kanavin
<alex.kanavin@gmail.com> wrote:
>
> Which newer commit, in which repo? I believe the latest SRCREV update
> in oe-core master that landed 12 hours ago or so is what broke things.
>
> Alex
>
> On Wed, 19 Jul 2023 at 22:32, Joshua Watt <jpewhacker@gmail.com> wrote:
> >
> > There is a newer commit that was merged to master just today; please
> > see if that fixes it.
> >
> > On Wed, Jul 19, 2023 at 2:26 PM Alexander Kanavin
> > <alex.kanavin@gmail.com> wrote:
> > >
> > > Unfortunately this seems to once again have regressed glib/codegen
> > > ptest (the test writes large amounts to stdout):
> > > https://bugzilla.yoctoproject.org/show_bug.cgi?id=15154
> > >
> > > I'll send a revert.
> > >
> > > Alex
> > >
> > > On Mon, 17 Jul 2023 at 16:36, Richard Purdie
> > > <richard.purdie@linuxfoundation.org> wrote:
> > > >
> > > > Pull in the commits:
> > > >
> > > >     Change test timeout to be total elapsed time
> > > >     Report if child dies from a signal
> > > >     Recreate pipe for each test
> > > >     Revert "runner: Correctly handle running parallel tests"
> > > >     runner: Correctly handle running parallel tests
> > > >
> > > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > > > ---
> > > >  meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb | 2 +-
> > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> > > > index 6f3104499f2..77e2b94f376 100644
> > > > --- a/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> > > > +++ b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> > > > @@ -7,7 +7,7 @@ HOMEPAGE = "http://git.yoctoproject.org/cgit/cgit.cgi/ptest-runner2/about/"
> > > >  LICENSE = "GPL-2.0-or-later"
> > > >  LIC_FILES_CHKSUM = "file://LICENSE;md5=751419260aa954499f7abaabaa882bbe"
> > > >
> > > > -SRCREV = "8259375d306a8129f6c5d8528314496fc6ae1ca3"
> > > > +SRCREV = "07d8a676aa962ecc5ec264ec33b0074adf2a8733"
> > > >  PV .= "+git${SRCPV}"
> > > >
> > > >  SRC_URI = "git://git.yoctoproject.org/ptest-runner2;branch=master;protocol=https \
> > > > --
> > > > 2.39.2
> > > >
> > > >
> > > > -=-=-=-=-=-=-=-=-=-=-=-
> > > > Links: You receive all messages sent to this group.
> > > > View/Reply Online (#184473): https://lists.openembedded.org/g/openembedded-core/message/184473
> > > > Mute This Topic: https://lists.openembedded.org/mt/100195650/1686489
> > > > Group Owner: openembedded-core+owner@lists.openembedded.org
> > > > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kanavin@gmail.com]
> > > > -=-=-=-=-=-=-=-=-=-=-=-
> > > >
Alexander Kanavin July 19, 2023, 8:47 p.m. UTC | #5
No, indeed they're all obsolete. They were added to the ticket today,
so I assumed they're fresh failures. Ignore the revert then, sorry for
hasty blaming.

Alex

On Wed, 19 Jul 2023 at 22:44, Joshua Watt <jpewhacker@gmail.com> wrote:
>
> Do you have a recent log of a failure? All the ones in the ticket
> buildbot says are from a month ago (or, maybe that's just a quirk of
> buildbot?)
>
> On Wed, Jul 19, 2023 at 2:35 PM Alexander Kanavin
> <alex.kanavin@gmail.com> wrote:
> >
> > Which newer commit, in which repo? I believe the latest SRCREV update
> > in oe-core master that landed 12 hours ago or so is what broke things.
> >
> > Alex
> >
> > On Wed, 19 Jul 2023 at 22:32, Joshua Watt <jpewhacker@gmail.com> wrote:
> > >
> > > There is a newer commit that was merged to master just today; please
> > > see if that fixes it.
> > >
> > > On Wed, Jul 19, 2023 at 2:26 PM Alexander Kanavin
> > > <alex.kanavin@gmail.com> wrote:
> > > >
> > > > Unfortunately this seems to once again have regressed glib/codegen
> > > > ptest (the test writes large amounts to stdout):
> > > > https://bugzilla.yoctoproject.org/show_bug.cgi?id=15154
> > > >
> > > > I'll send a revert.
> > > >
> > > > Alex
> > > >
> > > > On Mon, 17 Jul 2023 at 16:36, Richard Purdie
> > > > <richard.purdie@linuxfoundation.org> wrote:
> > > > >
> > > > > Pull in the commits:
> > > > >
> > > > >     Change test timeout to be total elapsed time
> > > > >     Report if child dies from a signal
> > > > >     Recreate pipe for each test
> > > > >     Revert "runner: Correctly handle running parallel tests"
> > > > >     runner: Correctly handle running parallel tests
> > > > >
> > > > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > > > > ---
> > > > >  meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb | 2 +-
> > > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > > >
> > > > > diff --git a/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> > > > > index 6f3104499f2..77e2b94f376 100644
> > > > > --- a/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> > > > > +++ b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
> > > > > @@ -7,7 +7,7 @@ HOMEPAGE = "http://git.yoctoproject.org/cgit/cgit.cgi/ptest-runner2/about/"
> > > > >  LICENSE = "GPL-2.0-or-later"
> > > > >  LIC_FILES_CHKSUM = "file://LICENSE;md5=751419260aa954499f7abaabaa882bbe"
> > > > >
> > > > > -SRCREV = "8259375d306a8129f6c5d8528314496fc6ae1ca3"
> > > > > +SRCREV = "07d8a676aa962ecc5ec264ec33b0074adf2a8733"
> > > > >  PV .= "+git${SRCPV}"
> > > > >
> > > > >  SRC_URI = "git://git.yoctoproject.org/ptest-runner2;branch=master;protocol=https \
> > > > > --
> > > > > 2.39.2
> > > > >
> > > > >
> > > > > -=-=-=-=-=-=-=-=-=-=-=-
> > > > > Links: You receive all messages sent to this group.
> > > > > View/Reply Online (#184473): https://lists.openembedded.org/g/openembedded-core/message/184473
> > > > > Mute This Topic: https://lists.openembedded.org/mt/100195650/1686489
> > > > > Group Owner: openembedded-core+owner@lists.openembedded.org
> > > > > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kanavin@gmail.com]
> > > > > -=-=-=-=-=-=-=-=-=-=-=-
> > > > >
diff mbox series

Patch

diff --git a/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
index 6f3104499f2..77e2b94f376 100644
--- a/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
+++ b/meta/recipes-support/ptest-runner/ptest-runner_2.4.2.bb
@@ -7,7 +7,7 @@  HOMEPAGE = "http://git.yoctoproject.org/cgit/cgit.cgi/ptest-runner2/about/"
 LICENSE = "GPL-2.0-or-later"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=751419260aa954499f7abaabaa882bbe"
 
-SRCREV = "8259375d306a8129f6c5d8528314496fc6ae1ca3"
+SRCREV = "07d8a676aa962ecc5ec264ec33b0074adf2a8733"
 PV .= "+git${SRCPV}"
 
 SRC_URI = "git://git.yoctoproject.org/ptest-runner2;branch=master;protocol=https \