Skip to content

Athena Migration Script: Add additional migration options#28560

Merged
logand22 merged 4 commits intomasterfrom
logan/athena-migration-updates
Jul 3, 2023
Merged

Athena Migration Script: Add additional migration options#28560
logand22 merged 4 commits intomasterfrom
logan/athena-migration-updates

Conversation

@logand22
Copy link
Copy Markdown
Contributor

@logand22 logand22 commented Jun 30, 2023

This PR adds an additional function to the public interface of the athena migration script:

func MigrateWithAWS(ctx context.Context, cfg Config, awsCfg aws.Config) error {
...
}

That makes it easier to use the migration script as a library as well. This would benefit the cloud team and allow us to use the migration script as a basis for a more cloud specific migration script.

Comment thread examples/dynamoathenamigration/migration.go Outdated
logand22 and others added 2 commits July 3, 2023 09:35
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from hugoShaka July 3, 2023 17:52
@logand22 logand22 added this pull request to the merge queue Jul 3, 2023
@logand22 logand22 removed this pull request from the merge queue due to a manual request Jul 3, 2023
@logand22 logand22 added this pull request to the merge queue Jul 3, 2023
Merged via the queue into master with commit 9bc3c7a Jul 3, 2023
@logand22 logand22 deleted the logan/athena-migration-updates branch July 3, 2023 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants