Skip to content

Commit

Permalink
feat: updates to support TF 0.13 (#268)
Browse files Browse the repository at this point in the history
  • Loading branch information
bharathkkb committed Mar 31, 2021
1 parent ad3fab2 commit c5c6c6c
Show file tree
Hide file tree
Showing 19 changed files with 37 additions and 36 deletions.
1 change: 1 addition & 0 deletions .kitchen.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
driver:
name: terraform
command_timeout: 2700
verify_version: false

provisioner:
name: terraform
Expand Down
2 changes: 1 addition & 1 deletion 0-bootstrap/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ resource "google_folder" "bootstrap" {

module "seed_bootstrap" {
source = "terraform-google-modules/bootstrap/google"
version = "~> 1.3"
version = "~> 1.5"
org_id = var.org_id
folder_id = google_folder.bootstrap.id
billing_account = var.billing_account
Expand Down
2 changes: 1 addition & 1 deletion 0-bootstrap/modules/jenkins-agent/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ resource "random_id" "suffix" {
*******************************************/
module "cicd_project" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.0"
version = "~> 9.2"
name = local.cicd_project_name
random_project_id = true
disable_services_on_destroy = false
Expand Down
2 changes: 1 addition & 1 deletion 0-bootstrap/modules/jenkins-agent/versions.tf
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
*/

terraform {
required_version = "~> 0.12.6"
required_version = ">=0.13, <0.14"

required_providers {
google = "~> 3.5"
Expand Down
2 changes: 1 addition & 1 deletion 0-bootstrap/versions.tf
Original file line number Diff line number Diff line change
Expand Up @@ -15,5 +15,5 @@
*/

terraform {
required_version = "~> 0.12.6"
required_version = ">=0.13, <0.14"
}
12 changes: 6 additions & 6 deletions 1-org/envs/shared/projects.tf
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@

module "org_audit_logs" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.0"
version = "~> 9.2"
random_project_id = "true"
impersonate_service_account = var.terraform_service_account
default_service_account = "deprivilege"
Expand All @@ -47,7 +47,7 @@ module "org_audit_logs" {

module "org_billing_logs" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.0"
version = "~> 9.2"
random_project_id = "true"
impersonate_service_account = var.terraform_service_account
default_service_account = "deprivilege"
Expand Down Expand Up @@ -78,7 +78,7 @@ module "org_billing_logs" {

module "org_secrets" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.0"
version = "~> 9.2"
random_project_id = "true"
impersonate_service_account = var.terraform_service_account
default_service_account = "deprivilege"
Expand Down Expand Up @@ -109,7 +109,7 @@ module "org_secrets" {

module "interconnect" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.0"
version = "~> 9.2"
random_project_id = "true"
impersonate_service_account = var.terraform_service_account
default_service_account = "deprivilege"
Expand Down Expand Up @@ -140,7 +140,7 @@ module "interconnect" {

module "scc_notifications" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.0"
version = "~> 9.2"
random_project_id = "true"
impersonate_service_account = var.terraform_service_account
default_service_account = "deprivilege"
Expand Down Expand Up @@ -171,7 +171,7 @@ module "scc_notifications" {

module "dns_hub" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.0"
version = "~> 9.2"
random_project_id = "true"
impersonate_service_account = var.terraform_service_account
default_service_account = "deprivilege"
Expand Down
2 changes: 1 addition & 1 deletion 2-environments/modules/env_baseline/monitoring.tf
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@

module "monitoring_project" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.0"
version = "~> 9.2"
random_project_id = "true"
impersonate_service_account = var.terraform_service_account
name = "prj-${var.environment_code}-monitoring"
Expand Down
4 changes: 2 additions & 2 deletions 2-environments/modules/env_baseline/networking.tf
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@

module "base_shared_vpc_host_project" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.0"
version = "~> 9.2"
random_project_id = "true"
impersonate_service_account = var.terraform_service_account
name = "prj-${var.environment_code}-shared-base"
Expand Down Expand Up @@ -54,7 +54,7 @@ module "base_shared_vpc_host_project" {

module "restricted_shared_vpc_host_project" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.0"
version = "~> 9.2"
random_project_id = "true"
impersonate_service_account = var.terraform_service_account
name = "prj-${var.environment_code}-shared-restricted"
Expand Down
2 changes: 1 addition & 1 deletion 2-environments/modules/env_baseline/secrets.tf
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@

module "env_secrets" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.0"
version = "~> 9.2"
random_project_id = "true"
impersonate_service_account = var.terraform_service_account
default_service_account = "deprivilege"
Expand Down
8 changes: 4 additions & 4 deletions 3-networks/envs/shared/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ module "dns-forwarding-zone" {

module "dns_hub_region1_router1" {
source = "terraform-google-modules/cloud-router/google"
version = "~> 0.2.0"
version = "~> 0.3.0"
name = "cr-c-dns-hub-${var.default_region1}-cr1"
project = local.dns_hub_project_id
network = module.dns_hub_vpc.network_name
Expand All @@ -122,7 +122,7 @@ module "dns_hub_region1_router1" {

module "dns_hub_region1_router2" {
source = "terraform-google-modules/cloud-router/google"
version = "~> 0.2.0"
version = "~> 0.3.0"
name = "cr-c-dns-hub-${var.default_region1}-cr2"
project = local.dns_hub_project_id
network = module.dns_hub_vpc.network_name
Expand All @@ -135,7 +135,7 @@ module "dns_hub_region1_router2" {

module "dns_hub_region2_router1" {
source = "terraform-google-modules/cloud-router/google"
version = "~> 0.2.0"
version = "~> 0.3.0"
name = "cr-c-dns-hub-${var.default_region2}-cr3"
project = local.dns_hub_project_id
network = module.dns_hub_vpc.network_name
Expand All @@ -148,7 +148,7 @@ module "dns_hub_region2_router1" {

module "dns_hub_region2_router2" {
source = "terraform-google-modules/cloud-router/google"
version = "~> 0.2.0"
version = "~> 0.3.0"
name = "cr-c-dns-hub-${var.default_region2}-cr4"
project = local.dns_hub_project_id
network = module.dns_hub_vpc.network_name
Expand Down
8 changes: 4 additions & 4 deletions 3-networks/modules/base_shared_vpc/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ resource "google_service_networking_connection" "private_vpc_connection" {

module "region1_router1" {
source = "terraform-google-modules/cloud-router/google"
version = "~> 0.2.0"
version = "~> 0.3.0"
name = "cr-${local.vpc_name}-${var.default_region1}-cr1"
project = var.project_id
network = module.main.network_name
Expand All @@ -109,7 +109,7 @@ module "region1_router1" {

module "region1_router2" {
source = "terraform-google-modules/cloud-router/google"
version = "~> 0.2.0"
version = "~> 0.3.0"
name = "cr-${local.vpc_name}-${var.default_region1}-cr2"
project = var.project_id
network = module.main.network_name
Expand All @@ -123,7 +123,7 @@ module "region1_router2" {

module "region2_router1" {
source = "terraform-google-modules/cloud-router/google"
version = "~> 0.2.0"
version = "~> 0.3.0"
name = "cr-${local.vpc_name}-${var.default_region2}-cr3"
project = var.project_id
network = module.main.network_name
Expand All @@ -137,7 +137,7 @@ module "region2_router1" {

module "region2_router2" {
source = "terraform-google-modules/cloud-router/google"
version = "~> 0.2.0"
version = "~> 0.3.0"
name = "cr-${local.vpc_name}-${var.default_region2}-cr4"
project = var.project_id
network = module.main.network_name
Expand Down
8 changes: 4 additions & 4 deletions 3-networks/modules/dedicated_interconnect/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ data "google_projects" "interconnect_project" {

module "interconnect_attachment1_region1" {
source = "terraform-google-modules/cloud-router/google//modules/interconnect_attachment"
version = "~> 0.2.0"
version = "~> 0.3.0"

name = "vl-${var.region1_interconnect1_location}-${var.vpc_name}-${var.region1}-${local.suffix1}"
project = local.interconnect_project_id
Expand All @@ -57,7 +57,7 @@ module "interconnect_attachment1_region1" {

module "interconnect_attachment2_region1" {
source = "terraform-google-modules/cloud-router/google//modules/interconnect_attachment"
version = "~> 0.2.0"
version = "~> 0.3.0"

name = "vl-${var.region1_interconnect2_location}-${var.vpc_name}-${var.region1}-${local.suffix2}"
project = local.interconnect_project_id
Expand All @@ -80,7 +80,7 @@ module "interconnect_attachment2_region1" {

module "interconnect_attachment1_region2" {
source = "terraform-google-modules/cloud-router/google//modules/interconnect_attachment"
version = "~> 0.2.0"
version = "~> 0.3.0"

name = "vl-${var.region2_interconnect1_location}-${var.vpc_name}-${var.region2}-${local.suffix3}"
project = local.interconnect_project_id
Expand All @@ -103,7 +103,7 @@ module "interconnect_attachment1_region2" {

module "interconnect_attachment2_region2" {
source = "terraform-google-modules/cloud-router/google//modules/interconnect_attachment"
version = "~> 0.2.0"
version = "~> 0.3.0"

name = "vl-${var.region2_interconnect2_location}-${var.vpc_name}-${var.region2}-${local.suffix4}"
project = local.interconnect_project_id
Expand Down
8 changes: 4 additions & 4 deletions 3-networks/modules/restricted_shared_vpc/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ resource "google_service_networking_connection" "private_vpc_connection" {

module "region1_router1" {
source = "terraform-google-modules/cloud-router/google"
version = "~> 0.2.0"
version = "~> 0.3.0"
name = "cr-${local.vpc_name}-${var.default_region1}-cr5"
project = var.project_id
network = module.main.network_name
Expand All @@ -111,7 +111,7 @@ module "region1_router1" {

module "region1_router2" {
source = "terraform-google-modules/cloud-router/google"
version = "~> 0.2.0"
version = "~> 0.3.0"
name = "cr-${local.vpc_name}-${var.default_region1}-cr6"
project = var.project_id
network = module.main.network_name
Expand All @@ -125,7 +125,7 @@ module "region1_router2" {

module "region2_router1" {
source = "terraform-google-modules/cloud-router/google"
version = "~> 0.2.0"
version = "~> 0.3.0"
name = "cr-${local.vpc_name}-${var.default_region2}-cr7"
project = var.project_id
network = module.main.network_name
Expand All @@ -139,7 +139,7 @@ module "region2_router1" {

module "region2_router2" {
source = "terraform-google-modules/cloud-router/google"
version = "~> 0.2.0"
version = "~> 0.3.0"
name = "cr-${local.vpc_name}-${var.default_region2}-cr8"
project = var.project_id
network = module.main.network_name
Expand Down
2 changes: 1 addition & 1 deletion 4-projects/modules/single_project/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ locals {

module "project" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.1"
version = "~> 9.2"
random_project_id = "true"
impersonate_service_account = var.impersonate_service_account
activate_apis = distinct(concat(var.activate_apis, ["billingbudgets.googleapis.com"]))
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
# Make will use bash instead of sh
SHELL := /usr/bin/env bash

DOCKER_TAG_VERSION_DEVELOPER_TOOLS := 0.12.2
DOCKER_TAG_VERSION_DEVELOPER_TOOLS := 0.13
DOCKER_IMAGE_DEVELOPER_TOOLS := cft/developer-tools
REGISTRY_URL := gcr.io/cloud-foundation-cicd

Expand Down
2 changes: 1 addition & 1 deletion build/int.cloudbuild.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -135,4 +135,4 @@ tags:
- 'integration'
substitutions:
_DOCKER_IMAGE_DEVELOPER_TOOLS: 'cft/developer-tools'
_DOCKER_TAG_VERSION_DEVELOPER_TOOLS: '0.12.0'
_DOCKER_TAG_VERSION_DEVELOPER_TOOLS: '0.13'
2 changes: 1 addition & 1 deletion build/lint.cloudbuild.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ tags:
- 'lint'
substitutions:
_DOCKER_IMAGE_DEVELOPER_TOOLS: 'cft/developer-tools'
_DOCKER_TAG_VERSION_DEVELOPER_TOOLS: '0.12.2'
_DOCKER_TAG_VERSION_DEVELOPER_TOOLS: '0.13'
options:
machineType: 'N1_HIGHCPU_8'
env:
Expand Down
2 changes: 1 addition & 1 deletion test/integration/projects/controls/gcloud_projects.rb
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@
end

it "has a peering with #{peering_networks[environment_code][business_unit]['network']}" do
expect(data[0]['peerings'][0]['network'].should eq peering_networks[environment_code][business_unit]['network'])
expect(data[0]['peerings'][0]['network'].should eq peering_networks[environment_code][business_unit][:network])
end
end
end
Expand Down
2 changes: 1 addition & 1 deletion test/setup/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ resource "google_folder" "test_folder" {

module "project" {
source = "terraform-google-modules/project-factory/google"
version = "~> 8.0"
version = "~> 9.2"

name = "ci-foundation"
random_project_id = true
Expand Down

0 comments on commit c5c6c6c

Please sign in to comment.