Skip to content

Commit fe84810

Browse files
authored
refactor: update folder hierarchy and top level folder (#14)
1 parent 1788c8d commit fe84810

File tree

7 files changed

+30
-39
lines changed

7 files changed

+30
-39
lines changed

3-projects/single-project-example.tf renamed to 3-projects/example_single_project.tf

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,20 +14,15 @@
1414
* limitations under the License.
1515
*/
1616

17-
resource "google_folder" "single_project_folder" {
18-
parent = google_folder.app.id
19-
display_name = "single-project-app"
20-
}
21-
22-
module "single_project_app" {
17+
module "example_single_project" {
2318
source = "./modules/single_project"
2419

2520
org_id = var.org_id
2621
billing_account = var.billing_account
2722
impersonate_service_account = var.terraform_service_account
2823
environment = "prod"
2924

30-
folder_id = google_folder.single_project_folder.id
25+
folder_id = module.example_team_folders.prod_folder_id
3126

3227
# Metadata
3328
project_prefix = "sample-single"

3-projects/single-project-example-optional.tf renamed to 3-projects/example_single_project_optional.tf

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,18 +14,13 @@
1414
* limitations under the License.
1515
*/
1616

17-
# resource "google_folder" "single_project_folder_optional" {
18-
# parent = google_folder.app.id
19-
# display_name = "single-project-app-optional"
20-
# }
21-
22-
# module "single_project_app_optional" {
17+
# module "example_single_project_optional" {
2318
# source = "./modules/single_project"
2419

25-
# org_id = var.organization_id
20+
# org_id = var.org_id
2621
# billing_account = var.billing_account
2722
# impersonate_service_account = var.terraform_service_account
28-
# folder_id = google_folder.single_project_folder_optional.id
23+
# folder_id = module.example_team_folders.prod_folder_id
2924
# environment = "prod"
3025

3126
# # Metadata

3-projects/standard-project-example-optional.tf renamed to 3-projects/example_standard_project_optional.tf

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -14,21 +14,15 @@
1414
* limitations under the License.
1515
*/
1616

17-
# module "standard_project_folders_optional" {
18-
# source = "./modules/folder_environments"
19-
# parent_folder_id = google_folder.app.id
20-
# folder_display_name = "standard_app_example_optional"
21-
# }
22-
23-
# module "standard_project_app_optional" {
17+
# module "example_standard_projects_optional" {
2418
# source = "./modules/standard_projects"
2519

26-
# org_id = var.organization_id
20+
# org_id = var.org_id
2721
# billing_account = var.billing_account
2822
# impersonate_service_account = var.terraform_service_account
2923

30-
# nonprod_folder_id = module.standard_project_folders_optional.nonprod_folder_id
31-
# prod_folder_id = module.standard_project_folders_optional.prod_folder_id
24+
# nonprod_folder_id = module.example_team_folders.nonprod_folder_id
25+
# prod_folder_id = module.example_team_folders.prod_folder_id
3226

3327
# # Metadata
3428
# project_prefix = "standard-optional"

3-projects/standard-project-example.tf renamed to 3-projects/example_standard_projects.tf

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -14,21 +14,15 @@
1414
* limitations under the License.
1515
*/
1616

17-
module "standard_project_folders" {
18-
source = "./modules/folder_environments"
19-
parent_folder_id = google_folder.app.id
20-
folder_display_name = "standard_app_example"
21-
}
22-
23-
module "standard_project_app" {
17+
module "example_standard_projects" {
2418
source = "./modules/standard_projects"
2519

2620
org_id = var.org_id
2721
billing_account = var.billing_account
2822
impersonate_service_account = var.terraform_service_account
2923

30-
nonprod_folder_id = module.standard_project_folders.nonprod_folder_id
31-
prod_folder_id = module.standard_project_folders.prod_folder_id
24+
nonprod_folder_id = module.example_team_folders.nonprod_folder_id
25+
prod_folder_id = module.example_team_folders.prod_folder_id
3226

3327
# Metadata
3428
project_prefix = "sample-standard"

3-projects/main.tf renamed to 3-projects/folders.tf

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,19 @@
1414
* limitations under the License.
1515
*/
1616

17-
resource "google_folder" "app" {
17+
/******************************************
18+
Business Unit Folders
19+
*****************************************/
20+
resource "google_folder" "example_business_unit" {
21+
display_name = "example-business-unit"
1822
parent = "organizations/${var.org_id}"
19-
display_name = "app"
23+
}
24+
25+
/******************************************
26+
Team Folders
27+
*****************************************/
28+
module "example_team_folders" {
29+
source = "./modules/folder_environments"
30+
parent_folder_id = google_folder.example_business_unit.name
31+
folder_display_name = "example-team"
2032
}

3-projects/modules/single_project/variables.tf

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ variable "environment" {
6666
# variable "enable_networking" {
6767
# description = "The flag to create subnets in shared VPC"
6868
# type = bool
69-
# default = true
69+
# default = false
7070
# }
7171

7272
# variable "subnet_ip_cidr_range" {
@@ -90,10 +90,11 @@ variable "environment" {
9090
# variable "enable_private_dns" {
9191
# type = bool
9292
# description = "The flag to create private dns zone in shared VPC"
93-
# default = true
93+
# default = false
9494
# }
9595

9696
# variable "domain" {
9797
# type = string
9898
# description = "The top level domain name for the organization"
99+
# default = ""
99100
# }

3-projects/modules/standard_projects/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ variable "activate_apis" {
105105
# variable "enable_private_dns" {
106106
# type = bool
107107
# description = "The flag to create private dns zone in shared VPC"
108-
# default = true
108+
# default = false
109109
# }
110110

111111
# variable "domain" {

0 commit comments

Comments
 (0)