On Wed, 2021-10-13 at 18:18 +0200, Alexander Kanavin wrote:
> From: Alexander Kanavin <[email protected]>
> 
> In addition to the testresult json, testimage class now also
> provides the testimage task log and qemu console output log
> which can be useful for debugging test failures or
> even checking qemu test runs when failures did not happen.
> 
> Rather than duplicate specific file/folder names, let's copy all
> that is available, and define what is published in the testimage
> class itself (with appropriate folder structure if/when needed).
> At the moment there's just three files, and they are copied into
> folders named after image names, so there's no clutter or risk
> of mixing them up with unrelated logs.
> 
> [YOCTO #14518]
> 
> Signed-off-by: Alexander Kanavin <[email protected]>
> ---
>  scripts/collect-results | 8 +++-----
>  1 file changed, 3 insertions(+), 5 deletions(-)
> 
> diff --git a/scripts/collect-results b/scripts/collect-results
> index 93834d6..3663330 100755
> --- a/scripts/collect-results
> +++ b/scripts/collect-results
> @@ -3,11 +3,9 @@ WORKDIR=$1
>  DEST=$2
>  target=$3
>  
> -RESFILE=$WORKDIR/tmp/log/oeqa/testresults.json 
> -
> -if [ -e $RESFILE ]; then
> -    mkdir -p $DEST/$target
> -    cp $WORKDIR/tmp/log/oeqa/testresults.json  $DEST/$target/
> +mkdir -p $DEST
> +if [ -e $WORKDIR/tmp/log/oeqa/ ]; then
> +    cp -rf $WORKDIR/tmp/log/oeqa/ $DEST/$target
>  fi
>  

Since that build directory contains symlinks, we may need something like cp -L
here? I didn't test but I think we might run into an issue?

Cheers,

Richard

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#55073): https://lists.yoctoproject.org/g/yocto/message/55073
Mute This Topic: https://lists.yoctoproject.org/mt/86291876/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to