Skip to content

Remove the aws_kms_multi_region_read_enabled flag#9187

Merged
jmhooper merged 4 commits intomainfrom
jmhooper-remove-kms-mr-read-flag
Sep 11, 2023
Merged

Remove the aws_kms_multi_region_read_enabled flag#9187
jmhooper merged 4 commits intomainfrom
jmhooper-remove-kms-mr-read-flag

Conversation

@jmhooper
Copy link
Contributor

@jmhooper jmhooper commented Sep 11, 2023

The aws_kms_multi_region_read_enabled was in place to toggle the changes introduced in #9047. Those changes have been deployed, enabled, and have been running without issue for some time.

This commit removes the tooling to toggle the feature since we will be using it going forward.

The `aws_kms_multi_region_read_enabled` was in place to toggle the changes introduced in #9047. Those changes have been deployed, enabled, and have been running without issue for some time.

This commit removes the tooling to toggle the feature since we will be using it going forward.

[skip changelog]
@jmhooper jmhooper requested a review from a team September 11, 2023 14:00
Copy link
Contributor

@soniaconnolly soniaconnolly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jmhooper jmhooper merged commit 995250d into main Sep 11, 2023
@jmhooper jmhooper deleted the jmhooper-remove-kms-mr-read-flag branch September 11, 2023 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants