Skip to content

Commit 18bd30f

Browse files
feat: updating service attachment count (#10)
* feat: updating service attachment count * docs: updated changelog for v1.0.1 * test: wrong amount of service attachments to test for
1 parent c913952 commit 18bd30f

File tree

3 files changed

+11
-5
lines changed

3 files changed

+11
-5
lines changed

CHANGELOG.md

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,15 @@
33
All notable changes to this project will be documented in this file.
44
See updating [Changelog example here](https://keepachangelog.com/en/1.0.0/)
55

6+
## 1.0.1
7+
8+
### Changed
9+
10+
- Changed the hardcoded service attachment count from 40 to 1.
11+
612
## 1.0.0
713

814
### Added
915

10-
- Initial version of the module which provision Private Service Connect service and endpoints for Active-Active and Pro
11-
subscriptions and related GCP resources.
16+
- Initial version of the module which provision Private Service Connect service and endpoints for Active-Active and Pro
17+
subscriptions and related GCP resources.

modules/gcp_service_attachments/main.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ locals {
33
# plan time, we cannot use `service_attachments` from the private service connect endpoint directly
44
# as those are only computed after applying the plan. Because of that we need a fixed count
55
# here unfortunately.
6-
service_attachment_count = 40
6+
service_attachment_count = 1
77
}
88

99
data "google_compute_subnetwork" "subnet" {
@@ -50,4 +50,4 @@ resource "google_dns_response_policy_rule" "redis_response_policy_rules" {
5050
rrdatas = [google_compute_address.redis_ip_addresses[count.index].address]
5151
}
5252
}
53-
}
53+
}

test/terraform_private_service_connect_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ import (
1414
"github.com/stretchr/testify/require"
1515
)
1616

17-
const expectNumberOfServiceAttachments = 40
17+
const expectNumberOfServiceAttachments = 1
1818

1919
func TestTerraformPrivateServiceConnect(t *testing.T) {
2020
gcpProject := os.Getenv("GCP_PROJECT_ID")

0 commit comments

Comments
 (0)