Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for CMK on Geo-backup enabled servers on Flexible Server for PostgreSQL #20750

Closed
1 task done
korsveien opened this issue Mar 2, 2023 · 6 comments · Fixed by #22612
Closed
1 task done

Support for CMK on Geo-backup enabled servers on Flexible Server for PostgreSQL #20750

korsveien opened this issue Mar 2, 2023 · 6 comments · Fixed by #22612

Comments

@korsveien
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

Creating a Flexible Server for PostgreSQL where both CMK and Geo-redundant backup is enabled will result in the following error:

Code="CMKNotSupportedForGeoRedundantBackupEnabledServer" Message="Customer Managed Key encryption is not supported for Geo redundant backup enabled server."

If this is to be supported there needs to be a way to define a secondary key id and identity that is used to encrypt the Geo-redundant backup in the paired region.

New or Affected Resource(s)/Data Source(s)

azurerm_postgresql_flexible_server

Potential Terraform Configuration

resource "azurerm_postgresql_flexible_server" "postgres_server" {

...

customer_managed_key {

    // primary
    primary_key_vault_key_id          = azurerm_key_vault_key.primary.id
    primary_user_assigned_identity_id = azurerm_user_assigned_identity.primary.id

    // secondary (this or some version of this needs to be added)
    secondary_key_vault_key_id          = azurerm_key_vault_key.secondary.id
    secondary_user_assigned_identity_id = azurerm_user_assigned_identity.secondary.id
 }

 geo_redundant_backup_enabled = true

...

}

References

Similar issue for mysql: #20408

@neil-yechenwei
Copy link
Contributor

Thanks for raising this issue. Service team confirmed this feature hasn't been supported by Azure Rest API. But it will be released to next API version. After that, we could take another look.

BTW, I filed an issue on azure rest api spec for tracking.

@martinshkrelijerry
Copy link

@neil-yechenwei what about the MySQL issue #20408? Will this also be dealt with next API version?

@Danielchui
Copy link

@neil-yechenwei any update on this issues? Any timeline for the fix? Thanks.

@patverderrie
Copy link

Any update on this open issue would be appreciated!

@baohuynh09-lxp
Copy link

baohuynh09-lxp commented Jul 6, 2023

any update on this ? we really need this API to migrate from single server -> flexible server with Terraform.

Copy link

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
7 participants