Change in vdsm[master]: Remove sparsify from vdsm

2016-08-20 Thread Jenkins CI RO
Jenkins CI RO has abandoned this change. Change subject: Remove sparsify from vdsm .. Abandoned Abandoned due to no activity - please restore if still relevant -- To view, visit https://gerrit.ovirt.org/54426 To unsubscribe,

Change in vdsm[master]: Remove sparsify from vdsm

2016-08-20 Thread automation
gerrit-hooks has posted comments on this change. Change subject: Remove sparsify from vdsm .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found -- To view, visit https://gerrit.ovirt.org/54426 To unsubscribe, visit http

Change in vdsm[master]: Remove sparsify from vdsm

2016-03-08 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: Remove sparsify from vdsm .. Patch Set 1: Code-Review-1 Looking at the replacement patch, this is not the right way to add the new sparsify. You should change the

Change in vdsm[master]: Remove sparsify from vdsm

2016-03-08 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Remove sparsify from vdsm .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/54426 To unsubscribe, visit https://gerrit.ovirt.org/setting

Change in vdsm[master]: Remove sparsify from vdsm

2016-03-08 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: Remove sparsify from vdsm .. Patch Set 1: Code-Review+1 Adding Allon to confirm this -- To view, visit https://gerrit.ovirt.org/54426 To unsubscribe, visit https:

Change in vdsm[master]: Remove sparsify from vdsm

2016-03-08 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Remove sparsify from vdsm .. Patch Set 1: Code-Review+1 it seems indeed this verb is not used. Awesome! Partial ACK until we have final proof. -- To view,

Change in vdsm[master]: Remove sparsify from vdsm

2016-03-07 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: Remove sparsify from vdsm .. Patch Set 1: > (1 comment) > > Can we just drop a method from schema? Seems too easy to be true... Engine doesn't use it, It was

Change in vdsm[master]: Remove sparsify from vdsm

2016-03-07 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Remove sparsify from vdsm .. Patch Set 1: Code-Review-1 (1 comment) Can we just drop a method from schema? Seems too easy to be true... https://gerrit.ovirt

Change in vdsm[master]: Remove sparsify from vdsm

2016-03-07 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Remove sparsify from vdsm .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/54426/1//COMMIT_MSG Commit Message: Line 6: Line 7: Remove sparsify from

Change in vdsm[master]: Remove sparsify from vdsm

2016-03-07 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: Remove sparsify from vdsm .. Patch Set 1: Verified+1 -- To view, visit https://gerrit.ovirt.org/54426 To unsubscribe, visit https://gerrit.ovirt.org/settings G

Change in vdsm[master]: Remove sparsify from vdsm

2016-03-07 Thread shavivi
Shahar Havivi has uploaded a new change for review. Change subject: Remove sparsify from vdsm .. Remove sparsify from vdsm We are changing the sparsify image to in place sparsify image, e.g. no need for a new image in order to s

Change in vdsm[master]: Remove sparsify from vdsm

2016-03-07 Thread automation
gerrit-hooks has posted comments on this change. Change subject: Remove sparsify from vdsm .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '