Change in vdsm[master]: vm: events: do not send hashes in status messages

fromani at redhat.com fromani at redhat.com
Thu Jul 9 13:35:54 UTC 2015


Francesco Romani has uploaded a new change for review.

Change subject: vm: events: do not send hashes in status messages
......................................................................

vm: events: do not send hashes in status messages

Engine doesn't really benefit of hashes in each VM_status
message. We should send it only whenever matters,
like when VM goes PoweringUp/Up.

Future patches may want to reintroduce it in other flows.

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


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/86/43386/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index adcf96d..1c3b79a 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -396,8 +396,6 @@
     def _make_event(self, vm_status, **kwargs):
         event_data = {
             'status': vm_status,
-            'hash': str(hash((self._domain.devices_hash,
-                             self.guestAgent.diskMappingHash))),
         }
         # TODO: DOWN and exitCode must be set atomically. Once this is
         # done we can remove the multiple conditions from this code.


-- 
To view, visit https://gerrit.ovirt.org/43386
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib0f4bc76947df86a978820e6eb8d5758532e14b3
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