Change in vdsm[master]: vm: log exceptional monitor unresponsive case

fromani at redhat.com fromani at redhat.com
Mon Oct 6 19:14:06 UTC 2014


Francesco Romani has uploaded a new change for review.

Change subject: vm: log exceptional monitor unresponsive case
......................................................................

vm: log exceptional monitor unresponsive case

In past ages we added an explicit case to mark
monitor unresponsive.
This case may be no long relevant. In order
to gather evidence against or supporting
this hypotesis, add an explicit log line
when this case is applied.

Change-Id: I04b88513d319d3f4530aa6d9f1b4a1c783b33347
Signed-off-by: Francesco Romani <fromani at redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/70/33870/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 1eebb69..414646c 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -2480,6 +2480,10 @@
                 if (not self.isMigrating()
                     and decStats['statsAge'] >
                         config.getint('vars', 'vm_command_timeout')):
+                    if stats['monitorResponse'] != '-1':
+                        self.log.warning('monitor become unresponsive'
+                                         ' (command timeout, age=%s)',
+                                         decStats['statsAge'])
                     stats['monitorResponse'] = '-1'
         except Exception:
             self.log.error("Error fetching vm stats", exc_info=True)


-- 
To view, visit http://gerrit.ovirt.org/33870
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I04b88513d319d3f4530aa6d9f1b4a1c783b33347
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>


More information about the vdsm-patches mailing list