Skip to content

Commit

Permalink
Remove outdated comments
Browse files Browse the repository at this point in the history
  • Loading branch information
jskinne3 committed Jul 26, 2024
1 parent 883b39c commit 0c1f824
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 25 deletions.
6 changes: 0 additions & 6 deletions terraform/demo/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -34,12 +34,6 @@ module "logo_upload_bucket" {
name = "${local.app_name}-logo-upload-bucket-${local.env}"
}

# ##########################################################################
# The following lines need to be commented out for the initial `terraform apply`
# It can be re-enabled after:
# 1) the api app has first been deployed
# 2) the admin app has first been deployed
###########################################################################
module "api_network_route" {
source = "../shared/container_networking"

Expand Down
6 changes: 0 additions & 6 deletions terraform/production/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -34,12 +34,6 @@ module "logo_upload_bucket" {
name = "${local.app_name}-logo-upload-bucket-${local.env}"
}

# ##########################################################################
# The following lines need to be commented out for the initial `terraform apply`
# It can be re-enabled after:
# 1) the api app has first been deployed
# 2) the admin app has first been deployed
###########################################################################
module "api_network_route" {
source = "../shared/container_networking"

Expand Down
8 changes: 1 addition & 7 deletions terraform/sandbox/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -34,13 +34,7 @@ module "logo_upload_bucket" {
name = "${local.app_name}-logo-upload-bucket-${local.env}"
}

# ##########################################################################
# Connection the allows notify-admin-sandbox and notify-api-sandbox to talk.
# https://cloud.gov/docs/management/container-to-container/
# Terraform block will fail to apply unless both apps exist in Cloud.gov.
# See also: /docs/all.md#api-request-failed
###########################################################################
module "api_network_route" {
module "api_network_route" { # API and Admin apps must both exist in Cloud
source = "../shared/container_networking"

cf_org_name = local.cf_org_name
Expand Down
6 changes: 0 additions & 6 deletions terraform/staging/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -34,12 +34,6 @@ module "logo_upload_bucket" {
name = "${local.app_name}-logo-upload-bucket-${local.env}"
}

# ##########################################################################
# The following lines need to be commented out for the initial `terraform apply`
# It can be re-enabled after:
# 1) the api app has first been deployed
# 2) the admin app has first been deployed
###########################################################################
module "api_network_route" {
source = "../shared/container_networking"

Expand Down

0 comments on commit 0c1f824

Please sign in to comment.