darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-09-09 09:19:48 -04:00
Code Issues Packages Projects Releases Wiki Activity

Remove outdated comments

Browse Source
This commit is contained in:
John Skiles Skinner
2024-07-25 19:21:20 -07:00
parent 883b39cd93
commit 0c1f824ca3
4 changed files with 1 additions and 25 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
terraform/demo/main.tf
View File

@@ -34,12 +34,6 @@ module "logo_upload_bucket" {
name = "${local.app_name}-logo-upload-bucket-${local.env}" 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" { module "api_network_route" {
source = "../shared/container_networking" source = "../shared/container_networking"

6
terraform/production/main.tf
View File

@@ -34,12 +34,6 @@ module "logo_upload_bucket" {
name = "${local.app_name}-logo-upload-bucket-${local.env}" 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" { module "api_network_route" {
source = "../shared/container_networking" source = "../shared/container_networking"

8
terraform/sandbox/main.tf
View File

@@ -34,13 +34,7 @@ module "logo_upload_bucket" {
name = "${local.app_name}-logo-upload-bucket-${local.env}" name = "${local.app_name}-logo-upload-bucket-${local.env}"
} }
# ########################################################################## module "api_network_route" { # API and Admin apps must both exist in Cloud
# 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" {
source = "../shared/container_networking" source = "../shared/container_networking"
cf_org_name = local.cf_org_name cf_org_name = local.cf_org_name

6
terraform/staging/main.tf
View File

@@ -34,12 +34,6 @@ module "logo_upload_bucket" {
name = "${local.app_name}-logo-upload-bucket-${local.env}" 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" { module "api_network_route" {
source = "../shared/container_networking" source = "../shared/container_networking"