diff options
author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2011-10-19 15:57:48 -0400 |
---|---|---|
committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2011-10-19 15:57:48 -0400 |
commit | 6c681bd7340c8526631766afe92e4f9f02117834 (patch) | |
tree | e5a6be9fa52363c94c2c34b94cb855cfab9af796 | |
parent | c4154ca4d2b7b2a7bcc266d3f473ff1d247cac9f (diff) | |
download | bcfg2-6c681bd7340c8526631766afe92e4f9f02117834.tar.gz bcfg2-6c681bd7340c8526631766afe92e4f9f02117834.tar.bz2 bcfg2-6c681bd7340c8526631766afe92e4f9f02117834.zip |
avoid backtrace when bcfg2-yum-helper does not exist
-rw-r--r-- | src/lib/Server/Plugins/Packages/Yum.py | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/src/lib/Server/Plugins/Packages/Yum.py b/src/lib/Server/Plugins/Packages/Yum.py index 12af46508..11500e09d 100644 --- a/src/lib/Server/Plugins/Packages/Yum.py +++ b/src/lib/Server/Plugins/Packages/Yum.py @@ -491,7 +491,14 @@ class YumCollection(Collection): # log to syslog. So would a unicorn. cmd = [self.helper, "-c", self.cfgfile, command] self.logger.debug("Packages: running %s" % " ".join(cmd)) - helper = Popen(cmd, stdin=PIPE, stdout=PIPE, stderr=PIPE) + try: + helper = Popen(cmd, stdin=PIPE, stdout=PIPE, stderr=PIPE) + except OSError: + err = sys.exc_info()[1] + self.logger.error("Packages: Failed to execute %s: %s" % + (" ".join(cmd), err)) + return None + if input: idata = json.dumps(input) (stdout, stderr) = helper.communicate(idata) |