Remove forms-api task from review apps

Forms-runner now makes requests to forms-admin rather than forms-api
to get forms. Update the review apps so that the forms-api container
is no longer created and forms-runner makes requests to forms-admin
rather than forms-api.

We need to also set the `ALLOWED_HOST_PATTERNS` environment variable
for the forms-admin container so that it allows requests from
forms-runner to localhost:3000 to make requests to the forms-admin
API without authorization being required.

Author: stephencdaly

.review_apps/README.md

@@ -5,7 +5,6 @@ The Terraform code in this directory is used to deploy a review copy of `forms-r
 It constructs a minimal, ephemeral version of a GOV.UK Forms environment in AWS ECS that can be used for reviews, then freely destroyed. This includes: 
 
 * a copy of `forms-runner` at the commit in question
-* a copy of the version of `forms-api` currently in production
 * a copy of the version of `forms-admin` currently in production
 * a local PostgreSQL database with seed data for both `forms-api` and `forms-admin`
 * a local Redis instance of session storage

.review_apps/ecs_task_definition.tf

@@ -22,7 +22,7 @@ locals {
     { name = "SETTINGS__ANALYTICS_ENABLED", value = "false" },
     { name = "SETTINGS__CLOUDWATCH_METRICS_ENABLED", value = "false" },
     { name = "SETTINGS__FORMS_ADMIN__BASE_URL", value = "https://${local.admin_app_hostname}" },
-    { name = "SETTINGS__FORMS_API__BASE_URL", value = "http://localhost:9292" },
+    { name = "SETTINGS__FORMS_API__BASE_URL", value = "http://localhost:3000" },
     { name = "SETTINGS__FORMS_ENV", value = "review" },
 
     ##
@@ -33,15 +33,6 @@ locals {
     ##
   ]
 
-  forms_api_env_vars = [
-    { name = "DATABASE_URL", value = "postgres://postgres:postgres@127.0.0.1:5432" },
-    { name = "EMAIL", value = "review-app-submissions@review.forms.service.gov.uk" },
-    { name = "RAILS_DEVELOPMENT_HOSTS", value = "localhost:9292" },
-    { name = "RAILS_ENV", value = "production" },
-    { name = "SECRET_KEY_BASE", value = "unsecured_secret_key_material" },
-    { name = "SETTINGS__FORMS_ENV", value = "review" },
-  ]
-
   forms_admin_env_vars = [
     { name = "DATABASE_URL", value = "postgres://postgres:postgres@127.0.0.1:5432" },
     { name = "GOVUK_APP_DOMAIN", value = "publishing.service.gov.uk" },
@@ -52,10 +43,9 @@ locals {
     { name = "SECRET_KEY_BASE", value = "unsecured_secret_key_material" },
     { name = "SETTINGS__ACT_AS_USER_ENABLED", value = "true" },
     { name = "SETTINGS__AUTH_PROVIDER", value = "developer" },
-    { name = "SETTINGS__FORMS_API__AUTH_KEY", value = "unsecured_api_key_for_review_apps_only" },
-    { name = "SETTINGS__FORMS_API__BASE_URL", value = "http://localhost:9292" },
     { name = "SETTINGS__FORMS_ENV", value = "review" },
     { name = "SETTINGS__FORMS_RUNNER__URL", value = "https://${local.runner_review_app_hostname}" },
+    { name = "ALLOWED_HOST_PATTERNS", value = "localhost:3000" }
   ]
 }
 
@@ -138,45 +128,6 @@ resource "aws_ecs_task_definition" "task" {
       ]
     },
 
-    # forms-api
-    {
-      name                   = "forms-api"
-      image                  = "711966560482.dkr.ecr.eu-west-2.amazonaws.com/forms-api-deploy:latest"
-      command                = []
-      essential              = true
-      environment            = local.forms_api_env_vars
-      readonlyRootFilesystem = true
-
-      portMappings = [{ containerPort = 9292 }]
-
-      logConfiguration = {
-        logDriver = "awslogs"
-        options = {
-          awslogs-group         = data.terraform_remote_state.review.outputs.review_apps_log_group_name
-          awslogs-region        = "eu-west-2"
-          awslogs-stream-prefix = "${local.logs_stream_prefix}/forms-api"
-        }
-      }
-
-      healthCheck = {
-        command     = ["CMD-SHELL", "wget -O - 'http://localhost:9292/up' || exit 1"]
-        interval    = 30
-        retries     = 5
-        startPeriod = 180
-      }
-
-      dependsOn = [
-        {
-          containerName = "postgres"
-          condition     = "HEALTHY"
-        },
-        {
-          containerName = "forms-api-seeding",
-          condition     = "SUCCESS"
-        }
-      ]
-    },
-
     # forms-admin
     {
       name                   = "forms-admin"
@@ -192,9 +143,9 @@ resource "aws_ecs_task_definition" "task" {
         "traefik.http.routers.forms-runner-pr-${var.pull_request_number}-admin-app.rule" : "Host(`${local.admin_app_hostname}`)",
         "traefik.http.routers.forms-runner-pr-${var.pull_request_number}-admin-app.service" : "forms-runner-pr-${var.pull_request_number}-admin-app",
         "traefik.http.routers.forms-runner-pr-${var.pull_request_number}-admin-app.middlewares" : "forms-runner-pr-${var.pull_request_number}-admin-app@ecs"
-
         "traefik.http.services.forms-runner-pr-${var.pull_request_number}-admin-app.loadbalancer.server.port" : "3000",
         "traefik.http.services.forms-runner-pr-${var.pull_request_number}-admin-app.loadbalancer.healthcheck.path" : "/up",
+
         "traefik.enable" : "true",
       },
 
@@ -315,32 +266,6 @@ resource "aws_ecs_task_definition" "task" {
       ]
     },
 
-    # forms-api-seeding
-    {
-      name                   = "forms-api-seeding"
-      image                  = "711966560482.dkr.ecr.eu-west-2.amazonaws.com/forms-api-deploy:latest"
-      command                = ["rake", "db:setup"]
-      essential              = false
-      environment            = local.forms_api_env_vars
-      readonlyRootFilesystem = true
-
-      logConfiguration = {
-        logDriver = "awslogs"
-        options = {
-          awslogs-group         = data.terraform_remote_state.review.outputs.review_apps_log_group_name
-          awslogs-region        = "eu-west-2"
-          awslogs-stream-prefix = "${local.logs_stream_prefix}/forms-api-seeding"
-        }
-      }
-
-      dependsOn = [
-        {
-          containerName = "postgres"
-          condition     = "HEALTHY"
-        }
-      ]
-    },
-
     # forms-admin-seeding
     {
       name                   = "forms-admin-seeding"