Skip to content

Commit 9be2aa8

Browse files
vkuma17terraform-ibm-modules-opsVipin Kumar
authored
fix: renovate fixes (dependencies version change)<br>incorporate the changes in structure of metrics_router_settings variable that got changed in monitoring module <br>incorporate the changes in test for trusted_profile_links and trusted_profile_policies, these were updated in trusted_profile module (#432)
* fix(deps): update terraform dependencies * fix: renovate fix * updated event notifications module version * trusted profile version change * fix --------- Co-authored-by: Terraform IBM Modules Operations <[email protected]> Co-authored-by: Vipin Kumar <[email protected]>
1 parent e99af42 commit 9be2aa8

File tree

5 files changed

+23
-25
lines changed

5 files changed

+23
-25
lines changed

solutions/instances/DA-types.md

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -155,9 +155,7 @@ The `metrics_router_settings` input variable allows you to set the account setti
155155

156156
```hcl
157157
metrics_router_settings = {
158-
default_targets = [{
159-
id = "c3af557f-fb0e-2222-85c3-0889e7fe7bc4"
160-
}]
158+
default_targets = ["c3af557f-fb0e-2222-85c3-0889e7fe7bc4"]
161159
primary_metadata_region = "us-south"
162160
backup_metadata_region = "us-east"
163161
permitted_target_regions = ["us-south", "eu-de", "us-east", "eu-es", "eu-gb", "au-syd", "br-sao", "ca-tor"]

solutions/instances/main.tf

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -245,7 +245,7 @@ module "cloud_monitoring_crn_parser" {
245245
module "cloud_monitoring" {
246246
count = var.cloud_monitoring_provision ? 1 : 0
247247
source = "terraform-ibm-modules/cloud-monitoring/ibm"
248-
version = "1.4.0"
248+
version = "1.5.0"
249249
region = var.region
250250
resource_group_id = module.resource_group.resource_group_id
251251
instance_name = local.cloud_monitoring_instance_name
@@ -304,7 +304,7 @@ module "cloud_logs" {
304304

305305
module "metrics_router" {
306306
source = "terraform-ibm-modules/cloud-monitoring/ibm//modules/metrics_routing"
307-
version = "1.4.0"
307+
version = "1.5.0"
308308
metrics_router_targets = var.enable_metrics_routing_to_cloud_monitoring ? [
309309
{
310310
destination_crn = var.cloud_monitoring_provision ? module.cloud_monitoring[0].crn : var.existing_cloud_monitoring_crn
@@ -320,7 +320,7 @@ module "metrics_router" {
320320
module "activity_tracker" {
321321
depends_on = [time_sleep.wait_for_atracker_cos_authorization_policy]
322322
source = "terraform-ibm-modules/activity-tracker/ibm"
323-
version = "1.1.6"
323+
version = "1.1.7"
324324
cos_targets = var.enable_at_event_routing_to_cos_bucket ? [
325325
{
326326
bucket_name = local.cos_target_bucket_name
@@ -466,7 +466,7 @@ module "cos_instance" {
466466
}
467467
count = var.existing_cos_instance_crn == null && length(coalesce(local.buckets_config, [])) != 0 ? 1 : 0 # no need to call COS module if consumer is using existing COS instance
468468
source = "terraform-ibm-modules/cos/ibm//modules/fscloud"
469-
version = "8.21.25"
469+
version = "10.1.1"
470470
resource_group_id = local.cos_resource_group_id
471471
create_cos_instance = true
472472
cos_instance_name = try("${local.prefix}-${var.cos_instance_name}", var.cos_instance_name)
@@ -483,7 +483,7 @@ module "cos_bucket" {
483483
}
484484
count = length(coalesce(local.buckets_config, [])) != 0 ? 1 : 0 # no need to call COS module if consumer is using existing COS bucket
485485
source = "terraform-ibm-modules/cos/ibm//modules/buckets"
486-
version = "8.21.25"
486+
version = "10.1.1"
487487
bucket_configs = [
488488
for value in local.buckets_config :
489489
{

solutions/instances/variables.tf

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -231,13 +231,11 @@ variable "enable_metrics_routing_to_cloud_monitoring" {
231231

232232
variable "metrics_router_settings" {
233233
type = object({
234-
permitted_target_regions = optional(list(string))
234+
permitted_target_regions = optional(list(string), [])
235235
primary_metadata_region = optional(string)
236236
backup_metadata_region = optional(string)
237237
private_api_endpoint_only = optional(bool, false)
238-
default_targets = optional(list(object({
239-
id = string
240-
})))
238+
default_targets = optional(list(string), [])
241239
})
242240
description = "Global settings for Metrics Routing. [Learn more](https://github.com/terraform-ibm-modules/terraform-ibm-observability-da/blob/main/solutions/instances/DA-types.md#metrics-router-settings-)."
243241
default = null

tests/resources/existing-resources/main.tf

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ module "resource_group" {
1616

1717
module "cos" {
1818
source = "terraform-ibm-modules/cos/ibm"
19-
version = "8.21.25"
19+
version = "10.1.1"
2020
resource_group_id = module.resource_group.resource_group_id
2121
region = var.region
2222
cos_instance_name = "${var.prefix}-cos"
@@ -28,7 +28,7 @@ module "cos" {
2828

2929
module "additional_cos_bucket" {
3030
source = "terraform-ibm-modules/cos/ibm"
31-
version = "8.21.25"
31+
version = "10.1.1"
3232
region = var.region
3333
create_cos_instance = false
3434
existing_cos_instance_id = module.cos.cos_instance_id
@@ -38,7 +38,7 @@ module "additional_cos_bucket" {
3838

3939
module "cloud_log_buckets" {
4040
source = "terraform-ibm-modules/cos/ibm//modules/buckets"
41-
version = "8.21.25"
41+
version = "10.1.1"
4242
bucket_configs = [
4343
{
4444
bucket_name = "${var.prefix}-data-bucket"
@@ -62,7 +62,7 @@ module "cloud_log_buckets" {
6262

6363
module "cloud_monitoring" {
6464
source = "terraform-ibm-modules/cloud-monitoring/ibm"
65-
version = "1.4.0"
65+
version = "1.5.0"
6666
region = var.region
6767
resource_group_id = module.resource_group.resource_group_id
6868
instance_name = var.prefix
@@ -76,7 +76,7 @@ module "cloud_monitoring" {
7676

7777
module "event_notification_1" {
7878
source = "terraform-ibm-modules/event-notifications/ibm"
79-
version = "2.4.22"
79+
version = "2.5.0"
8080
resource_group_id = module.resource_group.resource_group_id
8181
name = "${var.prefix}-en-1"
8282
tags = var.resource_tags
@@ -87,7 +87,7 @@ module "event_notification_1" {
8787

8888
module "event_notification_2" {
8989
source = "terraform-ibm-modules/event-notifications/ibm"
90-
version = "2.4.22"
90+
version = "2.5.0"
9191
resource_group_id = module.resource_group.resource_group_id
9292
name = "${var.prefix}-en-2"
9393
tags = var.resource_tags

tests/resources/main.tf

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ resource "ibm_is_subnet" "subnet_zone_1" {
4343

4444
module "cos" {
4545
source = "terraform-ibm-modules/cos/ibm"
46-
version = "8.21.25"
46+
version = "10.1.1"
4747
resource_group_id = module.resource_group.resource_group_id
4848
cos_instance_name = "${var.prefix}-cos"
4949
cos_tags = var.resource_tags
@@ -61,7 +61,7 @@ locals {
6161

6262
module "buckets" {
6363
source = "terraform-ibm-modules/cos/ibm//modules/buckets"
64-
version = "8.21.25"
64+
version = "10.1.1"
6565
bucket_configs = [
6666
{
6767
bucket_name = local.logs_bucket_name
@@ -86,7 +86,7 @@ module "buckets" {
8686

8787
module "cloud_monitoring" {
8888
source = "terraform-ibm-modules/cloud-monitoring/ibm"
89-
version = "1.4.0"
89+
version = "1.5.0"
9090
region = var.region
9191
resource_group_id = module.resource_group.resource_group_id
9292
instance_name = var.prefix
@@ -127,20 +127,22 @@ locals {
127127

128128
module "trusted_profile" {
129129
source = "terraform-ibm-modules/trusted-profile/ibm"
130-
version = "2.3.1"
130+
version = "3.1.1"
131131
trusted_profile_name = "${var.prefix}-profile"
132132
trusted_profile_description = "Logs agent Trusted Profile"
133133
# As a `Sender`, you can send logs to your IBM Cloud Logs service instance - but not query or tail logs. This role is meant to be used by agents and routers sending logs.
134134
trusted_profile_policies = [{
135-
roles = ["Sender"]
135+
unique_identifier = "${var.prefix}-profile-0"
136+
roles = ["Sender"]
136137
resources = [{
137138
service = "logs"
138139
}]
139140
}]
140141

141142
# Set up fine-grained authorization for `logs-agent` running in ROKS cluster in `ibm-observe` namespace.
142143
trusted_profile_links = [{
143-
cr_type = "ROKS_SA"
144+
unique_identifier = "${var.prefix}-profile-1"
145+
cr_type = "ROKS_SA"
144146
links = [{
145147
crn = module.ocp_base.cluster_crn
146148
namespace = local.logs_agent_namespace
@@ -178,7 +180,7 @@ locals {
178180

179181
module "ocp_base" {
180182
source = "terraform-ibm-modules/base-ocp-vpc/ibm"
181-
version = "3.52.3"
183+
version = "3.52.4"
182184
resource_group_id = module.resource_group.resource_group_id
183185
region = var.region
184186
tags = var.resource_tags

0 commit comments

Comments
 (0)