UI: Add authmethod field allowing to choose password or ssh key when adding host#6525
Conversation
|
Found UI changes, kicking a new UI QA build |
|
@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
|
UI build: ✔️ |
|
Thanks for the PR @utchoang. The thing I saw with other hypervisors is the password tab appears under "Authentication Method". Is there a way we can skip both the "Authentication Method" heading and Password tab completely? Following is the screenshot I saw when I go back and try to add Xenserver cluster and host |
|
Found UI changes, kicking a new UI QA build |
|
Found UI changes, kicking a new UI QA build |
|
@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
|
UI build: ✔️ |
|
Kudos, SonarCloud Quality Gate passed! |
|
@harikrishna-patnala Same with "Add Host" it remains for non-KVM hypervisors. I think it's okay to keep it, but if you want I can delete it. |
harikrishna-patnala
left a comment
There was a problem hiding this comment.
Yeah, let's keep it the same way here, may be it helps in future when we add it to other hypervisors. Thanks for all the changes @utchoang
LGTM
|
@sureshanaparti a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
|
UI build: ✔️ |
sureshanaparti
left a comment
There was a problem hiding this comment.
Verified zone wizard, LGTM.













Description
This PR for fixes #6499
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?