darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-03 09:51:11 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1221 from GSA/jskinne3-remove-lingering-var

Browse Source 
Cleanup: remove comment unused local var
This commit is contained in:
Carlo Costino
2024-07-29 14:30:42 -04:00
committed by GitHub
parent 1df7eab985 4de0965426
commit 678a9ea872
2 changed files with 0 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
terraform/development/main.tf
View File

@@ -1,7 +1,6 @@
locals { locals {
cf_org_name = "gsa-tts-benefits-studio" cf_org_name = "gsa-tts-benefits-studio"
cf_space_name = "notify-local-dev" cf_space_name = "notify-local-dev"
recursive_delete = true
key_name = "${var.username}-api-dev-key" key_name = "${var.username}-api-dev-key"
} }

20
terraform/production/main.tf
View File

@@ -74,23 +74,3 @@ module "sns_sms" {
aws_region = "us-gov-west-1" aws_region = "us-gov-west-1"
monthly_spend_limit = 1000 monthly_spend_limit = 1000
} }
###########################################################################
# The following lines need to be commented out for the initial `terraform apply`
# It can be re-enabled after:
# TODO: decide on public API domain name
# 1) the app has first been deployed
# 2) the route has been manually created by an OrgManager:
# `cf create-domain gsa-tts-benefits-studio api.notify.gov`
###########################################################################
# module "domain" {
# source = "github.com/18f/terraform-cloudgov//domain?ref=v0.7.1"
#
# cf_org_name = local.cf_org_name
# cf_space_name = local.cf_space_name
# app_name_or_id = "${local.app_name}-${local.env}"
# name = "${local.app_name}-domain-${local.env}"
# recursive_delete = local.recursive_delete
# cdn_plan_name = "domain"
# domain_name = "api.notify.gov"
# }