forseti-security / terraform-google-forseti

A Terraform module for installing Forseti on GCP
Apache License 2.0
132 stars 127 forks source link

Update run_forseti.sh.tpl #642

Closed edwardmedia closed 2 years ago

edwardmedia commented 2 years ago

fixed script by calling python3 instead

gkowalski-google commented 2 years ago

This seems ok. The terraform tests failed related to InSpec:

>>>>>> gcloud: Executing InSpec failed
Unable to resolve master to a specific git commit for https://github.com/inspec/inspec-gcp.git

I'm not sure if that's related to this change.

edwardmedia commented 2 years ago

@gkowalski-google can you approve it or wait for a 2nd check I can't merge it without approval.

rosmo commented 2 years ago

LGTM, was just about to make the same patch.

morgante commented 2 years ago

Merging since @rosmo has tested.