Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 29:
(2 comments)
https://gerrit.ovirt.org/#/c/36368/29/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 29:
(2 comments)
https://gerrit.ovirt.org/#/c/36368/29/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 30: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/36368
To unsubscribe, visit
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 30: Verified+1
Verified with latest engine containing event changes.
--
To view, visit
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 30:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 31:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
Dan Kenigsberg has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 30: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/36368
To unsubscribe, visit
Dan Kenigsberg has submitted this change and it was merged.
Change subject: stomp: client side subscription
..
stomp: client side subscription
In this patch we introduce concept of subscription for client
perspective. We move
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 29:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 29: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/36368
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 28:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 26: Verified-1
This patch set is broken due to wrong squash with Outbox class.
--
To view, visit
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 27:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 27: Verified+1
Verified by running with latest engine containing event changes.
--
To view, visit
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 26:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 25:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 24:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 24: Verified+1
Verified by running vdsm with the latest engine change. Tested vm status
changes:
- starting a
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 22:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 23: Verified+1
Verified by vdsm updated and see that there are no issues with communication.
--
To view, visit
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 23:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 21:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 21: Verified+1
Scope reduced, rebased and verified by installing a host and running a vm to
see sent events.
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 20:
(2 comments)
https://gerrit.ovirt.org/#/c/36368/20/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Dan Kenigsberg has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 20:
(1 comment)
https://gerrit.ovirt.org/#/c/36368/20//COMMIT_MSG
Commit Message:
Line 9: In this patch we
Dan Kenigsberg has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 20:
(9 comments)
https://gerrit.ovirt.org/#/c/36368/20/lib/yajsonrpc/__init__.py
File lib/yajsonrpc/__init__.py:
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 20:
(9 comments)
https://gerrit.ovirt.org/#/c/36368/20//COMMIT_MSG
Commit Message:
Line 9: In this patch we
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 20:
(2 comments)
https://gerrit.ovirt.org/#/c/36368/20/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19:
(2 comments)
https://gerrit.ovirt.org/#/c/36368/19/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/36368/19/lib/yajsonrpc/stompReactor.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19:
(1 comment)
https://gerrit.ovirt.org/#/c/36368/19/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19:
(1 comment)
https://gerrit.ovirt.org/#/c/36368/19/lib/yajsonrpc/stompReactor.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19:
(1 comment)
https://gerrit.ovirt.org/#/c/36368/19/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 20:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 20: Verified+1
Fixed comments, rebased and verified by installing a host and configuring nfs
storage domain.
Yeela Kaplan has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19: Code-Review-1
(6 comments)
https://gerrit.ovirt.org/#/c/36368/19/lib/yajsonrpc/stomp.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19:
(1 comment)
https://gerrit.ovirt.org/#/c/36368/19/tests/jsonRpcHelper.py
File tests/jsonRpcHelper.py:
Line
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19:
(3 comments)
https://gerrit.ovirt.org/#/c/36368/19/tests/jsonRpcHelper.py
File tests/jsonRpcHelper.py:
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19:
(5 comments)
https://gerrit.ovirt.org/#/c/36368/19/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19:
(1 comment)
https://gerrit.ovirt.org/#/c/36368/19/lib/yajsonrpc/stompReactor.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19:
(1 comment)
https://gerrit.ovirt.org/#/c/36368/19/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19:
(1 comment)
https://gerrit.ovirt.org/#/c/36368/19/lib/yajsonrpc/stompReactor.py
File
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19:
(3 comments)
https://gerrit.ovirt.org/#/c/36368/19/tests/jsonRpcHelper.py
File tests/jsonRpcHelper.py:
Line
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 19:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 18:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 17:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18274/ : SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 17:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18274/
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 17:
Build Started (2/2)
0 - http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1504/
--
To
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 17:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 17: Verified+1
http://ur1.ca/k9e9x
--
To view, visit https://gerrit.ovirt.org/36368
To unsubscribe, visit
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 17:
Verified by installing a host, creating vm and seeing events being processed by
the engine.
There are some
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 16:
(2 comments)
https://gerrit.ovirt.org/#/c/36368/16/lib/yajsonrpc/stompReactor.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 16:
(7 comments)
https://gerrit.ovirt.org/#/c/36368/16//COMMIT_MSG
Commit Message:
Line 7: stomp: client side
Francesco Romani has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 16:
(7 comments)
initial review, I need (much) more time to properly understand this big and
complex patch.
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 16:
(2 comments)
https://gerrit.ovirt.org/#/c/36368/16/lib/yajsonrpc/stompReactor.py
File
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 16:
Build Started (2/2)
0 - http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1318/
--
To
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 16:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 16:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18089/
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 16:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18089/ : SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 15:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18042/
--
To view,
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 15:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 15: Verified+1
Patch split into several smaller.
Verified by host deploying vdsm containing all patches from
Yaniv Bronheim has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 14: Code-Review-1
please split format changes and client removal from this patch. the changes to
keep the
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 14:
Will split this patch.
--
To view, visit https://gerrit.ovirt.org/36368
To unsubscribe, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 14:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17778/
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 14:
Build Started (2/2) -
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17949/
--
To
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 14:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 12:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17577/
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 12:
Build Started (2/2) -
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17751/
--
To
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 12:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 12: Verified+1
Verified by running with the latest engine. Host installed, NFS domain
configured and vm
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 12:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17577/ : SUCCESS
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 11: Code-Review-1
(2 comments)
https://gerrit.ovirt.org/#/c/36368/11//COMMIT_MSG
Commit Message:
Line 7: stomp:
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 11:
(2 comments)
https://gerrit.ovirt.org/#/c/36368/11//COMMIT_MSG
Commit Message:
Line 7: stomp: client side
Francesco Romani has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 10:
(4 comments)
initial review, more will come after
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 10:
(4 comments)
https://gerrit.ovirt.org/#/c/36368/10/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 11:
Build Started (2/2) -
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17129/
--
To
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 11:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 11:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16956/
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 11:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16956/ : SUCCESS
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 11: Verified+1
Rebased, fixed comments, verified with latest engine (master).
--
To view, visit
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 9:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 9:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16780/ : SUCCESS
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 8: Verified+1
Rebased, verified with engine 3.5
--
To view, visit https://gerrit.ovirt.org/36368
To
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 8:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 8:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16749/ : SUCCESS
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 6: Verified-1
We do not send heartbeats anymore.
--
To view, visit https://gerrit.ovirt.org/36368
To
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 7: Verified+1
Using engine 3.5 to host install vdsm and seeing no communication issues.
--
To view, visit
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: client side subscription
..
Patch Set 7:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16649/ : SUCCESS
92 matches
Mail list logo