Re: [OE-core][dunfell 08/19] buildstats.bbclass: add functionality to collect build system stats

["Steve Sakoman" <steve@sakoman.com>]

On Mon, Jan 18, 2021 at 1:34 PM Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
>
> On Mon, 2021-01-18 at 12:36 -1000, Steve Sakoman wrote:
> > From: Sakib Sajal <sakib.sajal@windriver.com>
> >
> > There are a number of timeout and hang defects where
> > it would be useful to collect statistics about what
> > is running on a build host when that condition occurs.
> >
> > This adds functionality to collect build system stats
> > on a regular interval and/or on task failure. Both
> > features are disabled by default.
> >
> > To enable logging on a regular interval, set:
> > BB_HEARTBEAT_EVENT = "<interval>"
> > BB_LOG_HOST_STAT_ON_INTERVAL = <boolean>
> > Logs are stored in ${BUILDSTATS_BASE}/<build_name>/host_stats
> >
> > To enable logging on a task failure, set:
> > BB_LOG_HOST_STAT_ON_FAILURE = "<boolean>"
> > Logs are stored in ${BUILDSTATS_BASE}/<build_name>/build_stats
> >
> > The list of commands, along with the desired options, need
> > to be specified in the BB_LOG_HOST_STAT_CMDS variable
> > delimited by ; as such:
> > BB_LOG_HOST_STAT_CMDS = "command1 ; command2 ;... ;"
> >
> > Signed-off-by: Sakib Sajal <sakib.sajal@windriver.com>
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > (cherry picked from commit edb7098e9e0a8978568a45057c1c3ad2c6cacd67)
> > Signed-off-by: Steve Sakoman <steve@sakoman.com>
> > ---
> >  meta/classes/buildstats.bbclass | 40 ++++++++++++++++++++++++++++++---
> >  1 file changed, 37 insertions(+), 3 deletions(-)
>
> Not sure this is backport material. We should probably see how it goes
> in master and try using it there first?

OK, I thought it was safe since the features are disabled by default.
Seemed like something good to have if needed, but I'll remove this
from the pull request and revisit later.

Steve