summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorSol Jerome <sol.jerome@gmail.com>2013-01-30 13:04:29 -0600
committerSol Jerome <sol.jerome@gmail.com>2013-01-30 13:04:29 -0600
commit7ca6230df2dc22e43228c3d8e8cfe9851b0e0f3d (patch)
treeaaeef20fa1640c71934f4432b07991581dfcc739 /src
parent51e6930af7319a71571237fa2642cee76c736057 (diff)
downloadbcfg2-7ca6230df2dc22e43228c3d8e8cfe9851b0e0f3d.tar.gz
bcfg2-7ca6230df2dc22e43228c3d8e8cfe9851b0e0f3d.tar.bz2
bcfg2-7ca6230df2dc22e43228c3d8e8cfe9851b0e0f3d.zip
YUM: Set logger to info for pkg verify failures
If the loglevel is set to debug, then a user running the client without -d will not be informed why the client is asking them to install Package entries which may already be installed (but are not verifying). Signed-off-by: Sol Jerome <sol.jerome@gmail.com>
Diffstat (limited to 'src')
-rw-r--r--src/lib/Bcfg2/Client/Tools/YUM.py15
1 files changed, 8 insertions, 7 deletions
diff --git a/src/lib/Bcfg2/Client/Tools/YUM.py b/src/lib/Bcfg2/Client/Tools/YUM.py
index c8414b4b2..1fe275c2c 100644
--- a/src/lib/Bcfg2/Client/Tools/YUM.py
+++ b/src/lib/Bcfg2/Client/Tools/YUM.py
@@ -603,17 +603,18 @@ class YUM(Bcfg2.Client.Tools.PkgTool):
if stat['verify'] != {}:
stat['verify_fail'] = True
package_fail = True
- self.logger.debug("It is suggested that you either manage "
- "these files, revert the changes, or ignore "
- "false failures:")
- self.logger.debug(" Verify Problems:")
+ self.logger.info("It is suggested that you either manage "
+ "these files, revert the changes, or ignore "
+ "false failures:")
+ self.logger.info(" Verify Problems: %s" %
+ stat['pkg'].get('name'))
for fname, probs in list(stat['verify'].items()):
if len(probs) > 1:
- self.logger.debug(" %s" % fname)
+ self.logger.info(" %s" % fname)
for prob in probs:
- self.logger.debug(" %s" % prob)
+ self.logger.info(" %s" % prob[1])
else:
- self.logger.debug(" %s: %s" % (fname, probs[0]))
+ self.logger.info(" %s: %s" % (fname, probs[0]))
if len(all_pkg_objs) > 0:
# Is this an install only package? We just look at the first one