Project

General

Profile

Actions
Copy link

Bug #32526

closed

password for VMs is marked as required, while it is not

Added by Evgeni Golov over 3 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Evgeni Golov
Category:
Foreman integration
Target version:
2.2.4
Difficulty:
trivial
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman_azure_rm/pull/118
Fixed in Releases:
2.2.4
Found in Releases:
Red Hat JIRA:

Description

Ohai,

when configuring a Compute Profile for Azure, the UI indicates that the "password" field is required:

But in fact, nothing asserts it to be a non-empty string, which actually is good.
Azure only really requires the password if there is no ssh key configured, which the plugin actually does for us:
https://github.com/theforeman/foreman_azure_rm/blob/f317f0d3d409a7ad10231bdafa44ace90478ab5f/app/models/concerns/foreman_azure_rm/vm_extensions/managed_vm.rb#L194-L207

It copies both, the key added in the UI, but also an own key.

Actions
Copy link
 #1

Updated by The Foreman Bot over 3 years ago

  • Assignee set to Evgeni Golov
  • Pull request https://github.com/theforeman/foreman_azure_rm/pull/118 added
Actions
Copy link
 #2

Updated by The Foreman Bot over 3 years ago

  • Fixed in Releases 2.2.3 added
Actions
Copy link
 #3

Updated by Chris Roberts over 3 years ago

  • Status changed from New to Resolved
  • Target version set to 2.2.4
  • Difficulty set to trivial
  • Fixed in Releases 2.2.4 added
  • Fixed in Releases deleted (2.2.3)
Actions
Copy link
 #4

Updated by Chris Roberts over 3 years ago

  • Triaged changed from No to Yes
Actions
Copy link

Also available in: Atom PDF