Dan Kenigsberg has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 29: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/38451
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 30:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view,
Dan Kenigsberg has submitted this change and it was merged.
Change subject: stomp: server side subscriptions
..
stomp: server side subscriptions
Vdsm gains abitlity to manage subsctiptions and act as mini broker.
We change
Dan Kenigsberg has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 29: Continuous-Integration+1
--
To view, visit https://gerrit.ovirt.org/38451
To unsubscribe, visit
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 28:
(8 comments)
https://gerrit.ovirt.org/#/c/38451/28/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 28:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/28/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 28:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/28/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 28:
(8 comments)
https://gerrit.ovirt.org/#/c/38451/28/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 28:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/28/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 28:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/28/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 29: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/38451
To unsubscribe, visit
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 29:
Build failure not related to this change.
--
To view, visit https://gerrit.ovirt.org/38451
To
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 29: Verified+1
Verified with latest engine containing event changes.
--
To view, visit
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
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: server side subscriptions
..
Patch Set 28:
(2 comments)
https://gerrit.ovirt.org/#/c/38451/28/lib/yajsonrpc/stompreactor.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 28:
(2 comments)
https://gerrit.ovirt.org/#/c/38451/28/lib/yajsonrpc/stompreactor.py
File
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 28:
* 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: server side subscriptions
..
Patch Set 27:
* 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: server side subscriptions
..
Patch Set 26:
* 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: server side subscriptions
..
Patch Set 26: 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: server side subscriptions
..
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: server side subscriptions
..
Patch Set 24:
* 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: server side subscriptions
..
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: server side subscriptions
..
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: server side subscriptions
..
Patch Set 22: Verified+1
Verified by vdsm updated and see that there are no issues with communication.
--
To view,
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 22:
* 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: server side subscriptions
..
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: server side subscriptions
..
Patch Set 20: Verified+1
Reduced scope, 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: server side subscriptions
..
Patch Set 19:
(3 comments)
https://gerrit.ovirt.org/#/c/38451/19/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Dan Kenigsberg has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 19:
(3 comments)
https://gerrit.ovirt.org/#/c/38451/19/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 19:
* 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: server side subscriptions
..
Patch Set 19: 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: server side subscriptions
..
Patch Set 18: Code-Review-1
(3 comments)
https://gerrit.ovirt.org/#/c/38451/18/lib/yajsonrpc/stomp.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 18:
(3 comments)
https://gerrit.ovirt.org/#/c/38451/18/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 18:
* 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: server side subscriptions
..
Patch Set 17:
* 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: server side subscriptions
..
Patch Set 16:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18272/
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 16:
Build Started (2/2)
0 - http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1502/
--
To
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 16:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18272/ : SUCCESS
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 16: Verified+1
Verified by installing a host, creating vm and seeing events being processed by
the engine.
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 16:
* 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: server side subscriptions
..
Patch Set 15:
(2 comments)
https://gerrit.ovirt.org/#/c/38451/15/lib/yajsonrpc/stompReactor.py
File
Michal Skrivanek has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 15:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/15/lib/yajsonrpc/stompReactor.py
File
Michal Skrivanek has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 15:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/15/lib/yajsonrpc/stompReactor.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 15:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/15/lib/yajsonrpc/stompReactor.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 15:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/15/lib/yajsonrpc/stompReactor.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 15:
(7 comments)
https://gerrit.ovirt.org/#/c/38451/15//COMMIT_MSG
Commit Message:
Line 5: CommitDate:
Francesco Romani has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 15:
(7 comments)
initial review, need more time to understand the changes and the context of them
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 15:
(2 comments)
https://gerrit.ovirt.org/#/c/38451/15/lib/yajsonrpc/stompReactor.py
File
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 15:
Build Started (2/2)
0 - http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1316/
--
To
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 15:
* 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: server side subscriptions
..
Patch Set 15:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18087/
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 15:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18087/ : SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 14:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18041/
--
To view,
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 14: Verified+1
Patch split into several smaller.
Verified by host deploying vdsm containing all patches from
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 14:
Build Started (2/2)
0 - http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1270/
--
To
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 14:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18041/ : SUCCESS
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 14:
* 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: server side subscriptions
..
Patch Set 13:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/13/lib/yajsonrpc/stompReactor.py
File
Yaniv Bronhaim has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 13: Code-Review-1
(14 comments)
https://gerrit.ovirt.org/#/c/38451/13/lib/yajsonrpc/__init__.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 13:
(14 comments)
https://gerrit.ovirt.org/#/c/38451/13/lib/yajsonrpc/__init__.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 12: -Code-Review
--
To view, visit https://gerrit.ovirt.org/38451
To unsubscribe, visit
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 13: Verified+1
Fixing issue in processing of unsubscribe frame.
--
To view, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 13:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/1/
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 13:
Build Started (2/2) -
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17948/
--
To
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 13:
* 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: server side subscriptions
..
Patch Set 13:
Verified by installing host with the latest events changes using engine with
latest event changes.
--
To
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
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: server side subscriptions
..
Patch Set 11:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17576/
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 11:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17576/ : SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 11:
Build Started (2/2) -
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17750/
--
To
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 11: -Code-Review Verified+1
Verified by running with the latest engine. Host installed, NFS domain
configured
Michal Skrivanek has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 9:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/9/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
Francesco Romani has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 9:
(1 comment)
VERY initial review
https://gerrit.ovirt.org/#/c/38451/9/lib/yajsonrpc/stompReactor.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 9:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/9/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
Michal Skrivanek has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 9:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/9/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 9:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/9/lib/yajsonrpc/stompReactor.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 9:
(1 comment)
https://gerrit.ovirt.org/#/c/38451/9/lib/yajsonrpc/stomp.py
File lib/yajsonrpc/stomp.py:
Line
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 10:
Build Started (2/2) -
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17128/
--
To
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 10:
* 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: server side subscriptions
..
Patch Set 10:
Build Started (1/2) -
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16955/
--
To view,
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 10: Code-Review+1
Rebased, fixed comments, verified with latest engine (master).
--
To view, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 10:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16955/ : SUCCESS
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
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: server side subscriptions
..
Patch Set 9:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16865/ : SUCCESS
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 8: Verified+1
Verified with engine based on latest master
--
To view, visit https://gerrit.ovirt.org/38451
To
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 7: Verified-1
Failed verification with engine using latest master.
--
To view, visit
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 8:
* 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: server side subscriptions
..
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: server side subscriptions
..
Patch Set 8:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16800/ : SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 7:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16788/ : SUCCESS
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 6: Verified+1
Rebased, fixed comments, Verified with engine 3.5
--
To view, visit
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 6:
* 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: server side subscriptions
..
Patch Set 6:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16757/ : SUCCESS
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 4: Verified-1
We do not send heartbeats anymore.
--
To view, visit https://gerrit.ovirt.org/38451
To
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 5:
Using engine 3.5 to host install vdsm and seeing no communication issues.
--
To view, visit
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 5: Verified+1
--
To view, visit https://gerrit.ovirt.org/38451
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: stomp: server side subscriptions
..
Patch Set 5:
* 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: server side subscriptions
..
Patch Set 4:
* 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: server side subscriptions
..
Patch Set 5:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16657/ : SUCCESS
100 matches
Mail list logo