Skip to content

Deploy via Cloud run - #664

Open
Sparrow0hawk wants to merge 13 commits into
mainfrom
terraform
Open

Deploy via Cloud run#664
Sparrow0hawk wants to merge 13 commits into
mainfrom
terraform

Conversation

@Sparrow0hawk

Copy link
Copy Markdown
Contributor

This PR introduces terraform to deploy the service via Cloud Run as opposed to app engine. It does not yet support IAP for the internal instance or include a load balancer/WAF for the public instance.

It introduces the following for the dft-rlg-atip-dev environment only:

  • A cloud storage bucket for terraform state (should be moved to a common project)
  • Imports the existing cloud storage bucket for client side geospatial data files
  • Introduces an artifact registry for docker images (should be moved to a common project)
  • Introduces a cloud run service that pulls the latest Docker image from the artifact registry and can proxy requests to the cloud storage bucket
  • Migrates the playwright.yml workflow to a ci.yml workflow
  • Introduces a deploy.yml workflow that can deploy new cloud run revisions
  • Configure GitHub environments to include credentials for the Dev environment so that CI will automatically deploy a new Docker image after each successful build

This is a reasonable increment to bring into main before addressing IAP.

@Sparrow0hawk
Sparrow0hawk requested a review from Pete-Y-CS June 26, 2026 13:10
Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/ci.yml Fixed
This bucket is used for data accessible to plan your active travel
schemes.
@Sparrow0hawk
Sparrow0hawk force-pushed the terraform branch 2 times, most recently from 8536008 to 2a3fb38 Compare June 26, 2026 14:33
Approximate like-for-like set up to App Engine without IAP.
Using the dev project whilst we lack a common project.
Imports existing cloud storage bucket for storing layers and
route-snapper files. Configuration differs from browse page with
multi-region bucket location and enforced public access policy.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants