[services][identity] add cargo run subcommand for populating DDB from MySQL
ClosedPublic
Actions

Authored by varun on Oct 19 2022, 3:22 PM.

Details

Reviewers

tomek
atul
• jon

Commits

rCOMM59f83eb68431: [services][identity] add cargo run subcommand for populating DDB from MySQL

Summary

We want to make sure that all the relevant existing user data is moved over before we begin double-writing user data to DynamoDB.
The script to handle moving this data will be a subcommand of the Identity service, because that lets us re-use some of the DynamoDB client code.

Test Plan

cargo run in the identity service directory displays the new subcommand

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Oct 19 2022, 3:22 PM

Herald added subscribers: • abosh, ashoat. · View Herald TranscriptOct 19 2022, 3:22 PM

Harbormaster completed remote builds in B12878: Diff 17669.Oct 19 2022, 3:34 PM

varun requested review of this revision.Oct 19 2022, 3:34 PM

ashoat added inline comments.Oct 20 2022, 5:16 AM

services/identity/.gitignore
1 ↗	(On Diff #17669)	I'm confused, wasn't this handled in D5293? If we're moving the `identity` `.gitignore` here, let's dedup it from the main `.gitignore`, and probably move the `services/identity/target` ignore here too

tomek accepted this revision.Oct 20 2022, 5:18 AM

This revision is now accepted and ready to land.Oct 20 2022, 5:18 AM

varun added inline comments.Oct 20 2022, 9:52 AM

services/identity/.gitignore
1 ↗	(On Diff #17669)	hm yeah you're right. i'm not sure why it was still showing up in the untracked files section for me... in any case, I think it makes sense to move `secrets` and `target` to an identity-specific `.gitignore`. i'll dedupe