automat...@ovirt.org has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, ma
automat...@ovirt.org has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, ma
automat...@ovirt.org has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 7:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_verify-error-codes_merged/6
Dan Kenigsberg has submitted this change and it was merged.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
utils: move nice ionice sid and sudo out of execCmd
Utility commands as nice, ionice, setsid and
Dan Kenigsberg has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 6: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/33691
To unsubscribe, visit http://
Adam Litke has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 6: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/33691
To unsubscribe, visit http://gerr
Francesco Romani has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 6: Code-Review+1
thought about naming (shellcmds? shell?) but can't find anything valuable to
Dan Kenigsberg has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 6: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/33691
To unsubscribe, visit http://
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 6:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21_cre
Nir Soffer has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 6: Code-Review+1
Lets have another *utils.py
--
To view, visit http://gerrit.ovirt.org/33691
To u
Federico Simoncelli has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 6: Verified+1
In use since few months and verified with unit tests.
--
To view, visit ht
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 5:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21_cre
Nir Soffer has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 5: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/33691
To unsubscribe, visit http://gerr
Dan Kenigsberg has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 4:
(1 comment)
http://gerrit.ovirt.org/#/c/33691/4/lib/vdsm/Makefile.am
File lib/vdsm/Makefile
Nir Soffer has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 4:
(2 comments)
http://gerrit.ovirt.org/#/c/33691/4//COMMIT_MSG
Commit Message:
Line 6:
Line 7:
Federico Simoncelli has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 4:
(2 comments)
http://gerrit.ovirt.org/#/c/33691/4//COMMIT_MSG
Commit Message:
Line 6:
Dima Kuznetsov has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 4: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/33691
To unsubscribe, visit http://
Nir Soffer has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 4:
(3 comments)
http://gerrit.ovirt.org/#/c/33691/4//COMMIT_MSG
Commit Message:
Line 6:
Line 7:
Nir Soffer has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 4:
(5 comments)
http://gerrit.ovirt.org/#/c/33691/4/lib/vdsm/cmdutils.py
File lib/vdsm/cmdutils.py
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 4:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-fc21_cre
Federico Simoncelli has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 4:
(1 comment)
http://gerrit.ovirt.org/#/c/33691/4/lib/vdsm/Makefile.am
File lib/vdsm/Mak
Nir Soffer has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 3:
(1 comment)
Looks good, except the names (e.g. sudo_cmd).
http://gerrit.ovirt.org/#/c/33691/3/
Francesco Romani has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 3:
(1 comment)
http://gerrit.ovirt.org/#/c/33691/3//COMMIT_MSG
Commit Message:
Line 3: Auth
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/13390/ : SUCCES
Dima Kuznetsov has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 2:
You go forward with this, I don't mind using this as starting point.
--
To view, visit htt
Federico Simoncelli has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 2:
No need to downvote on the promise of future patches. We can discuss the names
and the
Dima Kuznetsov has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 2:
No review material yet but I talked with Saggi about doing the same, I wanted
to go with ju
Saggi Mizrahi has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 2: Code-Review-1
We are already doing work on moving those out of execCmd. dkuznets should have
Francesco Romani has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 2:
I agree with Nir's comment in revision 1
--
To view, visit http://gerrit.ovirt.org/33691
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/12825/ :
Nir Soffer has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 1:
(1 comment)
http://gerrit.ovirt.org/#/c/33691/1//COMMIT_MSG
Commit Message:
Line 4: Commit:
Federico Simoncelli has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 1:
(2 comments)
http://gerrit.ovirt.org/#/c/33691/1//COMMIT_MSG
Commit Message:
Line 4:
oVirt Jenkins CI Server has posted comments on this change.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
Patch Set 1:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11
Federico Simoncelli has uploaded a new change for review.
Change subject: utils: move nice ionice sid and sudo out of execCmd
..
utils: move nice ionice sid and sudo out of execCmd
Change-Id: I442c64e8b9cfb3933dbe9879b1d442d0e96
35 matches
Mail list logo