New resource for Entra ID administrator #9
Conversation
There was a problem hiding this comment.
Would you also mind updating the README.md and adding an example to the https://github.com/telekom-mms/terraform-azurerm-database/blob/54b5291d99b50b4345ba0cca0a0579f598f6e7a3/examples/full_main.tf ?
Thanks!
| resource "azurerm_mysql_flexible_server_active_directory_administrator" "mysql_flexible_server_active_directory_administrator" { | ||
| for_each = var.mysql_flexible_server_active_directory_administrator | ||
|
|
||
| server_id = var.mysql_flexible_server_active_directory_administrator[each.key].server_id |
There was a problem hiding this comment.
for the other resources, there were mainly locals used. I think for consistency, this should be adapted
There was a problem hiding this comment.
The README.md will be automatically updated via the GitHub action teraform-docs once the PR has been merged. For a useful example of mysql_flexible_server_active_directory_administrator in full_main.tf, additional resources (e.g. user_managed_identity) are required, which would have to be provided using other modules. I'll see if I can add an example.
For the other resources in this module, default values for optional arguments are set in locals. For this resource, all arguments are necessary for mysql_flexible_server_active_directory_administrator (see https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/mysql_flexible_server_aad_administrator#arguments-reference). That's why I didn't use locals here.
Added the possibility to configure the resource mysql_flexible_server_active_directory_administrator in this module.