From: Alexandre Belloni <alexandre.belloni@bootlin.com>
To: "Alexis Lothoré" <alexis.lothore@bootlin.com>,
"Mathieu Dubois-Briand" <mathieu.dubois-briand@bootlin.com>
Cc: Richard Purdie <richard.purdie@linuxfoundation.org>,
Openembedded-core@lists.openembedded.org,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
Steve Sakoman <steve@sakoman.com>
Subject: Re: [OE-core] [PATCH 2/2] oeqa/postactions: do not uncompress retrieved archive on host
Date: Thu, 19 Sep 2024 17:00:54 +0200 [thread overview]
Message-ID: <20240919150054634aadd3@mail.local> (raw)
In-Reply-To: <f2d9c941-46f9-410d-84a0-f1861b6e1418@bootlin.com>
On 19/09/2024 16:29:06+0200, Alexis Lothor� wrote:
> Hello Richard,
>
> On 9/19/24 14:58, Richard Purdie wrote:
> > On Thu, 2024-09-19 at 08:42 +0100, Richard Purdie via
> > lists.openembedded.org wrote:
> >> On Mon, 2024-08-12 at 17:14 +0200, Alexis Lothor� via
> >> lists.openembedded.org wrote:
> >>> From: Alexis Lothor� <alexis.lothore@bootlin.com>
>
> [...]
>
> >>> --- a/meta/lib/oeqa/utils/postactions.py
> >>> +++ b/meta/lib/oeqa/utils/postactions.py
> >>> @@ -68,7 +68,8 @@ def list_and_fetch_failed_tests_artifacts(d, tc,
> >>> artifacts_list, outputdir):
> >>> �������� (status, output) = tc.target.run(cmd, raw = True)
> >>> �������� if status != 0 or not output:
> >>> ������������ raise Exception("Error while fetching compressed
> >>> artifacts")
> >>> -������� p = subprocess.run(["tar", "zxf", "-", "-C", outputdir],
> >>> input=output)
> >>> +������� with open(archive_name, "wb") as f:
> >>> +����������� f.write(output)
> >>> ���� except Exception as e:
> >>> �������� bb.warn(f"Can not retrieve artifacts from test target:
> >>> {e}")
> >>
> >>
> >> This change is causing failures since archive_name isn't defined
> >> anywhere :/
> >
> > I'd also note that scarthgap actually throws errors:
> >
> > https://valkyrie.yoctoproject.org/#/builders/73/builds/135/steps/12/logs/stdio
> >
> > Exception: NameError: name 'artifact_path' is not defined
> >
> > which is related to some of these patches :/.
>
> I am out of office right now with limited access to computer (I come back next
> week but then I will be traveling to attend Kernel Recipes). If no one fixes it
> before, I'll take a look and send a fix once I return to the office (end of next
> week)
>
Mathieu is going to take care of this tomorrow/beginning of next week.
--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
prev parent reply other threads:[~2024-09-19 15:01 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-12 15:14 [PATCH 0/2] oeqa/postactions: save ptests artifacts as archive on host Alexis Lothoré
2024-08-12 15:14 ` [PATCH 1/2] oeqa/postactions: fix exception handling Alexis Lothoré
2024-08-12 15:14 ` [PATCH 2/2] oeqa/postactions: do not uncompress retrieved archive on host Alexis Lothoré
2024-09-19 7:42 ` [OE-core] " Richard Purdie
[not found] ` <17F6955C7A69DDF6.31299@lists.openembedded.org>
2024-09-19 12:58 ` Richard Purdie
2024-09-19 14:29 ` Alexis Lothoré
2024-09-19 15:00 ` Alexandre Belloni [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240919150054634aadd3@mail.local \
--to=alexandre.belloni@bootlin.com \
--cc=Openembedded-core@lists.openembedded.org \
--cc=alexis.lothore@bootlin.com \
--cc=mathieu.dubois-briand@bootlin.com \
--cc=richard.purdie@linuxfoundation.org \
--cc=steve@sakoman.com \
--cc=thomas.petazzoni@bootlin.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox