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

DPR2-893: Glue connection and placeholder operational datastore secret #6670

Merged
merged 11 commits into from
Jun 20, 2024

Conversation

tom-ogle-moj
Copy link
Contributor

@tom-ogle-moj tom-ogle-moj commented Jun 19, 2024

  • Limit created resources to Dev/Sandbox only
  • Create a Glue JDBC Data Connection for attachment to DataHub Glue jobs to give them Operational Datastore access
  • Temporarily populate it with the manually created Dev/Sandbox RDS details which will be replaced by the actual Operational Datastore details
  • Create a secret for the credentials used by the DataHub Glue jobs to write to the Operational Datastore
  • Create a Security Group associated with the Glue JDBC Data Connection
    • it will be associated with a glue job automatically when the connection is attached to it
    • this is a requirement of Glue and it allows Glue to communicate with itself by allowing TCP ingress on all ports from the security group. See https://docs.aws.amazon.com/glue/latest/dg/setup-vpc-for-glue-access.html
    • the same security group is attached to the RDS as well as the glue job so the same ingress rule also allows access to the RDS instance. Instead, a separate security group could be attached to the RDS allowing Ingress from this security group.
    • it allows egress on all ports which allows it to communicate with services such as S3 without having an S3 service endpoint in the VPC.
  • I've tested this by attaching the connection to the activities glue batch job, manually giving the job access to read the secret and verifying it can write to the RDS database using the connection and secret provided here

@github-actions github-actions bot added the environments-repository Used to exclude PRs from this repo in our Slack PR update label Jun 19, 2024
@tom-ogle-moj tom-ogle-moj had a problem deploying to digital-prison-reporting-test June 20, 2024 10:00 — with GitHub Actions Error
@tom-ogle-moj tom-ogle-moj had a problem deploying to digital-prison-reporting-development June 20, 2024 10:00 — with GitHub Actions Error
@tom-ogle-moj tom-ogle-moj had a problem deploying to digital-prison-reporting-development June 20, 2024 10:18 — with GitHub Actions Failure
@tom-ogle-moj tom-ogle-moj had a problem deploying to digital-prison-reporting-test June 20, 2024 10:19 — with GitHub Actions Error
@tom-ogle-moj tom-ogle-moj temporarily deployed to digital-prison-reporting-development June 20, 2024 10:27 — with GitHub Actions Inactive
@tom-ogle-moj tom-ogle-moj temporarily deployed to digital-prison-reporting-development June 20, 2024 11:46 — with GitHub Actions Inactive
@tom-ogle-moj tom-ogle-moj temporarily deployed to digital-prison-reporting-development June 20, 2024 12:07 — with GitHub Actions Inactive
@tom-ogle-moj tom-ogle-moj had a problem deploying to digital-prison-reporting-development June 20, 2024 13:51 — with GitHub Actions Error
…elated resources to development environment only for now.
@tom-ogle-moj tom-ogle-moj temporarily deployed to digital-prison-reporting-development June 20, 2024 13:59 — with GitHub Actions Inactive
@tom-ogle-moj tom-ogle-moj changed the title DPR2-893: Glue connection and placeholder operational datastore secre… DPR2-893: Glue connection and placeholder operational datastore secret Jun 20, 2024
@tom-ogle-moj tom-ogle-moj temporarily deployed to digital-prison-reporting-test June 20, 2024 14:21 — with GitHub Actions Inactive
@tom-ogle-moj tom-ogle-moj marked this pull request as ready for review June 20, 2024 14:31
@tom-ogle-moj tom-ogle-moj requested review from a team as code owners June 20, 2024 14:31
Copy link
Contributor

@gavriil-g-moj gavriil-g-moj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, not sure if the static analysis failure is an issue.

@tom-ogle-moj
Copy link
Contributor Author

LGTM, not sure if the static analysis failure is an issue.

It seems that it fails for our project continuously for every PR

Copy link
Contributor

@harichintala1 harichintala1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@tom-ogle-moj tom-ogle-moj merged commit b7477ea into main Jun 20, 2024
13 of 15 checks passed
@tom-ogle-moj tom-ogle-moj deleted the DPR2-893-glue-jdbc-connection branch June 20, 2024 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
environments-repository Used to exclude PRs from this repo in our Slack PR update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants