[PATCH] Show more info about source failure in the spoke

[Brian C. Lane]

From: "Brian C. Lane"

When package metadata fails to download suggest that the user check the
repo url and, if enabled, the proxy settings.
---
 pyanaconda/ui/gui/spokes/source.py |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/pyanaconda/ui/gui/spokes/source.py b/pyanaconda/ui/gui/spokes/source.py
index 7752439..68020c8 100644
--- a/pyanaconda/ui/gui/spokes/source.py
+++ b/pyanaconda/ui/gui/spokes/source.py
@@ -500,11 +500,17 @@ class SourceSpoke(NormalSpoke):
         try:
             self.payload.updateBaseRepo(self.storage, fallback=False)
         except PayloadError as e:
+            log.error("PayloadError: %s" % (e,))
             self._error = True
             communication.send_message(self.__class__.__name__,
                                        _("Failed to set up install source"))
+            if not self.data.method.proxy:
+                self.window.set_info(Gtk.MessageType.WARNING, _("Failed to set up install source, check the repo url"))
+            else:
+                self.window.set_info(Gtk.MessageType.WARNING, _("Failed to set up install source, check the repo url and proxy settings"))
         else:
             self._error = False
+            self.window.clear_info()
             threadMgr.add(AnacondaThread(name="AnaPayloadMDThread",
                                          target=self.getRepoMetadata))

[Jesse Keating]

On 07/11/2012 01:23 PM, Brian C. Lane wrote:
> From: "Brian C. Lane"
>
> When package metadata fails to download suggest that the user check the
> repo url and, if enabled, the proxy settings.
>
> pyanaconda/ui/gui/spokes/source.py |    6 ++++++
>   1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/pyanaconda/ui/gui/spokes/source.py b/pyanaconda/ui/gui/spokes/source.py
> index 7752439..68020c8 100644
> --- a/pyanaconda/ui/gui/spokes/source.py
> +++ b/pyanaconda/ui/gui/spokes/source.py
> @@ -500,11 +500,17 @@ class SourceSpoke(NormalSpoke):
>           try:
>               self.payload.updateBaseRepo(self.storage, fallback=False)
>           except PayloadError as e:
>        log.error("PayloadError: %s" % (e,))
>         self._error = True
>         communication.send_message(self.__class__.__name__,
>                                    _("Failed to set up install source"))
>        if not self.data.method.proxy:
>            self.window.set_info(Gtk.MessageType.WARNING, _("Failed to set up install source, check the repo url"))
>        else:
>            self.window.set_info(Gtk.MessageType.WARNING, _("Failed to set up install source, check the repo url and proxy settings"))
>     else:
>         self._error = False
>        self.window.clear_info()
>         threadMgr.add(AnacondaThread(name="AnaPayloadMDThread",
>                                      target=self.getRepoMetadata))

ACK