Change in vdsm[master]: jsonrpc: change instance type to strings

danken at redhat.com danken at redhat.com
Thu May 5 10:19:56 UTC 2016


Dan Kenigsberg has submitted this change and it was merged.

Change subject: jsonrpc: change instance type to strings
......................................................................


jsonrpc: change instance type to strings

Hosted engine deployment failed, because to narrow check
for jsonrpc response.
Change check of responce instance type from unicode
to strings

Change-Id: Ic4eab12560b63d82223babe0b14870b6c0b65fbb
Bug-Url: https://bugzilla.redhat.com/1332923
Signed-off-by: Lukianov Artyom <alukiano at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/57017
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Simone Tiraboschi <stirabos at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-by: Piotr Kliczewski <piotr.kliczewski at gmail.com>
---
M lib/vdsm/jsonrpcvdscli.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Piotr Kliczewski: Looks good to me, approved
  Simone Tiraboschi: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, but someone else must approve
  Francesco Romani: Looks good to me, but someone else must approve
  Artyom Lukianov: Verified



-- 
To view, visit https://gerrit.ovirt.org/57017
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic4eab12560b63d82223babe0b14870b6c0b65fbb
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Artyom Lukianov <alukiano at redhat.com>
Gerrit-Reviewer: Artyom Lukianov <alukiano at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list