Bug 907931 - [RFE] ovirt-shell vnc console does not work with tigervnc client
Summary: [RFE] ovirt-shell vnc console does not work with tigervnc client
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-cli
Version: 3.2
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.3.4
Assignee: Michael Pasternak
QA Contact:
URL:
Whiteboard: infra
Depends On:
Blocks: 908016 909365
TreeView+ depends on / blocked
 
Reported: 2013-02-05 14:59 UTC by Sander Hoentjen
Modified: 2014-07-20 06:13 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
: 908016 909365 (view as bug list)
Environment:
Last Closed: 2013-12-02 07:53:09 UTC
oVirt Team: ---
Embargoed:


Attachments (Terms of Use)
patch that solves the issue for me (1.02 KB, patch)
2013-02-06 20:23 UTC, Sander Hoentjen
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 11857 0 None None None Never

Description Sander Hoentjen 2013-02-05 14:59:19 UTC
Description of problem:
the console command runs vncviewer <host>:<port> -passwdInput, but -passwdInput is not supported with tigervnc

Version-Release number of selected component (if applicable):
ovirt-engine-cli-3.2.0.9-1.fc18.noarch

How reproducible:
Always

Steps to Reproduce:
1. connect with ovirt-shell 
2. console <vmname> where vm has display set as VNC
  
Actual results:
no console is displayed

Expected results:
console is displayed

Additional info:
command run now is:
echo <ticket> | <host>::<port> -passwdInput

what would work is:
echo <ticket> | vncpasswd -f | vncviewer <host>::<port> -passwordFile /dev/stdin

relevant code is in /usr/lib/python2.7/site-packages/ovirtcli/platform/posix/vnc.py

Comment 1 Sander Hoentjen 2013-02-06 20:21:35 UTC
This is because in F18 the patch was dropped to add -passwdInput to vncviewer:
See: http://koji.fedoraproject.org/koji/buildinfo?buildID=365068
"""
* Fri Aug 17 2012 Adam Tkac <atkac redhat com> 1.2.80-0.1.20120905svn4996
- update to 1.2.80
- remove deprecated patches
 - tigervnc-102434.patch
 - tigervnc-viewer-reparent.patch
 - tigervnc11-java7.patch
- patches merged
 - tigervnc11-xorg111.patch
 - tigervnc11-xorg112.patch
"""
tigervnc-102434.patch is the patch that implemented -passwdInput:
http://pkgs.fedoraproject.org/cgit/tigervnc.git/commit/?id=5e164de8bf7b74db5b65e29123dd63b6fc6fd1fc

Comment 2 Sander Hoentjen 2013-02-06 20:23:06 UTC
Created attachment 694128 [details]
patch that solves the issue for me

Comment 3 Juan Hernández 2013-02-08 09:03:52 UTC
Sander, I think that this patch is worh adding upstream, would you consider pushing it to the upstream gerrit? Meanwhile I will apply it to the package.

Comment 4 Sander Hoentjen 2013-02-08 09:57:38 UTC
Juan: http://gerrit.ovirt.org/#/c/11857/

Comment 8 Itamar Heim 2013-12-01 10:22:27 UTC
i see patch was merged a while back - should this be closed current release by now?


Note You need to log in before you can comment on or make changes to this bug.