convert console_supported? to supports? - #864
Conversation
c8ee806 to
8ca203f
Compare
… supports
vnc, webmks, and console are not consistent
webmks also adds login validation checking.
Added the vm must be running check from :launch_{}_console
feature. That feature has been rolled into :{}_console
8ca203f to
2e9ced0
Compare
|
This pull request has been automatically marked as stale because it has not been updated for at least 3 months. If these changes are still valid, please remove the Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation. |
|
This pull request has been automatically closed because it has not been updated for at least 3 months. Feel free to reopen this pull request if these changes are still valid. Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation. |
|
outstanding: |
|
Checked commits kbrock/manageiq-providers-vmware@0aef0d0~...2e9ced0 with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint app/models/manageiq/providers/vmware/infra_manager/vm/remote_console.rb
|
|
This pull request has been automatically marked as stale because it has not been updated for at least 3 months. If these changes are still valid, please remove the |
7 similar comments
|
This pull request has been automatically marked as stale because it has not been updated for at least 3 months. If these changes are still valid, please remove the |
|
This pull request has been automatically marked as stale because it has not been updated for at least 3 months. If these changes are still valid, please remove the |
|
This pull request has been automatically marked as stale because it has not been updated for at least 3 months. If these changes are still valid, please remove the |
|
This pull request has been automatically marked as stale because it has not been updated for at least 3 months. If these changes are still valid, please remove the |
|
This pull request has been automatically marked as stale because it has not been updated for at least 3 months. If these changes are still valid, please remove the |
|
This pull request has been automatically marked as stale because it has not been updated for at least 3 months. If these changes are still valid, please remove the |
|
This pull request has been automatically marked as stale because it has not been updated for at least 3 months. If these changes are still valid, please remove the |
|
|
||
| included do | ||
| supports :console | ||
| supports :html5_console |
There was a problem hiding this comment.
It looks like this only supported webmks previously, would html5 have been supported before even though console_supported?(html5) would have been false?
| supports :html5_console |
part of ManageIQ/manageiq#21990