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