David Caro has posted comments on this change.

Change subject: Added possible json option value encoding
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/35238/1/mock_configs/mock_genconfig
File mock_configs/mock_genconfig:

Line 57: value
> Because mock expects a python file, not a json one
Until now with simple types, the json strings matched the python strings, so no 
problem, but now that we want more complex options like arrays json output is 
no longer valid


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I66feb645fab0308b7fea9e759cdb5d88b2278a41
Gerrit-PatchSet: 1
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: David Caro <dcaro...@redhat.com>
Gerrit-Reviewer: Barak Korren <bkor...@redhat.com>
Gerrit-Reviewer: David Caro <dcaro...@redhat.com>
Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ew...@kohlvanwijngaarden.nl>
Gerrit-Reviewer: Eyal Edri <ee...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to