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

Login with console authentication if available #125

Merged

Conversation

agrare
Copy link
Member

@agrare agrare commented Oct 24, 2017

Allow different credentials for connecting to a vmware console.
Passing :console will fallback to standard credentials if they are missing.

@agrare
Copy link
Member Author

agrare commented Oct 24, 2017

cc @skateman

@skateman
Copy link
Member

@agrare I think we'd need a supports authentication type thingie in the same PR

@agrare
Copy link
Member Author

agrare commented Oct 30, 2017

@skateman 👍 added, PTAL

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.

The Seal of Approval

@agrare
Copy link
Member Author

agrare commented Nov 1, 2017

@blomquisg this is ready can you review?

@skateman
Copy link
Member

skateman commented Nov 1, 2017

@agrare @blomquisg I tested this and it works as expected 🎉
But, what happens if these credentials aren't specified? Does it fall back to the default ones?

@blomquisg
Copy link
Member

@skateman, @agrare said:

Passing :console will fallback to standard credentials if they are missing.

@agrare is there a test you can add for that?

@agrare agrare force-pushed the add_option_for_console_authentication branch from f5875d2 to 6818be1 Compare November 1, 2017 12:56
@agrare
Copy link
Member Author

agrare commented Nov 1, 2017

@agrare is there a test you can add for that?

@blomquisg it was a little gross because we actually only get the username and password wayyyy down in MiqFaultTolerantVim but yes, there is :)

@miq-bot
Copy link
Member

miq-bot commented Nov 1, 2017

Checked commits agrare/manageiq-providers-vmware@e2ed49c~...6818be1 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. ⭐

@agrare
Copy link
Member Author

agrare commented Nov 1, 2017

Okay @blomquisg new test is 🍏 :)

@blomquisg blomquisg merged commit f8943a1 into ManageIQ:master Nov 2, 2017
@blomquisg blomquisg added this to the Sprint 73 Ending Nov 13, 2017 milestone Nov 2, 2017
@agrare agrare deleted the add_option_for_console_authentication branch November 2, 2017 20:53
@simaishi
Copy link
Contributor

simaishi commented Nov 6, 2017

@agrare Is there a BZ for this?

simaishi pushed a commit that referenced this pull request Nov 6, 2017
…tion

Login with console authentication if available
(cherry picked from commit f8943a1)
@simaishi
Copy link
Contributor

simaishi commented Nov 6, 2017

Gaprindashvili backport details:

$ git log -1
commit f07dd9e142089620f9565af470ceb0a6de4d7085
Author: Greg Blomquist <[email protected]>
Date:   Thu Nov 2 16:13:44 2017 -0400

    Merge pull request #125 from agrare/add_option_for_console_authentication
    
    Login with console authentication if available
    (cherry picked from commit f8943a1fcc8c85ec01c2ff440611609b180449fb)

simaishi pushed a commit that referenced this pull request Nov 9, 2017
…tion

Login with console authentication if available
(cherry picked from commit f8943a1)
@simaishi
Copy link
Contributor

simaishi commented Nov 9, 2017

Fine backport details:

$ git log -1
commit c341956f132e54d1be96393688abca99e6971d98
Author: Greg Blomquist <[email protected]>
Date:   Thu Nov 2 16:13:44 2017 -0400

    Merge pull request #125 from agrare/add_option_for_console_authentication
    
    Login with console authentication if available
    (cherry picked from commit f8943a1fcc8c85ec01c2ff440611609b180449fb)

agrare pushed a commit to agrare/manageiq-providers-vmware that referenced this pull request Apr 15, 2019
Enhanced messaging and error notification for update_migration_status method.
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.

5 participants