From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Robert Yang <liezhi.yang@windriver.com>
Cc: bitbake-devel@lists.openembedded.org
Subject: Re: [PATCH 1/2] V2 Disk space monitoring
Date: Tue, 10 Jan 2012 17:51:09 +0000 [thread overview]
Message-ID: <1326217869.23315.5.camel@ted> (raw)
In-Reply-To: <bb164b343a162cee39a0ba2f76a7ffb906bc5381.1326199289.git.liezhi.yang@windriver.com>
On Tue, 2012-01-10 at 20:45 +0800, Robert Yang wrote:
> * Problems:
>
> 1) The disk space monitor is running in a sub process of
> bin/bitbake, so it can't use the same logger as bitbake(or I don't
> know how to use it, I've tried to use the logger.warn('some
> information'), but it would print nothing in the subprocess),
> another solution maybe use an independent logger in the subprocess,
> but it seems that it doesn't worth because of the two reasons:
> a) The logger of bin/bitbake can't control the monitor's
> independent logger
> b) The monitor doesn't need print many things
>
> It would be great if anyone could give me more instructions.
This is an issue unfortunately since its fine to have the warnings about
low space but its not much use unless bitbake can act on the warnings
somehow (stop executing more tasks for example).
I agree with Chris' comments about being able to specify the warning
threshold per directory being monitored too.
So I think before this can merge its going to need some further work. I
can't give pointers to the right way to send log messages from the
subprocess right at the moment. Perhaps you can insert yourself as an
"idle" task into bitbake's "main loop" instead of running as a
subprocess? This would also make it simpler to stop executing more
tasks. There probably should be a way for the user to choose
"immediately abort, no new tasks, show warning" as the options for
running out of space.
Cheers,
Richard
next prev parent reply other threads:[~2012-01-10 17:58 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-10 12:45 [PATCH 0/2] V2 Disk space monitoring Robert Yang
2012-01-10 12:45 ` [PATCH 1/2] " Robert Yang
2012-01-10 17:51 ` Richard Purdie [this message]
2012-01-10 19:54 ` Chris Larson
2012-01-10 12:45 ` [PATCH 2/2] V2 Add config sample for disk " Robert Yang
2012-01-10 16:50 ` [PATCH 0/2] V2 Disk " Chris Larson
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=1326217869.23315.5.camel@ted \
--to=richard.purdie@linuxfoundation.org \
--cc=bitbake-devel@lists.openembedded.org \
--cc=liezhi.yang@windriver.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.