[livemedia-creator 1/4] livemedia-creator: add more errors

Vratislav Podzimek vpodzime at redhat.com
Thu Jul 3 07:09:52 UTC 2014


On Wed, 2014-07-02 at 15:17 -0700, Brian C. Lane wrote:
> Add check for running traceback script and when the retry fails.
> 
> This also indicates something went wrong with the installation, exit
> when they are seen in the logs.
> 
> Also drop looking for WARNING in the regex errors, they will be errors
> after the syslog level name remap patch goes into anaconda.
> ---
>  src/sbin/livemedia-creator | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/src/sbin/livemedia-creator b/src/sbin/livemedia-creator
> index e5f4f77..378282f 100755
> --- a/src/sbin/livemedia-creator
> +++ b/src/sbin/livemedia-creator
> @@ -143,9 +143,11 @@ class LogRequestHandler(SocketServer.BaseRequestHandler):
>          simple_tests = ["Traceback (",
>                          "Out of memory:",
>                          "Call Trace:",
> -                        "insufficient disk space:"]
> -        re_tests =     [r"WARNING packaging: base repo .* not valid",
> -                        r"WARNING packaging: .* requires .*"]
> +                        "insufficient disk space:",
> +                        "error populating transaction after",
> +                        "traceback script(s) have been run"]
> +        re_tests =     [r"packaging: base repo .* not valid",
> +                        r"packaging: .* requires .*"]
>          for t in simple_tests:
>              if t in line:
>                  self.server.log_error = True
These all look good to me.

-- 
Vratislav Podzimek

Anaconda Rider | RHCE | Red Hat, Inc. | Brno - Czech Republic




More information about the anaconda-patches mailing list