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

Queue authentication check only for ems_cloud and ems_infra providers. #2643

Merged
merged 1 commit into from
Nov 6, 2017

Conversation

jntullo
Copy link

@jntullo jntullo commented Nov 6, 2017

The current check allows middleware provider checks to be directed to the queue, but at this time should be restricted to only cloud and infra providers.

@miq-bot add_label bug
cc: @agrare

The current check allows middleware provider checks to be directed to the queue, but at this time should be restricted to only cloud and infra providers.
@miq-bot miq-bot added the bug label Nov 6, 2017
@miq-bot
Copy link
Member

miq-bot commented Nov 6, 2017

Checked commit jntullo@a1bbc09 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks fine. ⭐

Copy link
Member

@agrare agrare left a comment

Choose a reason for hiding this comment

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

LGTM tested with a few infra providers (scvmm, vmware) and an openshift provider

@agrare
Copy link
Member

agrare commented Nov 6, 2017

@miq-bot add-label gaprindashvili/yes

@agrare
Copy link
Member

agrare commented Nov 6, 2017

cc @simaishi

@agrare
Copy link
Member

agrare commented Nov 6, 2017

@miq-bot assign @Fryguy

@Fryguy Fryguy merged commit 16d30e3 into ManageIQ:master Nov 6, 2017
@Fryguy Fryguy added this to the Sprint 73 Ending Nov 13, 2017 milestone Nov 6, 2017
simaishi pushed a commit that referenced this pull request Nov 7, 2017
Queue authentication check only for ems_cloud and ems_infra providers.
(cherry picked from commit 16d30e3)
@simaishi
Copy link
Contributor

simaishi commented Nov 7, 2017

Gaprindashvili backport details:

$ git log -1
commit 0d79c6d26a1b8522273e2b03db12d689d00de4a6
Author: Jason Frey <[email protected]>
Date:   Mon Nov 6 14:18:35 2017 -0500

    Merge pull request #2643 from jntullo/bug/only_queue_infra_cloud
    
    Queue authentication check only for ems_cloud and ems_infra providers.
    (cherry picked from commit 16d30e351f01dd378224690a625489477a5884bd)

@abonas
Copy link
Member

abonas commented Nov 7, 2017

@tumido
Copy link
Member

tumido commented Nov 7, 2017

@abonas, yes. It's reverting Hawkular back to the old way of validating credentials. The PRs we're working on (ManageIQ/manageiq-providers-hawkular#100, #2577) are required, so we can join in displaying the new view in close future (?). This PR should unblock us on testing Hawkular for the release.

@aljesusg
Copy link
Member

aljesusg commented Nov 7, 2017

Yes @abonas now is working with this PR, and I validate with CA too. with correct error and sucess messages. We can close #2577.

@tumido
Copy link
Member

tumido commented Nov 7, 2017

@aljesusg, @abonas no, we shouldn't be closing these PRs (#2577, ManageIQ/manageiq-providers-hawkular#100), we're gonna need that code at some point.

@tumido
Copy link
Member

tumido commented Nov 7, 2017

@abonas, we (me adn @aljesusg) have PRs ready for the new dialog. Once they are merged we can be transitioned to the new dialog. Also, this change does not have to happen at the same moment, we're accepting both workflows.

PRs required for Hawkular to work with the new form:

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.

8 participants