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
*
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
*
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
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
*
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
*
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
*
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
*
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
*
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
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
*
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
*
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
*
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
*
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
*
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
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
*
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
*
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
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
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
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
*
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
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
*
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
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
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
*
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
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
*
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
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
*
30 matches
Mail list logo