Skip to content

Commit

Permalink
Merge pull request #140 from apigee/feature/bump-cff-dependency
Browse files Browse the repository at this point in the history
Bump CFF dependencies to v28.0.0 (latest)
  • Loading branch information
danistrebel authored Dec 18, 2023
2 parents 58b1cac + d34508d commit 2a3f8bb
Show file tree
Hide file tree
Showing 52 changed files with 280 additions and 264 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -13,3 +13,4 @@ modules/apigee-hybrid-workload/overrides/
**.key
modules/apigee-hybrid-workload/apigeectl_*
__pycache__
.venv
4 changes: 2 additions & 2 deletions modules/apigee-x-bridge-mig/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,8 @@

| Name | Source | Version |
|------|--------|---------|
| <a name="module_bridge-mig"></a> [bridge-mig](#module\_bridge-mig) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig | v16.0.0 |
| <a name="module_bridge-template"></a> [bridge-template](#module\_bridge-template) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm | v16.0.0 |
| <a name="module_bridge-mig"></a> [bridge-mig](#module\_bridge-mig) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig | v28.0.0 |
| <a name="module_bridge-template"></a> [bridge-template](#module\_bridge-template) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm | v28.0.0 |

## Resources

Expand Down
29 changes: 13 additions & 16 deletions modules/apigee-x-bridge-mig/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ locals {
}

module "bridge-template" {
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm?ref=v16.0.0"
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm?ref=v28.0.0"
project_id = var.project_id
name = local.bridge_name
zone = "${var.region}-b"
Expand All @@ -33,31 +33,31 @@ module "bridge-template" {
alias_ips = null
}]
boot_disk = {
image = "debian-cloud/debian-11"
type = "pd-standard"
size = 20
initialize_params = {
image = "debian-cloud/debian-11"
type = "pd-standard"
size = 20
}
}
create_template = true
metadata = {
ENDPOINT = var.endpoint_ip
startup-script-url = "gs://apigee-5g-saas/apigee-envoy-proxy-release/latest/conf/startup-script.sh"
}
service_account_create = true
service_account_scopes = ["cloud-platform"]
service_account = {
auto_create = true
scopes = ["cloud-platform"]
}
}

module "bridge-mig" {
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig?ref=v16.0.0"
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig?ref=v28.0.0"
project_id = var.project_id
location = var.region
regional = true
name = local.bridge_name
target_size = var.target_size
autoscaler_config = var.autoscaler_config
default_version = {
instance_template = module.bridge-template.template.self_link
name = "default"
}
instance_template = module.bridge-template.template.self_link
named_ports = {
https = 443
}
Expand All @@ -66,13 +66,10 @@ module "bridge-mig" {
initial_delay_sec = 30
}
health_check_config = {
type = "https"
check = {
https = {
port = 443,
request_path = "/healthz/ingress"
}
config = {}
logging = false
}
}

Expand Down
6 changes: 3 additions & 3 deletions modules/apigee-x-core/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -61,9 +61,9 @@

| Name | Source | Version |
|------|--------|---------|
| <a name="module_apigee"></a> [apigee](#module\_apigee) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/apigee | v26.0.0 |
| <a name="module_kms-inst-disk"></a> [kms-inst-disk](#module\_kms-inst-disk) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/kms | v26.0.0 |
| <a name="module_kms-org-db"></a> [kms-org-db](#module\_kms-org-db) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/kms | v26.0.0 |
| <a name="module_apigee"></a> [apigee](#module\_apigee) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/apigee | v28.0.0 |
| <a name="module_kms-inst-disk"></a> [kms-inst-disk](#module\_kms-inst-disk) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/kms | v28.0.0 |
| <a name="module_kms-org-db"></a> [kms-org-db](#module\_kms-org-db) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/kms | v28.0.0 |

## Resources

Expand Down
6 changes: 3 additions & 3 deletions modules/apigee-x-core/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ resource "google_project_service_identity" "apigee_sa" {
}

module "kms-org-db" {
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/kms?ref=v26.0.0"
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/kms?ref=v28.0.0"
project_id = var.project_id
iam = {
"roles/cloudkms.cryptoKeyEncrypterDecrypter" = ["serviceAccount:${google_project_service_identity.apigee_sa.email}"]
Expand All @@ -49,7 +49,7 @@ module "kms-org-db" {

module "kms-inst-disk" {
for_each = var.apigee_instances
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/kms?ref=v26.0.0"
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/kms?ref=v28.0.0"
project_id = var.project_id
iam = {
"roles/cloudkms.cryptoKeyEncrypterDecrypter" = ["serviceAccount:${google_project_service_identity.apigee_sa.email}"]
Expand All @@ -68,7 +68,7 @@ module "kms-inst-disk" {
}

module "apigee" {
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/apigee?ref=v26.0.0"
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/apigee?ref=v28.0.0"
project_id = var.project_id
organization = {
display_name = var.org_display_name
Expand Down
10 changes: 5 additions & 5 deletions modules/apigee-x-mtls-mig/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,11 @@ route_config:
| Name | Source | Version |
|------|--------|---------|
| <a name="module_apigee-mtls-proxy-mig"></a> [apigee-mtls-proxy-mig](#module\_apigee-mtls-proxy-mig) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig | v16.0.0 |
| <a name="module_apigee-mtls-proxy-template"></a> [apigee-mtls-proxy-template](#module\_apigee-mtls-proxy-template) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm | v16.0.0 |
| <a name="module_config-bucket"></a> [config-bucket](#module\_config-bucket) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/gcs | v16.0.0 |
| <a name="module_mtls-proxy-sa"></a> [mtls-proxy-sa](#module\_mtls-proxy-sa) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/iam-service-account | v16.0.0 |
| <a name="module_nat"></a> [nat](#module\_nat) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/net-cloudnat | v16.0.0 |
| <a name="module_apigee-mtls-proxy-mig"></a> [apigee-mtls-proxy-mig](#module\_apigee-mtls-proxy-mig) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig | v28.0.0 |
| <a name="module_apigee-mtls-proxy-template"></a> [apigee-mtls-proxy-template](#module\_apigee-mtls-proxy-template) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm | v28.0.0 |
| <a name="module_config-bucket"></a> [config-bucket](#module\_config-bucket) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/gcs | v28.0.0 |
| <a name="module_mtls-proxy-sa"></a> [mtls-proxy-sa](#module\_mtls-proxy-sa) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/iam-service-account | v28.0.0 |
| <a name="module_nat"></a> [nat](#module\_nat) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/net-cloudnat | v28.0.0 |
## Resources
Expand Down
30 changes: 15 additions & 15 deletions modules/apigee-x-mtls-mig/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,13 @@ resource "random_id" "bucket" {
}

module "mtls-proxy-sa" {
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/iam-service-account?ref=v16.0.0"
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/iam-service-account?ref=v28.0.0"
project_id = var.project_id
name = "apigee-mtls-proxy-vm"
}

module "config-bucket" {
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/gcs?ref=v16.0.0"
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/gcs?ref=v28.0.0"
project_id = var.project_id
name = "apigee-mtls-ingress-${random_id.bucket.dec}"
location = "EU"
Expand Down Expand Up @@ -65,7 +65,7 @@ resource "google_storage_bucket_object" "tls_key" {
}

module "apigee-mtls-proxy-template" {
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm?ref=v16.0.0"
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm?ref=v28.0.0"
project_id = var.project_id
name = "apigee-nb-mtls-proxy"
zone = "${var.region}-b"
Expand All @@ -79,17 +79,21 @@ module "apigee-mtls-proxy-template" {
alias_ips = null
}]
boot_disk = {
image = "projects/debian-cloud/global/images/family/debian-11"
type = "pd-standard"
size = 10
initialize_params = {
image = "projects/debian-cloud/global/images/family/debian-11"
type = "pd-standard"
size = 10
}
}
create_template = true
metadata = {
BUCKET = module.config-bucket.name
startup-script-url = "gs://${module.config-bucket.name}/setup.sh"
}
service_account = module.mtls-proxy-sa.email
service_account_scopes = ["cloud-platform"]
service_account = {
scopes = ["cloud-platform"]
email = module.mtls-proxy-sa.email
}
depends_on = [
google_storage_bucket_object.setup_script,
google_storage_bucket_object.ca_cert,
Expand All @@ -101,24 +105,20 @@ module "apigee-mtls-proxy-template" {
}

module "apigee-mtls-proxy-mig" {
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig?ref=v16.0.0"
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig?ref=v28.0.0"
project_id = var.project_id
location = var.region
regional = true
name = "apigee-mtls-proxy-${var.region}"
target_size = var.target_size
autoscaler_config = var.autoscaler_config
named_ports = {
https = 443
}
default_version = {
instance_template = module.apigee-mtls-proxy-template.template.self_link
name = "default"
}
instance_template = module.apigee-mtls-proxy-template.template.self_link
}

module "nat" {
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/net-cloudnat?ref=v16.0.0"
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/net-cloudnat?ref=v28.0.0"
project_id = var.project_id
region = var.region
name = "nat-${var.region}"
Expand Down
6 changes: 3 additions & 3 deletions modules/development-backend/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,9 @@

| Name | Source | Version |
|------|--------|---------|
| <a name="module_demo-backend-mig"></a> [demo-backend-mig](#module\_demo-backend-mig) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig | v16.0.0 |
| <a name="module_demo-backend-template"></a> [demo-backend-template](#module\_demo-backend-template) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm | v16.0.0 |
| <a name="module_ilb-backend"></a> [ilb-backend](#module\_ilb-backend) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/net-ilb | v16.0.0 |
| <a name="module_demo-backend-mig"></a> [demo-backend-mig](#module\_demo-backend-mig) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig | v28.0.0 |
| <a name="module_demo-backend-template"></a> [demo-backend-template](#module\_demo-backend-template) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm | v28.0.0 |
| <a name="module_ilb-backend"></a> [ilb-backend](#module\_ilb-backend) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/net-lb-int | v28.0.0 |

## Resources

Expand Down
51 changes: 27 additions & 24 deletions modules/development-backend/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
*/

module "demo-backend-template" {
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm?ref=v16.0.0"
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm?ref=v28.0.0"
project_id = var.project_id
name = var.name
zone = "${var.region}-b"
Expand All @@ -29,40 +29,46 @@ module "demo-backend-template" {
alias_ips = null
}]
boot_disk = {
image = "projects/debian-cloud/global/images/family/debian-11"
type = "pd-standard"
size = 10
initialize_params = {
image = "projects/debian-cloud/global/images/family/debian-11"
type = "pd-standard"
size = 10
}
}
create_template = true
metadata = {
startup-script = "sudo mkdir -p /var/www && cd /var/www && echo \"hello from $(hostname)\" > index.html && python3 -m http.server 80"
}
service_account_create = true
service_account_scopes = ["cloud-platform"]
service_account = {
auto_create = true
scopes = ["cloud-platform"]
}
}

module "demo-backend-mig" {
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig?ref=v16.0.0"
project_id = var.project_id
location = var.region
regional = true
name = "${var.name}-${var.region}"
target_size = 2
default_version = {
instance_template = module.demo-backend-template.template.self_link
name = "default"
}
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig?ref=v28.0.0"
project_id = var.project_id
location = var.region
name = "${var.name}-${var.region}"
target_size = 2
instance_template = module.demo-backend-template.template.self_link
}

module "ilb-backend" {
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/net-ilb?ref=v16.0.0"
source = "github.com/terraform-google-modules/cloud-foundation-fabric//modules/net-lb-int?ref=v28.0.0"
project_id = var.project_id
region = var.region
name = var.name
service_label = var.name
network = var.network
subnetwork = var.subnet
ports = [80]
vpc_config = {
network = var.network
subnetwork = var.subnet
}
forwarding_rules_config = {
"" = {
ports = [80]
}
}
backends = [
{
group = module.demo-backend-mig.group_manager.instance_group,
Expand All @@ -71,10 +77,7 @@ module "ilb-backend" {
}
]
health_check_config = {
type = "tcp"
check = { port = 80 }
config = {}
logging = false
tcp = { port = 80 }
}
}

Expand Down
6 changes: 3 additions & 3 deletions modules/development-backend/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -21,15 +21,15 @@ output "instance_group" {

output "ilb_forwarding_rule_address" {
description = "ILB forwarding rule IP address."
value = module.ilb-backend.forwarding_rule_address
value = module.ilb-backend.forwarding_rule_addresses[""]
}

output "ilb_forwarding_rule_self_link" {
description = "ILB forwarding rule self link."
value = module.ilb-backend.forwarding_rule_self_link
value = module.ilb-backend.forwarding_rule_self_links[""]
}

output "region" {
description = "Backend Service region."
value = module.ilb-backend.forwarding_rule.region
value = module.ilb-backend.forwarding_rules[""].region
}
10 changes: 5 additions & 5 deletions modules/routing-appliance/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@

| Name | Source | Version |
|------|--------|---------|
| <a name="module_appliance-sa"></a> [appliance-sa](#module\_appliance-sa) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/iam-service-account | v16.0.0 |
| <a name="module_config-bucket"></a> [config-bucket](#module\_config-bucket) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/gcs | v16.0.0 |
| <a name="module_ilb-appliance"></a> [ilb-appliance](#module\_ilb-appliance) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/net-ilb | v16.0.0 |
| <a name="module_routing-appliance-mig"></a> [routing-appliance-mig](#module\_routing-appliance-mig) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig | v16.0.0 |
| <a name="module_routing-appliance-template"></a> [routing-appliance-template](#module\_routing-appliance-template) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm | v16.0.0 |
| <a name="module_appliance-sa"></a> [appliance-sa](#module\_appliance-sa) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/iam-service-account | v28.0.0 |
| <a name="module_config-bucket"></a> [config-bucket](#module\_config-bucket) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/gcs | v28.0.0 |
| <a name="module_ilb-appliance"></a> [ilb-appliance](#module\_ilb-appliance) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/net-lb-int | v28.0.0 |
| <a name="module_routing-appliance-mig"></a> [routing-appliance-mig](#module\_routing-appliance-mig) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-mig | v28.0.0 |
| <a name="module_routing-appliance-template"></a> [routing-appliance-template](#module\_routing-appliance-template) | github.com/terraform-google-modules/cloud-foundation-fabric//modules/compute-vm | v28.0.0 |

## Resources

Expand Down
Loading

0 comments on commit 2a3f8bb

Please sign in to comment.