Skip to content
This repository has been archived by the owner on Nov 22, 2023. It is now read-only.

Commit

Permalink
chore: update keyvault module + suppress checkov warning in examples
Browse files Browse the repository at this point in the history
  • Loading branch information
piaverous committed Aug 11, 2023
1 parent 6006e3c commit 754d62a
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ resource "random_pet" "random" {}
data "azurerm_client_config" "this" {}

module "key_vault" {
source = "[email protected]:padok-team/terraform-azurerm-keyvault.git?ref=v0.4.0"
# checkov:skip=CKV_TF_1:Do not want to use commit tags for source ref
source = "[email protected]:padok-team/terraform-azurerm-keyvault.git?ref=v0.5.0"

name = random_pet.random.id # KeyVault names are globally unique
resource_group = azurerm_resource_group.this
Expand Down
3 changes: 2 additions & 1 deletion examples/log-analytics-workspace-logger-metrics-only/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ resource "random_pet" "random" {}
data "azurerm_client_config" "this" {}

module "key_vault" {
source = "[email protected]:padok-team/terraform-azurerm-keyvault.git?ref=v0.4.0"
# checkov:skip=CKV_TF_1:Do not want to use commit tags for source ref
source = "[email protected]:padok-team/terraform-azurerm-keyvault.git?ref=v0.5.0"

name = random_pet.random.id # KeyVault names are globally unique
resource_group = azurerm_resource_group.this
Expand Down
2 changes: 1 addition & 1 deletion main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ resource "azurerm_monitor_diagnostic_setting" "logs" {
dynamic "enabled_log" {
for_each = data.azurerm_monitor_diagnostic_categories.logs[count.index].logs
content {
category = log.value
category = enabled_log.value
retention_policy {
enabled = false
days = 0
Expand Down

0 comments on commit 754d62a

Please sign in to comment.