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,
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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.
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
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
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
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
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
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
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/
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,
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
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
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
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
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
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
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
31 matches
Mail list logo