var.admin_username is required, var.admin_ssh_keys.*.username is optional and in case its present should be equal to var.admin_username

[juanjojulian]

var.admin_username is required, var.admin_ssh_keys.*.username is optional and in case its present should be equal to var.admin_username

Describe your changes

• var.admin_username is required and non nullable.
• var.admin_ssh_keys.*.username is optional and in case its present should be equal to var.admin_username

Issue number

#000

Checklist before requesting a review

• [ X ] The pr title can be used to describe what this pr did in CHANGELOG.md file
• [ X ] I have executed pre-commit on my machine
• [ X ] I have passed pr-check on my machine

Thanks for your cooperation!

[github-actions]

Potential Breaking Changes in 85a8a19:
[update] "Variables.admin_ssh_keys.Type" from 'set(object({
public_key = string
username = string
}))' to 'set(object({
public_key = string
username = optional(string)
}))'
[update] "Variables.admin_username.Default" from '"azureuser"' to ''
[update] "Variables.admin_username.Nullable" from '' to 'false'

[lonegunmanb]

Thanks @juanjojulian for opening this pr! Some tiny issues that I'd like to solve, WDYT?

[github-actions]

Potential Breaking Changes in 4f67050:
[update] "Variables.admin_ssh_keys.Type" from 'set(object({
public_key = string
username = string
}))' to 'set(object({
public_key = string
username = optional(string)
}))'
[update] "Variables.admin_username.Nullable" from '' to 'false'

[lonegunmanb]

Thanks @juanjojulian for the update, please add var.admin_username to outputs as you suggested.

[github-actions]

Potential Breaking Changes in 0defb81:
[update] "Variables.admin_ssh_keys.Type" from 'set(object({
public_key = string
username = string
}))' to 'set(object({
public_key = string
username = optional(string)
}))'
[update] "Variables.admin_username.Nullable" from '' to 'false'

[lonegunmanb]

Thanks @juanjojulian, LGTM! 🚀