Timothy Hildred has posted comments on this change. Change subject: ux: remote-viewer is a graphical app ......................................................................
Patch Set 1: Sorry I'm late the the party, I was on PTO for a couple of weeks. I have two comments: - We sell RHEV as "easy to use" to shops that generally use windows. As it is, the reminder is essentially meaningless to Windows admins: "Please note that in order to use remote-viewer you need to be able to run graphical applications." Be specific about what your users need. This would be better: "In order to use the remote-viewer client to connect to and control your hosted-engine virtual machine, the hosted-engine virtual machine requires software allowing graphical applications to run. At the very least, you need the following packages installed on your hosted-engine virtual machine to allow remote-viewer connections:{insert required packages here}" - my other comment: there is no mention of console connections in this code. Is that on purpose? -- To view, visit http://gerrit.ovirt.org/18930 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id5e24474546caf3067ad48eab3cdb699e2fa6af6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: Alex Lourie <alou...@redhat.com> Gerrit-Reviewer: Greg Padgett <gpadg...@redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnata...@gmail.com> Gerrit-Reviewer: Lev Veyde <lve...@gmail.com> Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: Timothy Hildred <thild...@redhat.com> Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com> Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches