Project

General

Profile

Actions

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:
Category:
Foreman integration
Target version:
Difficulty:
trivial
Triaged:
Yes
Fixed in Releases:
Found in Releases:

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

Also available in: Atom PDF