From a1c4d6ac73a5240ba308deffe8c2eb706af90ccd Mon Sep 17 00:00:00 2001 From: "Chris St. Pierre" Date: Tue, 16 Oct 2012 10:01:09 -0400 Subject: Packages: better error handling for GPG key downloading --- src/lib/Bcfg2/Server/Plugins/Packages/__init__.py | 18 +++++++++++++++--- 1 file changed, 15 insertions(+), 3 deletions(-) (limited to 'src/lib/Bcfg2') diff --git a/src/lib/Bcfg2/Server/Plugins/Packages/__init__.py b/src/lib/Bcfg2/Server/Plugins/Packages/__init__.py index f0ddaacbe..f1ffb1d67 100644 --- a/src/lib/Bcfg2/Server/Plugins/Packages/__init__.py +++ b/src/lib/Bcfg2/Server/Plugins/Packages/__init__.py @@ -440,9 +440,21 @@ class Packages(Bcfg2.Server.Plugin.Plugin, not os.path.exists(localfile)): self.logger.info("Packages: Downloading and parsing %s" % key) - response = urlopen(key) - open(localfile, 'w').write(response.read()) - keys.append(key) + try: + open(localfile, 'w').write(urlopen(key).read()) + keys.append(key) + except HTTPError: + err = sys.exc_info()[1] + self.logger.error("Packages: Error downloading %s: %s" + % (key, err)) + except IOError: + err = sys.exc_info()[1] + self.logger.error("Packages: Error writing %s to %s: " + "%s" % (key, localfile, err)) + except: + err = sys.exc_info()[1] + self.logger.error("Packages: Unknown error fetching " + "%s: %s" % (key, err)) for kfile in glob.glob(os.path.join(self.keypath, "*")): if kfile not in keyfiles: -- cgit v1.2.3-1-g7c22