Mark Wu has uploaded a new change for review.
Change subject: Add a VDSM verb to change MOM policy
..
Add a VDSM verb to change MOM policy
Change-Id: I9f4293b3c6669db91b9c8d6212b01a06ee40ab02
Signed-off-by: Mark Wu
---
M vdsm/A
Dan Kenigsberg has posted comments on this change.
Change subject: Add a VDSM verb to change MOM policy
..
Patch Set 1: I would prefer that you didn't submit this
(1 inline comment)
Mark Wu has posted comments on this change.
Change subject: Add a VDSM verb to change MOM policy
..
Patch Set 1: (1 inline comment)
File vdsm/vdsmMOM.py
Line 53: # No
Doron Fediuck has posted comments on this change.
Change subject: Add a VDSM verb to change MOM policy
..
Patch Set 2: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/5069
To unsubscrib
Dan Kenigsberg has posted comments on this change.
Change subject: Add a VDSM verb to change MOM policy
..
Patch Set 2: I would prefer that you didn't submit this
(1 inline comment)
just a little question
Doron Fediuck has posted comments on this change.
Change subject: Add a VDSM verb to change MOM policy
..
Patch Set 5: Looks good to me, but someone else must approve
Unless the given policy is escaped, this may cause trouble i
Doron Fediuck has posted comments on this change.
Change subject: Add a VDSM verb to change MOM policy
..
Patch Set 6:
Mark,
no need to escape the policy string?
--
To view, visit http://gerrit.ovirt.org/5069
To unsubscribe, v
Mark Wu has posted comments on this change.
Change subject: Add a VDSM verb to change MOM policy
..
Patch Set 6:
Doron,
Sorry, I didn't understand what you're talking about. Any characters required
to escape in XMLRPC? Coul
Doron Fediuck has posted comments on this change.
Change subject: Add a VDSM verb to change MOM policy
..
Patch Set 6:
I'm referring to setMOMPolicy(self, policy)
Is this an escaped xml struct?
--
To view, visit http://gerrit
Dan Kenigsberg has posted comments on this change.
Change subject: Add a VDSM verb to change MOM policy
..
Patch Set 6: Looks good to me, but someone else must approve
Doron, I do not understand your issue, too. xmlrpclib does
Doron Fediuck has posted comments on this change.
Change subject: Add a VDSM verb to change MOM policy
..
Patch Set 6: Looks good to me, but someone else must approve
I was trying to understand if the policy argument is an esca
Dan Kenigsberg has posted comments on this change.
Change subject: Add a VDSM verb to change MOM policy
..
Patch Set 6: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/5069
To unsubscribe, visit http://gerr
Mark Wu has posted comments on this change.
Change subject: Add a VDSM verb to change MOM policy
..
Patch Set 6: Verified
--
To view, visit http://gerrit.ovirt.org/5069
To unsubscribe, visit http://gerrit.ovirt.org/settings
Ge
Dan Kenigsberg has submitted this change and it was merged.
Change subject: Add a VDSM verb to change MOM policy
..
Add a VDSM verb to change MOM policy
Change-Id: I9f4293b3c6669db91b9c8d6212b01a06ee40ab02
Signed-off-by: Mark W
14 matches
Mail list logo