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

Add decorator for VNC Console #167

Merged
merged 1 commit into from
Nov 3, 2017
Merged

Add decorator for VNC Console #167

merged 1 commit into from
Nov 3, 2017

Conversation

jntullo
Copy link

@jntullo jntullo commented Nov 1, 2017

SUI requires a check on VNC console because the existing decorators lead to a false positive about console enablement for VMWare consoles.

@miq-bot add_label bug, gaprindashvili/yes, fine/yes
cc: @AllenBW

SUI requires a check on VNC console because the existing decorators lead to a false positive about console enablement for VMWare consoles.
@jntullo
Copy link
Author

jntullo commented Nov 1, 2017

@miq-bot assign @abellotti

@miq-bot
Copy link
Member

miq-bot commented Nov 1, 2017

Checked commit jntullo@57943ce with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 🍰

@abellotti
Copy link
Member

@jntullo is this PR really fine/yes ? shouldn't that be in a different repo ?

@abellotti
Copy link
Member

@simaishi this needs to be back-ported to Fine via ManageIQ/manageiq#16382. Thanks.

@abellotti abellotti added this to the Sprint 73 Ending Nov 13, 2017 milestone Nov 3, 2017
@abellotti
Copy link
Member

LGTM!! Thanks @jntullo for fixing this. 👍

@abellotti abellotti merged commit 9957510 into ManageIQ:master Nov 3, 2017
simaishi pushed a commit that referenced this pull request Nov 6, 2017
@simaishi
Copy link
Contributor

simaishi commented Nov 6, 2017

Gaprindashvili backport details:

$ git log -1
commit a132cb400dce012823384dd670eada3c0ff4b901
Author: Alberto Bellotti <abellotti@users.noreply.github.com>
Date:   Fri Nov 3 08:30:54 2017 -0400

    Merge pull request #167 from jntullo/bug/vnc_console
    
    Add decorator for VNC Console
    (cherry picked from commit 995751041d9f0d2667bc700f923de5796e93a024)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1510131

@simaishi
Copy link
Contributor

Backported to Euwe via ManageIQ/manageiq#16377

@simaishi
Copy link
Contributor

Backported to Fine via ManageIQ/manageiq#16382

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