Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-11 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 11: (1 comment) https://gerrit.ovirt.org/#/c/51421/11/lib/vdsm/host.py Fil

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-11 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 11: Code-Review-1 (1 comment) https://gerrit.ovirt.org/#/c/51421/11/lib/

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-10 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 11: Dan: hopefully the failing tests are fixed by https://gerrit.ovirt.org/5161

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-10 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 11: Continuous-Integration+1 Would you please mark these test as broken-on-

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-10 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 11: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/51421 To unsu

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-10 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 11: Verified+1 ran nosetests and edy ran functional tests -- To view, vis

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-10 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 11: Code-Review+1 Waiting for ack from other maintainers. -- To view, visit h

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-10 Thread automation
gerrit-hooks has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 11: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-10 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 10: Code-Review-1 no biggies, but please fix $ git grep utils.execCmd|cat

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-09 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 10: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/51421 To unsub

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-09 Thread edwardh
Edward Haas has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 10: Verified+1 Net Functional tests are passing -- To view, visit https://ge

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-09 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 10: Verified+1 -- To view, visit https://gerrit.ovirt.org/51421 To unsubsc

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-08 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 10: Code-Review+1 Looks good, needs approval from all subsystem maintainers. -

Change in vdsm[master]: Introduce commands module to separate execCmd and AsyncProc ...

2016-01-08 Thread automation
gerrit-hooks has posted comments on this change. Change subject: Introduce commands module to separate execCmd and AsyncProc from utils .. Patch Set 10: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug