Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix VNC console connection to Windows VMs and Ctrl-Alt-Del button #1672

Conversation

bmclaughlin
Copy link
Contributor

@miq-bot add_labels bug, compute/infrastructure, blocker

https://bugzilla.redhat.com/show_bug.cgi?id=1464152

@h-kataria
Copy link
Contributor

verified fix in UI, VNC console works with the fix.

@h-kataria
Copy link
Contributor

Ready to be merged once tests are green.

@miq-bot
Copy link
Member

miq-bot commented Jul 11, 2017

Checked commits bmclaughlin/manageiq-ui-classic@859dbe0~...3103ec5 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🏆

@h-kataria h-kataria added this to the Sprint 65 Ending Jul 24, 2017 milestone Jul 11, 2017
@h-kataria h-kataria merged commit c9729ad into ManageIQ:master Jul 11, 2017
@bmclaughlin bmclaughlin deleted the fix-vnc-consoles-for-windows-vms-on-vmware branch July 12, 2017 11:26
simaishi pushed a commit that referenced this pull request Jul 12, 2017
…s-vms-on-vmware

Fix VNC console connection to Windows VMs and Ctrl-Alt-Del button
(cherry picked from commit c9729ad)

https://bugzilla.redhat.com/show_bug.cgi?id=1470179
@simaishi
Copy link
Contributor

Fine backport details:

$ git log -1
commit 3083683fd9e05f0f72cce55b2de16e0d07f0c847
Author: Harpreet Kataria <[email protected]>
Date:   Tue Jul 11 17:49:45 2017 -0400

    Merge pull request #1672 from bmclaughlin/fix-vnc-consoles-for-windows-vms-on-vmware
    
    Fix VNC console connection to Windows VMs and Ctrl-Alt-Del button
    (cherry picked from commit c9729ad614f26af344d009d16315d4023be45ebc)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1470179

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants