[PATCH 1/2] Log the reason if GUI import fails (#1229747)

Anne Mulhern amulhern at redhat.com
Wed Jun 17 12:49:00 UTC 2015





----- Original Message -----
> From: "Martin Kolman" <mkolman at redhat.com>
> To: anaconda-patches at lists.fedorahosted.org
> Sent: Tuesday, June 16, 2015 12:23:09 PM
> Subject: [PATCH 1/2] Log the reason if GUI import fails (#1229747)
> 
> Log the exception that caused the import to fail.
> 
> Related: rhbz#1229747
> Signed-off-by: Martin Kolman <mkolman at redhat.com>
> ---
>  initial_setup/__main__.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/initial_setup/__main__.py b/initial_setup/__main__.py
> index 06b6ea5..642e93c 100644
> --- a/initial_setup/__main__.py
> +++ b/initial_setup/__main__.py
> @@ -108,7 +108,7 @@ if mode == "gui":
>          log.debug("trying to import GUI")
>          import gui
>      except ImportError:
> -        log.error("GUI import failed, falling back to TUI")
> +        log.exception("GUI import failed, falling back to TUI")
>          mode = "tui"
>  
>  if mode == "gui":
> --
> 2.4.2
> 
> _______________________________________________
> anaconda-patches mailing list
> anaconda-patches at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches
> 

In itself this looks fine to me, but I remember that long ago the presence of
tracebacks in the blivet log caused problems for some other tools that searched
for tracebacks as a sign of failure. It hindered some of bcl's building for a while.

See blivet commit f3ff0a62b029525c83253b07800edaf2868c67cd and lorax commit
29de1d43a5f9f2574d94824bdc0dd45742763abb for some details.

- mulhern


More information about the anaconda-patches mailing list