From: Cyril Hrubis <chrubis@suse.cz>
To: Dylan Jhong <dylan@andestech.com>
Cc: randolph@andestech.com, x5710999x@gmail.com,
dminus@andestech.com, alankao@andestech.com, ltp@lists.linux.it
Subject: Re: [LTP] [PATCH] pan/ltp-pan.c: Add execution time field in results/logs
Date: Thu, 11 Aug 2022 13:41:06 +0200 [thread overview]
Message-ID: <YvTq0qDxxFTarRbi@yuki> (raw)
In-Reply-To: <20220809082701.2366582-1-dylan@andestech.com>
Hi!
> The current log only shows the success or failure of testcases, but
> execution time is also one of the important indicators to observe LTP
> testcases.
>
> In this patch, we modified the format of the logs in results/ and added
> a column to record the execution time.
I think that there are at least two reasons not to apply this patch:
1. The ltp-pan is in maintenance mode only and is going to be replaced
by a different solution that is way more powerful [1]
2. There are likely automated tools parsing the ltp-pan logfiles that
may get broken by this
[1] https://lists.linux.it/pipermail/ltp/2022-August/029933.html
--
Cyril Hrubis
chrubis@suse.cz
--
Mailing list info: https://lists.linux.it/listinfo/ltp
prev parent reply other threads:[~2022-08-11 11:41 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-09 8:27 [LTP] [PATCH] pan/ltp-pan.c: Add execution time field in results/logs Dylan Jhong
2022-08-11 11:41 ` Cyril Hrubis [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=YvTq0qDxxFTarRbi@yuki \
--to=chrubis@suse.cz \
--cc=alankao@andestech.com \
--cc=dminus@andestech.com \
--cc=dylan@andestech.com \
--cc=ltp@lists.linux.it \
--cc=randolph@andestech.com \
--cc=x5710999x@gmail.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.