Skip to content

Commit

Permalink
fix: add VPC Flow logs exceptions for REGIONAL_MANAGED_PROXY and INTE…
Browse files Browse the repository at this point in the history
…RNAL_HTTPS_LOAD_BALANCER (#976)

Co-authored-by: Grant Sorbo <gtsorbo@google.com>
  • Loading branch information
daniel-cit and gtsorbo committed Nov 8, 2023
1 parent 9e06ccd commit dd4ff91
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -62,9 +62,12 @@ spec:
legacy_enable_flow_logs := lib.get_default(network, "enableFlowLogs", false)
log_config := lib.get_default(network, "logConfig", {})
log_config_enable_flow_logs := lib.get_default(log_config, "enable", false)
purpose := lib.get_default(network, "purpose", "PRIVATE")
log_config_enable_flow_logs != true
legacy_enable_flow_logs != true
purpose != "REGIONAL_MANAGED_PROXY"
purpose != "INTERNAL_HTTPS_LOAD_BALANCER"
message := sprintf("Flow logs are disabled in subnetwork %v.", [asset.name])
metadata := {"resource": asset.name}
Expand Down

0 comments on commit dd4ff91

Please sign in to comment.