Francesco Romani has posted comments on this change. Change subject: virt: Don't fail when existingConnAction is unset for a SPICE device ......................................................................
Patch Set 3: Code-Review+2 (2 comments) Indeed the tests are starting to become too complex. We should oimprove this, but in a future patch set. This patch is good enough I believe. https://gerrit.ovirt.org/#/c/56836/3/lib/api/vdsmapi-schema.json File lib/api/vdsmapi-schema.json: Line 7402: # @password: The desired connection password Line 7403: # Line 7404: # @ttl: The number of seconds before the password expires Line 7405: # Line 7406: # @existingConnAction: #optional Indicate what to do with any existing connections please add a line like (made optional in version 4.1y.z) there are other examples in the schema Line 7407: # Line 7408: # @params: Arbitrary key:val pairs that will be passed to hooks Line 7409: # Line 7410: # Since: 4.15.0 https://gerrit.ovirt.org/#/c/56836/3/tests/vmOperationsTests.py File tests/vmOperationsTests.py: PS3, Line 146: graphics_params = {k: v for k, v in _GRAPHICS_DEVICE_PARAMS.items() : if k != 'existingConnAction'} silly nit: I think graphic_params = dict(_GRAPHICS_DEVICE_PARAMS) del graphic_params['existingConnAction'] is a bit clearer, but surely *not* worth resubmit for this reason. Please just consider this for next patches. -- To view, visit https://gerrit.ovirt.org/56836 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2fde83ceb994eaafd2aa956d1fa82ce6cb16094c Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeen...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches