Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-19 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 32: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-18 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 31: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-18 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 30: Verified+1 -- To view, visit https://gerrit.ovirt.org/54849 To unsubscribe, visit

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-18 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 30: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-17 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 29: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-17 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 28: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-16 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 27: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-16 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 26: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-16 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 25: Verified+1 -- To view, visit https://gerrit.ovirt.org/54849 To unsubscribe, visit

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-16 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 25: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-16 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 24: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-16 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 23: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-15 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 22: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-15 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 21: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-14 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 20: Verified+1 -- To view, visit https://gerrit.ovirt.org/54849 To unsubscribe, visit

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-05-13 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 20: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-04-28 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 19: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-04-27 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 18: (3 comments) https://gerrit.ovirt.org/#/c/54849/18/lib/vdsm/storage/asyncevent.py File

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-04-27 Thread fromani
Francesco Romani has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 18: Code-Review+1 (3 comments) very nice patch and sensible approach, keeping one eye on python3

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-04-25 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 18: Verified+1 -- To view, visit https://gerrit.ovirt.org/54849 To unsubscribe, visit

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-04-25 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 18: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-04-08 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 17: Verified+1 -- To view, visit https://gerrit.ovirt.org/54849 To unsubscribe, visit

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-04-08 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 17: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-04-04 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 16: Francesco, I did not compare this to vmchannels or any other implementation. Without scheduled calls

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-04-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 16: Nir, ok, understood what betterAsyncore and vmchannels does not provide. Vmchannels has poor

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-03-30 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 16: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-03-25 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 14: Verified+1 Verified using the tests. -- To view, visit https://gerrit.ovirt.org/54849 To

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-03-25 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 14: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-03-25 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 13: This version rename the module using the same style as asyncore and asynchat. The code is mostly take

Change in vdsm[master]: monitoring: Introduce the asyncevent module

2016-03-25 Thread automation
gerrit-hooks has posted comments on this change. Change subject: monitoring: Introduce the asyncevent module .. Patch Set 13: * #1081962::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1081962::OK, public bug *