Martin Sivák has posted comments on this change.

Change subject: Make sure the balloon info uses numbers when provided by VDSM
......................................................................


Patch Set 2: (1 inline comment)

Maybe the commit message should be updated to reflect this, but it needed to be 
there.

....................................................
File mom/Monitor.py
Line 83
Line 84
Line 85
Line 86
Line 87
Actually yes, VDSM patch also adds balloon_target field to the API. That field 
is not necessary for MoM and so it is missing from fields. So the equivalence 
was not true here, but the error message was "Incomplete data: missing set([])"


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I74b227a5d945397a69702fc0758d864b33014fc6
Gerrit-PatchSet: 2
Gerrit-Project: mom
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Martin Sivák <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to