Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-05 Thread fromani
Francesco Romani has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 4: Code-Review+1 (2 comments) overall looks good, I'm not 100% sold but not enough to stop this

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-05 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 5: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view,

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-05 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 4: (1 comment) https://gerrit.ovirt.org/#/c/51217/4/tests/jobsTests.py File tests/jobsTests.py: Line

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-05 Thread nsoffer
Nir Soffer has submitted this change and it was merged. Change subject: jobs: Add Job.__repr__ for easier debugging .. jobs: Add Job.__repr__ for easier debugging Every object should have __repr__ to make it easy to have more

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-04 Thread alitke
Adam Litke has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 3: Code-Review+1 Not +2 because I am unsure if we need the ProgressingJob class. -- To view, visit

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-04 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-04 Thread alitke
Adam Litke has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 4: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/51217 To unsubscribe, visit

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-04 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 4: Verified+1 -- To view, visit https://gerrit.ovirt.org/51217 To unsubscribe, visit

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-03 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 3: Verified+1 -- To view, visit https://gerrit.ovirt.org/51217 To unsubscribe, visit

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-03 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/51217 To unsubscribe, visit

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-03 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 1: Verified+1 -- To view, visit https://gerrit.ovirt.org/51217 To unsubscribe, visit

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-02 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-02 Thread nsoffer
Nir Soffer has uploaded a new change for review. Change subject: jobs: Add Job.__repr__ for easier debugging .. jobs: Add Job.__repr__ for easier debugging Every object should have __repr__ to make it easy to have more detailed

Change in vdsm[master]: jobs: Add Job.__repr__ for easier debugging

2016-01-02 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Add Job.__repr__ for easier debugging .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header