Dan Kenigsberg has submitted this change and it was merged.
Change subject: utils: add CommandStream class
..
utils: add CommandStream class
This patch adds CommandStream a generic class to process the output of
external
automat...@ovirt.org has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 11:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
automat...@ovirt.org has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Adam Litke has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 9: Code-Review+2
Looks good.
--
To view, visit https://gerrit.ovirt.org/33909
To unsubscribe, visit
Adam Litke has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 10: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/33909
To unsubscribe, visit
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 10: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/33909
To unsubscribe, visit
Federico Simoncelli has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 10: Verified+1
--
To view, visit https://gerrit.ovirt.org/33909
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 9:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 9: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/33909
To unsubscribe, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 8: Code-Review-1 Verified-1
Build Unstable
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18440/ :
automat...@ovirt.org has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 8:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 8:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18440/
--
To view, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 8:
Build Started (2/2) -
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1671/
--
To view,
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 8:
(2 comments)
https://gerrit.ovirt.org/#/c/33909/8/tests/utilsTests.py
File tests/utilsTests.py:
Line 777:
Line
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 8: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/33909
To unsubscribe, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 7:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18265/
--
To view, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 7:
Build Started (2/2)
0 - http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1495/
--
To
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 7:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18265/ : SUCCESS
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 7:
(2 comments)
https://gerrit.ovirt.org/#/c/33909/7/tests/utilsTests.py
File tests/utilsTests.py:
Line 770:
Line
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 7:
(4 comments)
https://gerrit.ovirt.org/#/c/33909/7/tests/utilsTests.py
File tests/utilsTests.py:
Line 754:
automat...@ovirt.org has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 6:
Build Started (2/2) -
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17075/
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 6:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16903/ : SUCCESS
automat...@ovirt.org has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 6:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16903/
--
To view, visit
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/33909/4/lib/vdsm/utils.py
File lib/vdsm/utils.py:
Line 362: for
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 5:
Would you like to check my comments on the tests in version 3?
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 5:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16807/ : SUCCESS
automat...@ovirt.org has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Federico Simoncelli has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/33909/4/lib/vdsm/utils.py
File lib/vdsm/utils.py:
Line 362:
automat...@ovirt.org has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 4:
(2 comments)
http://gerrit.ovirt.org/#/c/33909/4/lib/vdsm/utils.py
File lib/vdsm/utils.py:
Line 356: #
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 3:
(2 comments)
http://gerrit.ovirt.org/#/c/33909/3/lib/vdsm/utils.py
File lib/vdsm/utils.py:
Line 341:
Line 342:
Federico Simoncelli has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 3:
(5 comments)
http://gerrit.ovirt.org/#/c/33909/3/lib/vdsm/utils.py
File lib/vdsm/utils.py:
Line 335:
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 4:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/15357/ : SUCCESS
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 3:
(18 comments)
http://gerrit.ovirt.org/#/c/33909/3//COMMIT_MSG
Commit Message:
Line 4: Commit: Federico
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 3:
(7 comments)
http://gerrit.ovirt.org/#/c/33909/3/lib/vdsm/utils.py
File lib/vdsm/utils.py:
Line 331:
Line 332:
Federico Simoncelli has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 2:
(2 comments)
http://gerrit.ovirt.org/#/c/33909/2/lib/vdsm/utils.py
File lib/vdsm/utils.py:
Line 343:
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/13568/ : FAILURE
Nir Soffer has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 2:
(12 comments)
I like the direction.
http://gerrit.ovirt.org/#/c/33909/2/lib/vdsm/utils.py
File lib/vdsm/utils.py:
Francesco Romani has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 2:
(9 comments)
preliminary review - easy parts first
http://gerrit.ovirt.org/#/c/33909/2/lib/vdsm/utils.py
Francesco Romani has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 2:
(1 comment)
http://gerrit.ovirt.org/#/c/33909/2//COMMIT_MSG
Commit Message:
Line 4: Commit: Federico
Federico Simoncelli has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 2:
(1 comment)
http://gerrit.ovirt.org/#/c/33909/2//COMMIT_MSG
Commit Message:
Line 4: Commit: Federico
Federico Simoncelli has uploaded a new change for review.
Change subject: utils: add CommandStream class
..
utils: add CommandStream class
Change-Id: Ie015368bb9c5992e5c73a149277c59fc4ffbd570
Signed-off-by: Federico Simoncelli
Federico Simoncelli has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 1:
(1 comment)
http://gerrit.ovirt.org/#/c/33909/1/tests/utilsTests.py
File tests/utilsTests.py:
Line 736:
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: add CommandStream class
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/12827/ : SUCCESS
46 matches
Mail list logo