Skip to content

Commit f7ea39d

Browse files
committed
need to use all bedrock accts and regions for delete
1 parent ad42f90 commit f7ea39d

File tree

1 file changed

+4
-3
lines changed
  • aws_sra_examples/solutions/genai/bedrock_org/lambda/src

1 file changed

+4
-3
lines changed

aws_sra_examples/solutions/genai/bedrock_org/lambda/src/app.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1245,10 +1245,11 @@ def delete_event(event, context):
12451245
LOGGER.info("CloudWatch observability access manager sink not found")
12461246

12471247
# 3) Delete metric alarms and filters
1248+
accounts, regions = get_accounts_and_regions(event["ResourceProperties"])
12481249
for filter in CLOUDWATCH_METRIC_FILTERS:
1249-
filter_deploy, filter_accounts, filter_regions, filter_params = get_filter_params(filter, event)
1250-
for acct in filter_accounts:
1251-
for region in filter_regions:
1250+
# filter_deploy, filter_accounts, filter_regions, filter_params = get_filter_params(filter, event)
1251+
for acct in accounts:
1252+
for region in regions:
12521253
# 3a) Delete KMS key (schedule deletion) and delete kms alias
12531254
kms.KMS_CLIENT = sts.assume_role(acct, sts.CONFIGURATION_ROLE, "kms", region)
12541255
search_alarm_kms_key, alarm_key_alias, alarm_key_id, alarm_key_arn = kms.check_alias_exists(kms.KMS_CLIENT, f"alias/{ALARM_SNS_KEY_ALIAS}")

0 commit comments

Comments
 (0)