Change in vdsm[master]: do not rely on default values on restoreFlags invocation

2013-07-25 Thread ahadas
Arik Hadas has uploaded a new change for review. Change subject: do not rely on default values on restoreFlags invocation .. do not rely on default values on restoreFlags invocation The signature of libvirt's verb

Change in vdsm[master]: do not rely on default values on restoreFlags invocation

2013-07-25 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: do not rely on default values on restoreFlags invocation .. Patch Set 1: Build Successful

Change in vdsm[master]: do not rely on default values on restoreFlags invocation

2013-07-25 Thread ewarszaw
Eduardo has posted comments on this change. Change subject: do not rely on default values on restoreFlags invocation .. Patch Set 1: Is a minimal libvirt version required for this feature? -- To view, visit

Change in vdsm[master]: do not rely on default values on restoreFlags invocation

2013-07-25 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: do not rely on default values on restoreFlags invocation .. Patch Set 1: yes but the current dependencies configuration is ok since we require libvirt =

Change in vdsm[master]: do not rely on default values on restoreFlags invocation

2013-07-25 Thread michal . skrivanek
Michal Skrivanek has posted comments on this change. Change subject: do not rely on default values on restoreFlags invocation .. Patch Set 1: why don't we just bump up the requirement on RHEL? -- To view, visit

Change in vdsm[master]: do not rely on default values on restoreFlags invocation

2013-07-25 Thread michal . skrivanek
Michal Skrivanek has posted comments on this change. Change subject: do not rely on default values on restoreFlags invocation .. Patch Set 1: libvirt version i meant... -- To view, visit http://gerrit.ovirt.org/17298 To

Change in vdsm[master]: do not rely on default values on restoreFlags invocation

2013-07-25 Thread ykaplan
Yeela Kaplan has posted comments on this change. Change subject: do not rely on default values on restoreFlags invocation .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[master]: do not rely on default values on restoreFlags invocation

2013-07-25 Thread michal . skrivanek
Michal Skrivanek has posted comments on this change. Change subject: do not rely on default values on restoreFlags invocation .. Patch Set 1: I'd prefer bump up libvirt(or just rely on it's there) since pre-alpha 6.5 and

Change in vdsm[master]: do not rely on default values on restoreFlags invocation

2013-07-25 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: do not rely on default values on restoreFlags invocation .. Patch Set 1: Looks good to me, approved Since passing the argument explicitly has not been

Change in vdsm[master]: do not rely on default values on restoreFlags invocation

2013-07-25 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: do not rely on default values on restoreFlags invocation .. do not rely on default values on restoreFlags invocation The signature of libvirt's verb