Eli Mesika has posted comments on this change. Change subject: core: added support for version check for remote-viewer ......................................................................
Patch Set 6: (1 comment) https://gerrit.ovirt.org/#/c/41879/6/packaging/etc/engine-config/engine-config.properties File packaging/etc/engine-config/engine-config.properties: Line 281: SpiceProxyDefault.type=String Line 282: RemoteViewerSupportedVersions.description='List of os/remote-viewer minimal version supported by oVirt. Format: "linux:3.0;windows:2.5' Line 283: RemoteViewerSupportedVersions.type=String Line 284: RemoteViewerNewerVersionUrl.description='URL shown when the remote-viewer version check does not pass' Line 285: RemoteViewerNewerVersionUrl.type=String since those implies a specific format , I would expect also a type that validates the input and not getting a wrong format as an exception in run-time Line 286: RemapCtrlAltDelDefault.description="Determines whether remapping Ctrl-Alt-Delete to Ctrl-Alt-End should be preselected by default in the UI." Line 287: RemapCtrlAltDelDefault.type=Boolean Line 288: ClientModeSpiceDefault.description="Default SPICE console invocation behavior in the engine." Line 289: ClientModeSpiceDefault.validValues=Auto,Plugin,Native -- To view, visit https://gerrit.ovirt.org/41879 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4f99a1b41b193a172d10257a910675c6058b037b Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjeli...@redhat.com> Gerrit-Reviewer: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
