Skip to content

[v17] keystore: add support for aws kms multi-region key replication (#53927)#55212

Merged
dboslee merged 3 commits intobranch/v17from
david/v17/backport-53927
Jun 10, 2025
Merged

[v17] keystore: add support for aws kms multi-region key replication (#53927)#55212
dboslee merged 3 commits intobranch/v17from
david/v17/backport-53927

Conversation

@dboslee
Copy link
Copy Markdown
Contributor

@dboslee dboslee commented May 28, 2025

Backport #53927 to branch/v17

changelog: Added support for AWS KMS multi-region keys with key replication

* keystore: add support for aws kms multi-region key replication

* update func name ApplyConfig -> ApplyMultiRegionConfig

* more descriptive var out -> describeKeyOut

* fix typo

* better var names

* add comment

* renaming vars for readability

* refactor multi-region auth config

* add comment about cert authority lock

* fix typo

* move funcs up

* update comment

* copy whole struct instead of individual values
@github-actions github-actions Bot requested review from eriktate and nklaassen May 28, 2025 13:28
@dboslee dboslee changed the title keystore: add support for aws kms multi-region key replication (#53927) [v17] keystore: add support for aws kms multi-region key replication (#53927) May 28, 2025
@dboslee
Copy link
Copy Markdown
Contributor Author

dboslee commented Jun 10, 2025

FYI pulled in #55274 which added some retry logic around one of the calls which was failing somewhat consistently due to key state/ async replication

@dboslee dboslee enabled auto-merge June 10, 2025 13:49
@dboslee dboslee added this pull request to the merge queue Jun 10, 2025
Merged via the queue into branch/v17 with commit 9a527c1 Jun 10, 2025
38 checks passed
@dboslee dboslee deleted the david/v17/backport-53927 branch June 10, 2025 14:32
@fheinecke fheinecke mentioned this pull request Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants