Skip to content

LG-10891 Add MultiRegionKmsMigration::UserMigrationJob to the background job schedule#9240

Merged
jmhooper merged 6 commits intomainfrom
jmhooper-configure-user-mr-kms-job
Sep 20, 2023
Merged

LG-10891 Add MultiRegionKmsMigration::UserMigrationJob to the background job schedule#9240
jmhooper merged 6 commits intomainfrom
jmhooper-configure-user-mr-kms-job

Conversation

@jmhooper
Copy link
Contributor

A previous commit added a class for re-encrypting single-region KMS ciphertexts on user records with a multi-region key. That job was not added to the job schedule so it could be tested in lower environments before being enabled.

We have confirmed the job works as expected in lower environments.

This commit also addresses LG-11030 by making the record count and statement timeout params for the migration jobs configurable with the IdentityConfig tooling.

…und job schedule

A previous commit added a class for re-encrypting single-region KMS ciphertexts on user records with a multi-region key. That job was not added to the job schedule so it could be tested in lower environments before being enabled.

We have confirmed the job works as expected in lower environments.

This commit also addresses LG-11030 by making the record count and statement timeout params for the migration jobs configurable with the IdentityConfig tooling.

[skip changelog]
@jmhooper jmhooper requested review from a team and mitchellhenke September 20, 2023 13:24
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 and others added 3 commits September 20, 2023 15:41
Co-authored-by: Sonia Connolly <sonia.connolly@gsa.gov>
Co-authored-by: Sonia Connolly <sonia.connolly@gsa.gov>
Co-authored-by: Sonia Connolly <sonia.connolly@gsa.gov>
@jmhooper jmhooper merged commit b1b4873 into main Sep 20, 2023
@jmhooper jmhooper deleted the jmhooper-configure-user-mr-kms-job branch September 20, 2023 19:59
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