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

Disable VNC Console for VMs hosted on ESXi 6.5 or greater #355

Merged
merged 1 commit into from
Feb 14, 2017

Conversation

bmclaughlin
Copy link
Contributor

It appears per ManageIQ/manageiq#13798 VMware disabled VNC consoles in 6.5. Testing resulted in a "WebSocket connection to failed: Connection closed before receiving a handshake response".

@bmclaughlin
Copy link
Contributor Author

@miq-bot add-label enhancement

@bmclaughlin
Copy link
Contributor Author

@skateman, please review.

@skateman
Copy link
Member

skateman commented Feb 13, 2017

@bmclaughlin do we have a provider with 6.5 to test this? PM me please!

@skateman
Copy link
Member

Almost perfect, could you please squash the two commits? The tests should be together with the feature. Thanks!

@bmclaughlin bmclaughlin force-pushed the disable-vnc-button-for-vc65 branch from 52692d4 to a64c9be Compare February 13, 2017 18:06
@bmclaughlin
Copy link
Contributor Author

@skateman, squashed.

@miq-bot
Copy link
Member

miq-bot commented Feb 13, 2017

Checked commit bmclaughlin@a64c9be with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks good. 🍪

Copy link
Member

@skateman skateman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mzazrivec mzazrivec self-assigned this Feb 14, 2017
@mzazrivec mzazrivec added this to the Sprint 55 Ending Feb 27, 2017 milestone Feb 14, 2017
@mzazrivec mzazrivec merged commit f6021af into ManageIQ:master Feb 14, 2017
@bmclaughlin bmclaughlin deleted the disable-vnc-button-for-vc65 branch February 14, 2017 14:23
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