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.

Sign up for GitHub

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

Jump to bottom

λ: Migrate JML Lambda to analytical-platform-data-production #6517

Open
Gary-H9 opened this issue Jan 7, 2025 · 3 comments
Open

λ: Migrate JML Lambda to analytical-platform-data-production #6517

Gary-H9 opened this issue Jan 7, 2025 · 3 comments
Assignees
@Gary-H9
Labels
story

Comments

@Gary-H9
Copy link
Contributor

Gary-H9 commented Jan 7, 2025 •
edited
Loading

The purpose of this story is to migrate the lambda function:

  • data_platform_jml_extract in the data-platform-apps-and-tools-production AWS account
  • to the Analytical Platform Compute account analytical-platform-compute-production .
  • to the Analytical Platform Data Production account analytical-platform-data-production .

The code for this lives in the modernisation-platform-environments repository. For example here.

Notes:

  • GOVUK Notify API key and JML email address stored in AWS Secrets Manager
  • Triggered by EventBridge

Approach:

  • One PR to add Lambda to APDP Account
  • One PR to remove from Apps and Tools Production
@Gary-H9 Gary-H9 added the story label Jan 7, 2025
@Gary-H9 Gary-H9 mentioned this issue Jan 7, 2025
Open
@Gary-H9
Copy link
Contributor Author

Gary-H9 commented Jan 7, 2025

Created a quick PR to remove the Lambda function. Will review tomorrow to make sure nothing is missed.

@Gary-H9 Gary-H9 mentioned this issue Jan 7, 2025
Closed
@Gary-H9
Copy link
Contributor Author

Gary-H9 commented Jan 8, 2025

Turns out this is a duplicate of this ticket. 🤕 Editing this one appropriately as I've been working in here before closing the previous one - sorry Jacob!

@Gary-H9 Gary-H9 changed the title ✉ Migrate JML Lambda to AP Compute Account λ: Migrate JML Lambda to AP Compute Account Jan 8, 2025
@julialawrence julialawrence moved this from 👀 TODO to 🚀 In Progress in Analytical Platform Jan 8, 2025
@Gary-H9 Gary-H9 changed the title λ: Migrate JML Lambda to AP Compute Account λ: Migrate JML Lambda to analytical-platform-data-production Jan 8, 2025
This was referenced Jan 8, 2025
Draft
Merged
@Gary-H9
Copy link
Contributor Author

Gary-H9 commented Jan 8, 2025 •
edited
Loading

PR to add the correct permissions to the ECR repo. PR MERGED.

TODO: Another PR later to remove the unneeded ones.

This was referenced Jan 9, 2025
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Gary-H9 Gary-H9

Labels
story
Projects
Analytical Platform
Status: 🚀 In Progress
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Gary-H9