[elbe-devel] [PATCH 00/12] obey logging error

Bastian Germann bage at linutronix.de
Fri Sep 27 17:37:47 CEST 2019


> currently logged errors do not result in a build 
> marked bad, and elbe initvm submit returning error...
> 
> this needs a lot of fixes in multiple places.
> 
> also contains a lot of pylint stuff, i have found along
> the way.
> 
> 1. make the log remeber that an error has been logged.
> 2. allow to ask the project status via soap, which already covers most
>    erors
> 3. make "elbe control wait_busy" check the status after wait
> 4. make "elbe initvm submit" handle errors in "wait_busy"
> 5. actually make the log error relevant by promoting it into prj
>    status
> 
> numbers are NOT patch numbers.
> 
> Torben Hohn (12):
>   log: create class LoggingQueue(collections.deque)
>   pylint esoap: disable no-self-use in get_version(),
>     install_elbe_version()
>   pylint esoap: instead of disabling unused-argument, check access
>   pylint esoap: fix signature of ESoap.get_files()
>   pylint esoap: mak shutdown_initvm() @authenticated_admin and
>     no-self-use
>   esoap: return ELBE-FINISH only when the msg queue is empty
>   esoap: implement get_project() returning a single SoapProject
>   soapclient: make wait_busy check for build success and exit if not
>   initvmaction: be verbose, when "elbe control wait_busy" fails
>   log: save maximum loglevel in LoggingQueue.max_level and export
>     accessor
>   pylint log: fix style problems and one pylint error
>   asyncworker: use read_maxlevel() and honor it when there is an error
> 
>  elbepack/asyncworker.py        | 72 ++++++++++++++++++++++++----------
>  elbepack/daemons/soap/esoap.py | 30 +++++++++-----
>  elbepack/initvmaction.py       | 41 ++++++++++++++++---
>  elbepack/log.py                | 47 +++++++++++++++++++---
>  elbepack/soapclient.py         |  9 +++++
>  5 files changed, 157 insertions(+), 42 deletions(-)
> 

For patches 1-3 and 7-12
Reviewed-by: Bastian Germann <bage at linutronix.de>



More information about the elbe-devel mailing list