Change in vdsm[master]: Add multi-threading option to JsonRpcServer

List overview All Threads
Download

newer

older

Change in vdsm[master]: Federate...

Change in vdsm[master]: Fix race...

smizrahi＠redhat.com

25 Dec 2012 25 Dec '12

9:14 p.m.

Saggi Mizrahi has uploaded a new change for review.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Add multi-threading option to JsonRpcServer

Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Signed-off-by: Saggi Mizrahi smizrahi@redhat.com --- M vdsm_api/BindingJsonRpc.py M vdsm_api/jsonrpc/__init__.py 2 files changed, 14 insertions(+), 3 deletions(-)

git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/74/10374/1

diff --git a/vdsm_api/BindingJsonRpc.py b/vdsm_api/BindingJsonRpc.py index 645428e..11080b5 100644 --- a/vdsm_api/BindingJsonRpc.py +++ b/vdsm_api/BindingJsonRpc.py @@ -28,13 +28,19 @@ pass

+def _simpleThreadFactory(func): + t = threading.Thread(target=func) + t.setDaemon(False) + t.start() + + class BindingJsonRpc(object): log = logging.getLogger('BindingJsonRpc')

def __init__(self, bridge, backendConfig): reactors = [] self.bridge = bridge - self.server = JsonRpcServer(bridge) + self.server = JsonRpcServer(bridge, _simpleThreadFactory) for backendType, cfg in backendConfig: if backendType == "tcp": reactors.append(self._createTcpReactor(cfg)) diff --git a/vdsm_api/jsonrpc/__init__.py b/vdsm_api/jsonrpc/__init__.py index 9463262..2ef6e36 100644 --- a/vdsm_api/jsonrpc/__init__.py +++ b/vdsm_api/jsonrpc/__init__.py @@ -15,6 +15,7 @@ import json import logging from Queue import Queue +from functools import partial

__all__ = ["tcpReactor"]

@@ -93,9 +94,10 @@ class JsonRpcServer(object): log = logging.getLogger("jsonrpc.JsonRpcServer")

- def __init__(self, bridge): + def __init__(self, bridge, threadFactory=None): self._bridge = bridge self._workQueue = Queue() + self._threadFactory = threadFactory

def _parseMessage(self, msg): try: @@ -180,7 +182,10 @@ break

if isinstance(obj, _JsonRpcRequest): - self._serveRequest(obj) + if self._threadFactory is None: + self._serveRequest(obj) + else: + self._threadFactory(partial(self._serveRequest, obj)) else: self._processResponse(obj)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com

Show replies by date

Vinzenz Feenstra

26 Dec 26 Dec

7:13 a.m.

Vinzenz Feenstra has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 1: Looks good to me, but someone else must approve

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com

Jenkins CI RO

3 Jan 3 Jan

4:25 p.m.

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 2:

Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/536/ (1/2)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server

Jenkins CI RO

4:35 p.m.

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 2:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/571/ (2/2)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

4:42 p.m.

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 2:

Build Successful

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/536/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/571/ : SUCCESS

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Vinzenz Feenstra

6:16 p.m.

Vinzenz Feenstra has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 2: Looks good to me, but someone else must approve

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

6:18 p.m.

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 3:

Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/553/ (1/2)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server

Jenkins CI RO

6:30 p.m.

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 3:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/588/ (2/2)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

6:35 p.m.

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 3:

Build Successful

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/553/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/588/ : SUCCESS

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Vinzenz Feenstra

6:49 p.m.

Vinzenz Feenstra has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 3: Looks good to me, but someone else must approve

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

agl＠us.ibm.com

14 Jan 14 Jan

3:32 p.m.

Adam Litke has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 3: Looks good to me, but someone else must approve

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server

Jenkins CI RO

4:25 p.m.

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 5:

Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/701/ (1/2)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server

Jenkins CI RO

4:25 p.m.

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 5:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/736/ (2/2)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

4:32 p.m.

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 5:

Build Successful

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/701/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/736/ : SUCCESS

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

15 Jan 15 Jan

9:48 p.m.

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 6:

Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/734/ (1/2)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server

Jenkins CI RO

9:57 p.m.

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 6:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/769/ (2/2)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

10:02 p.m.

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 6:

Build Successful

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/734/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/769/ : SUCCESS

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

26 Jan 26 Jan

12:08 a.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 7:

Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/925/ (1/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server

Jenkins CI RO

12:10 a.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 7:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/70/ (2/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

12:13 a.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 7:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/960/ (3/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

12:15 a.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 7: Fails

Build Failed

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/925/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/960/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/70/ : FAILURE

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

28 Jan 28 Jan

10:46 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 8:

Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/965/ (1/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server

Jenkins CI RO

10:47 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 8:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1000/ (2/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

10:49 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 8:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/111/ (3/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

10:53 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 8:

Build Successful

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/965/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1000/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/111/ : SUCCESS

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

31 Jan 31 Jan

6:01 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 9:

Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1021/ (1/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 9 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server

Jenkins CI RO

6:04 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 9:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/167/ (2/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

6:05 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 9:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1056/ (3/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

6:08 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 9:

Build Successful

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1021/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1056/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/167/ : SUCCESS

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

4 Feb 4 Feb

8:43 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 10:

Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1088/ (1/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 10 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server

Jenkins CI RO

8:52 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 10:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/234/ (2/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

8:54 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 10:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1123/ (3/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

8:57 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 10:

Build Successful

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1088/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1123/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/234/ : SUCCESS

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

5 Feb 5 Feb

4:27 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 11:

Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1136/ (1/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 11 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server

Jenkins CI RO

4:33 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 11:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/282/ (2/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

4:40 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 11:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1171/ (3/3)

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Jenkins CI RO

4:44 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

oVirt Jenkins CI Server has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 11:

Build Successful

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1136/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1171/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/282/ : SUCCESS

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

agl＠us.ibm.com

6 Feb 6 Feb

5:23 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

Adam Litke has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 11: Looks good to me, but someone else must approve

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

agl＠us.ibm.com

5:58 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

Adam Litke has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 11: Verified

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Dan Kenigsberg

7 Feb 7 Feb

9:51 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

Dan Kenigsberg has posted comments on this change.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

Patch Set 11: Looks good to me, approved

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 11 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server

Dan Kenigsberg

9:51 p.m.

New subject: Change in vdsm[master]: jsonrpc: Add multi-threading option to JsonRpcServer

Dan Kenigsberg has submitted this change and it was merged.

Change subject: jsonrpc: Add multi-threading option to JsonRpcServer ......................................................................

jsonrpc: Add multi-threading option to JsonRpcServer

Approvals: Adam Litke: Verified; Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved

-- To view, visit http://gerrit.ovirt.org/10374 To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged Gerrit-Change-Id: I0a37dfcf3cfde92c8552e6c6eef5016efe3fbd6d Gerrit-PatchSet: 11 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Vinzenz Feenstra vfeenstr@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server

4163

Age (days ago)

4207

Last active (days ago)

vdsm-patches@lists.fedorahosted.org

40 comments

5 participants

tags (0)

participants (5)

agl＠us.ibm.com
Dan Kenigsberg
Jenkins CI RO
smizrahi＠redhat.com
Vinzenz Feenstra