Change in vdsm[master]: virt: Limit the number of workers in executor

2016-07-25 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 22: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view,

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-07-25 Thread nsoffer
Nir Soffer has submitted this change and it was merged. Change subject: virt: Limit the number of workers in executor .. virt: Limit the number of workers in executor In some situations, periodic operations may hang inside exec

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-07-25 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 21: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure heade

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-07-25 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 20: Code-Review+2 (1 comment) https://gerrit.ovirt.org/#/c/57754/20/tests/executorTests.py File tests/ex

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-07-25 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 20: (3 comments) https://gerrit.ovirt.org/#/c/57754/20/tests/executorTests.py File tests/executorTest

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-07-22 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 20: Code-Review-1 (3 comments) https://gerrit.ovirt.org/#/c/57754/20/tests/executorTests.py File tests/e

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-07-22 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 20: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure heade

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-07-13 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 19: Verified+1 Verified by injecting time.sleep(10) into _RunnableOnVm.__call__, printing the num

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-07-13 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 19: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure heade

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-07-11 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 18: Code-Review+2 (1 comment) https://gerrit.ovirt.org/#/c/57754/18/tests/executorTests.py File te

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-07-11 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 18: (1 comment) https://gerrit.ovirt.org/#/c/57754/18/tests/executorTests.py File tests/executorTests

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-07-07 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 18: Code-Review+1 (1 comment) one minor comment in the tests, overall looks good to me. https://g

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-30 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 18: (1 comment) https://gerrit.ovirt.org/#/c/57754/16/tests/executorTests.py File tests/executorTests

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-30 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 16: (1 comment) https://gerrit.ovirt.org/#/c/57754/16/tests/executorTests.py File tests/executorTests.py

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-30 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 17: (11 comments) https://gerrit.ovirt.org/#/c/57754/16/tests/executorTests.py File tests/executorTes

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 16: (13 comments) https://gerrit.ovirt.org/#/c/57754/16/tests/executorTests.py File tests/executorTests.

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-29 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 16: (4 comments) https://gerrit.ovirt.org/#/c/57754/15/tests/executorTests.py File tests/executorTest

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 16: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure heade

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-28 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 15: (4 comments) Partial review https://gerrit.ovirt.org/#/c/57754/15/tests/executorTests.py File tests

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-27 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 15: The code looks fine, need to review the tests again. -- To view, visit https://gerrit.ovirt.org/577

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-27 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 15: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure heade

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-27 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 14: (1 comment) https://gerrit.ovirt.org/#/c/57754/14/lib/vdsm/executor.py File lib/vdsm/executor.py: L

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-27 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 14: Code-Review+1 (1 comment) https://gerrit.ovirt.org/#/c/57754/14/lib/vdsm/executor.py File lib/

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-20 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 14: Missing comma in a tuple added. -- To view, visit https://gerrit.ovirt.org/57754 To unsubscribe,

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-20 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 14: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure heade

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-17 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 13: (13 comments) https://gerrit.ovirt.org/#/c/57754/12/lib/vdsm/executor.py File lib/vdsm/executor.p

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-06-17 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 13: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure heade

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-05-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 2: (2 comments) Initial review. Mostly OK, but the test could be perhaps improved. Will follow up

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-05-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 2: (3 comments) https://gerrit.ovirt.org/#/c/57754/2/lib/vdsm/executor.py File lib/vdsm/executor.p

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-05-20 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: virt: Limit the number of workers in executor

2016-05-19 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: Limit the number of workers in executor .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header