Change in vdsm[master]: vm: use proper threading.Event()s

2016-04-05 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 7: * #912390::Update tracker: OK * Set MODIFIED::bug 912390#912390IGNORE, not all related patches are

Change in vdsm[master]: vm: use proper threading.Event()s

2016-04-05 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 6: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/54792 To unsubscribe, visit

Change in vdsm[master]: vm: use proper threading.Event()s

2016-04-05 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: vm: use proper threading.Event()s .. vm: use proper threading.Event()s We use plain booleans in Vm class to synchronize with the asynchronous creation

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-24 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 6: * #912390::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#912390::OK, public bug * Check

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-24 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 5: * #912390::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#912390::OK, public bug * Check

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-23 Thread Jenkins CI
Jenkins CI has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 4: Continuous-Integration+1 Propagate review hook: Continuous Integration value inherited from patch 3 -- To view,

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-23 Thread mpolednik
Martin Polednik has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 4: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/54792 To unsubscribe, visit

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-23 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 4: * #912390::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#912390::OK, public bug * Check

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-23 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 3: Code-Review+1 (Assuming you fix the commit message.) -- To view, visit https://gerrit.ovirt.org/54792 To

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-23 Thread mpolednik
Martin Polednik has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 3: Code-Review-1 You haven't really updated the commit message you -1'd yourself? -- To view, visit

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-23 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 3: Verified+1 verified with 44989 -- To view, visit https://gerrit.ovirt.org/54792 To unsubscribe, visit

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-23 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 3: * #912390::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#912390::OK, public bug * Check

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-23 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 2: (2 comments) https://gerrit.ovirt.org/#/c/54792/2//COMMIT_MSG Commit Message: PS2, Line 9: palin plain

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-23 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 2: jenkins failure is unrelated: 0:08:34.061 raise JsonRpcNoResponseError(methodName) 00:08:34.061

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-23 Thread mpolednik
Martin Polednik has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 2: Code-Review+1 I don't trust jenkins. -- To view, visit https://gerrit.ovirt.org/54792 To unsubscribe,

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-23 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 2: * #912390::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#912390::OK, public bug * Check

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-16 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vm: use proper threading.Event()s .. Patch Set 1: * #912390::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#912390::OK, public bug * Check

Change in vdsm[master]: vm: use proper threading.Event()s

2016-03-16 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: vm: use proper threading.Event()s .. vm: use proper threading.Event()s We use palin booleans in Vm class to synchronize with asynchronous creation thread.