Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-03-15 Thread automation
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-03-15 Thread automation
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-03-15 Thread automation
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.

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-28 Thread oVirt Jenkins CI Server
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-28 Thread danken
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-28 Thread danken
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://

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-27 Thread alitke
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-23 Thread fromani
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-21 Thread danken
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://

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-21 Thread oVirt Jenkins CI Server
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-21 Thread nsoffer
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-21 Thread Federico Simoncelli
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-21 Thread oVirt Jenkins CI Server
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-21 Thread nsoffer
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-21 Thread danken
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-16 Thread nsoffer
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:

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2015-01-16 Thread Federico Simoncelli
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:

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-12-20 Thread dkuznets
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://

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-12-17 Thread nsoffer
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:

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-12-17 Thread nsoffer
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-12-17 Thread oVirt Jenkins CI Server
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-12-17 Thread Federico Simoncelli
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-11-23 Thread nsoffer
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/

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-11-16 Thread fromani
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-11-13 Thread oVirt Jenkins CI Server
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-11-11 Thread dkuznets
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-11-11 Thread Federico Simoncelli
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-11-10 Thread dkuznets
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-11-04 Thread smizrahi
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-10-24 Thread fromani
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-10-07 Thread oVirt Jenkins CI Server
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/ :

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-10-01 Thread nsoffer
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:

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-10-01 Thread Federico Simoncelli
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:

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-10-01 Thread oVirt Jenkins CI Server
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

Change in vdsm[master]: utils: move nice ionice sid and sudo out of execCmd

2014-10-01 Thread Federico Simoncelli
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