[NEW PATCH] BZ#728958 Don't check read delay when domain was not produced (via gerrit-bot)

Saggi Mizrahi smizrahi at redhat.com
Tue Aug 30 14:22:47 UTC 2011


New patch submitted by Saggi Mizrahi (smizrahi at redhat.com)

You can review this change at: http://gerrit.usersys.redhat.com/801

commit dc4260c41bada2bed436ac96d5403af775817879
Author: Saggi Mizrahi <smizrahi at redhat.com>
Date:   Mon Aug 8 16:54:11 2011 +0300

    BZ#728958 Don't check read delay when domain was not produced
    
    Change-Id: Idb9a0d62299136cd8011f0e550373900b04ec3af

diff --git a/vdsm/storage/sp.py b/vdsm/storage/sp.py
index 4df9efc..bdb3082 100644
--- a/vdsm/storage/sp.py
+++ b/vdsm/storage/sp.py
@@ -107,6 +107,11 @@ class StatsThread(threading.Thread):
                 # This is handled seperatly because in case of this kind
                 # of failure we don't want to print stack trace
                 delay = self._domain.getReadDelay()
+            except AttributeError:
+                # If we couldn't retrieve the domain object there is something
+                # wrong with the domain. No need to log because is was logged
+                # when the produce() that failed
+                code = 200
             except Exception, e:
                 self.log.error("Could not figure out delay for domain `%s` (%s)", self._sdUUID, e)
                 code = 200




More information about the vdsm-patches mailing list