[PATCH 2/2] Change "failed to download" messages from critical to warning.

Chris Lumens clumens at redhat.com
Tue Sep 1 14:58:16 UTC 2015


This message gets logged if a single mirror fails, in which case DNF
moves on to another.  That's not critical, and the presence of that log
level is causing kickstart_tests to assume a test failed.  We also still
have an exception that gets raised should DNF run out of mirrors which
handles the really critical case.
---
 pyanaconda/packaging/dnfpayload.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pyanaconda/packaging/dnfpayload.py b/pyanaconda/packaging/dnfpayload.py
index ee999d4..6df8e37 100644
--- a/pyanaconda/packaging/dnfpayload.py
+++ b/pyanaconda/packaging/dnfpayload.py
@@ -167,7 +167,7 @@ class DownloadProgress(dnf.callback.DownloadProgress):
             self.downloads[nevra] = payload.download_size
             self._update()
             return
-        log.critical("Failed to download '%s': %d - %s", nevra, status, err_msg)
+        log.warning("Failed to download '%s': %d - %s", nevra, status, err_msg)
 
     def progress(self, payload, done):
         nevra = str(payload)
-- 
2.4.3



More information about the anaconda-patches mailing list