Skip to content

LG-10344 Add a rake task for detecting mismatch multi-region and single-region KMS records#9058

Merged
jmhooper merged 1 commit intomainfrom
jmhooper-mismatched-record-detection
Aug 23, 2023
Merged

LG-10344 Add a rake task for detecting mismatch multi-region and single-region KMS records#9058
jmhooper merged 1 commit intomainfrom
jmhooper-mismatched-record-detection

Conversation

@jmhooper
Copy link
Contributor

We are in the midst of a migration between KMS keys. We are changing from a single-region only key to a key that supports multi-region use.

As part of this migration we are encrypting data twice: once with each of the keys. Under no circumstance should the data that is encrypted by KMS and stored be different for a given record.

This commit introduces a rake task that confirms that the same data is being encrypted and written. It does so by grabbing a sample of records, removing the KMS encryption, and comparing the contents.

@jmhooper jmhooper requested a review from a team August 23, 2023 15:31
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.

…le-region KMS records

We are in the midst of a migration between KMS keys. We are changing from a single-region only key to a key that supports multi-region use.

As part of this migration we are encrypting data twice: once with each of the keys. Under no circumstance should the data that is encrypted by KMS and stored be different for a given record.

This commit introduces a rake task that confirms that the same data is being encrypted and written. It does so by grabbing a sample of records, removing the KMS encryption, and comparing the contents.

[skip changelog]
@jmhooper jmhooper force-pushed the jmhooper-mismatched-record-detection branch from 460c01c to d2f340e Compare August 23, 2023 17:17
@jmhooper jmhooper merged commit 8c9f9b0 into main Aug 23, 2023
@jmhooper jmhooper deleted the jmhooper-mismatched-record-detection branch August 23, 2023 17:34
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