Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding ETL Infrastructure for plugin data workflow #985

Merged
merged 6 commits into from
May 15, 2023
Merged

Conversation

manasaV3
Copy link
Collaborator

@manasaV3 manasaV3 commented Apr 5, 2023

Description

Relates to #860

Introduces following changes:

  • Invokes the data workflow to refresh plugin data with sqs message
  • Provides data workflow lambda with required permissions for plugin data in dynamo

@manasaV3 manasaV3 requested review from codemonkey800 and klai95 April 5, 2023 20:05
@manasaV3 manasaV3 marked this pull request as ready for review April 5, 2023 20:05
@manasaV3 manasaV3 force-pushed the plugin-etl-infra branch 2 times, most recently from 1f6066b to 9953c5d Compare April 5, 2023 22:44
codemonkey800
codemonkey800 previously approved these changes Apr 6, 2023
klai95
klai95 previously approved these changes Apr 7, 2023
@codemonkey800 codemonkey800 added the new-feature Release Label: Used for categorizing features in automated release notes label Apr 11, 2023
@manasaV3 manasaV3 force-pushed the plugin-db-infra branch from 7296a3b to f454243 Compare May 4, 2023 21:58
@manasaV3 manasaV3 force-pushed the plugin-etl-infra branch from 9953c5d to a086225 Compare May 5, 2023 22:16
@manasaV3 manasaV3 force-pushed the plugin-db-infra branch from f454243 to 757630c Compare May 8, 2023 18:01
@manasaV3 manasaV3 force-pushed the plugin-etl-infra branch from a086225 to e4e7568 Compare May 8, 2023 18:10
@codemonkey800
Copy link
Collaborator

still LGTM, after #972 is merged, this will require another rebase so I'll wait after those two things are complete before approving again 🫡

Base automatically changed from plugin-db-infra to main May 8, 2023 23:54
@manasaV3 manasaV3 dismissed stale reviews from klai95 and codemonkey800 via e4e7568 May 8, 2023 23:54
@manasaV3 manasaV3 force-pushed the plugin-etl-infra branch from e4e7568 to 4b207a0 Compare May 9, 2023 17:05
@manasaV3 manasaV3 merged commit 2fe1743 into main May 15, 2023
@manasaV3 manasaV3 deleted the plugin-etl-infra branch May 15, 2023 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature Release Label: Used for categorizing features in automated release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants