Change in vdsm[master]: call stop_event_loop upon exit

2015-08-04 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: call stop_event_loop upon exit .. call stop_event_loop upon exit For cleanliness, whomever starts a thread should stop it when it is no longer needed.

Change in vdsm[master]: call stop_event_loop upon exit

2015-08-04 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 7: * Update tracker::IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit

Change in vdsm[master]: call stop_event_loop upon exit

2015-08-04 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 6: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/26532 To unsubscribe, visit

Change in vdsm[master]: call stop_event_loop upon exit

2015-08-04 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 6: Continuous-Integration+1 ExecError: Command ['/usr/sbin/brctl', 'delbr', 'vdsm-UVI4U1q4Xm'] returned non-zero

Change in vdsm[master]: call stop_event_loop upon exit

2015-07-31 Thread fromani
Francesco Romani has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 5: Code-Review+1 will verify ASAP -- To view, visit https://gerrit.ovirt.org/26532 To unsubscribe, visit

Change in vdsm[master]: call stop_event_loop upon exit

2015-07-31 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 6: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: call stop_event_loop upon exit

2015-07-31 Thread fromani
Francesco Romani has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 5: Verified+1 (1 comment) VDSM still stops, the newly added function call seem to have negligible (near zero)

Change in vdsm[master]: call stop_event_loop upon exit

2015-07-31 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 5: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: call stop_event_loop upon exit

2015-07-30 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: call stop_event_loop upon exit .. 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]: call stop_event_loop upon exit

2015-07-28 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: call stop_event_loop upon exit .. 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]: call stop_event_loop upon exit

2015-07-28 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/26532 To unsubscribe, visit

Change in vdsm[master]: call stop_event_loop upon exit

2014-06-23 Thread iheim
Itamar Heim has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 2: ping -- To view, visit http://gerrit.ovirt.org/26532 To unsubscribe, visit http://gerrit.ovirt.org/settings

Change in vdsm[master]: call stop_event_loop upon exit

2014-04-23 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/26532 To unsubscribe, visit

Change in vdsm[master]: call stop_event_loop upon exit

2014-04-23 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/26532/2/vdsm/vdsm File vdsm/vdsm: Line 83: finally: Line 84:

Change in vdsm[master]: call stop_event_loop upon exit

2014-04-16 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 2: Build Failed http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/8107/ : SUCCESS

Change in vdsm[master]: call stop_event_loop upon exit

2014-04-14 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 1: Code-Review-1 (1 comment) http://gerrit.ovirt.org/#/c/26532/1/vdsm/vdsm File vdsm/vdsm: Line 80: while

Change in vdsm[master]: call stop_event_loop upon exit

2014-04-14 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 1: post new version. +1 for the order -- To view, visit http://gerrit.ovirt.org/26532 To unsubscribe, visit

Change in vdsm[master]: call stop_event_loop upon exit

2014-04-08 Thread danken
Dan Kenigsberg has uploaded a new change for review. Change subject: call stop_event_loop upon exit .. call stop_event_loop upon exit For cleanliness, whomever starts a thread should stop it when it is no longer needed.

Change in vdsm[master]: call stop_event_loop upon exit

2014-04-08 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/7896/ : SUCCESS

Change in vdsm[master]: call stop_event_loop upon exit

2014-04-08 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: call stop_event_loop upon exit .. Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/26532/1/vdsm/vdsm File vdsm/vdsm: Line 80: while running[0]: