Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-25 Thread ybronhei
Yaniv Bronhaim has submitted this change and it was merged. Change subject: vm: recover must finish with libvirt connection .. vm: recover must finish with libvirt connection bz106336 highlights an scenario on which a Vm can

Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-24 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: vm: recover must finish with libvirt connection .. Patch Set 5: no need for relates-to title. only one Bug-Url targeted to 3.3 (relates can be shown in the

Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-24 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: recover must finish with libvirt connection .. Patch Set 5: Just cloned the 3.4 bz into https://bugzilla.redhat.com/1069095. Will drop the relates-to.

Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-24 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: vm: recover must finish with libvirt connection .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/24822 To unsubscribe, visit

Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-24 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: vm: recover must finish with libvirt connection .. Patch Set 7: Verified+1 -- To view, visit http://gerrit.ovirt.org/24822 To unsubscribe, visit

Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-24 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: vm: recover must finish with libvirt connection .. Patch Set 7: -Verified -- To view, visit http://gerrit.ovirt.org/24822 To unsubscribe, visit

Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-23 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: vm: recover must finish with libvirt connection .. Patch Set 3: Code-Review-1 the bz you attached is targeted to 3.4 -- To view, visit

Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: recover must finish with libvirt connection .. Patch Set 1: Verified-1 Code-Review-1 needs amendment like the patch for branch 3.4 -- To view, visit

Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: recover must finish with libvirt connection .. Patch Set 3: Verified+1 verified the recovering flow the same way of the patch for branch 3.4 and for

Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-20 Thread fromani
Hello Nir Soffer, Federico Simoncelli, Vinzenz Feenstra, Dan Kenigsberg, Michal Skrivanek, I'd like you to do a code review. Please visit http://gerrit.ovirt.org/24822 to review the following change. Change subject: vm: recover must finish with libvirt connection

Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-20 Thread fromani
Francesco Romani has abandoned this change. Change subject: vm: recover must finish with libvirt connection .. Abandoned -- To view, visit http://gerrit.ovirt.org/24822 To unsubscribe, visit http://gerrit.ovirt.org/settings

Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-20 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: vm: recover must finish with libvirt connection .. Patch Set 1: to recap, each api that uses password=*** field, which is easier to understand, will get new

Change in vdsm[ovirt-3.3]: vm: recover must finish with libvirt connection

2014-02-20 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: vm: recover must finish with libvirt connection .. Patch Set 1: sorry, ignore my comment. wrong location :) -- To view, visit http://gerrit.ovirt.org/24822