forseti cron script fails because python do not have an alias

[e-koma]

There are 2 problems in the forseti cron script.

• The first one is a terraform configuration issue.
• The second one is a forseti self issue, so I will propose an issue to the forseti repository.

Condition

• terraform-google-forseti: modulerelease522
• forseti version: v2.25.2

The first issue

run_forseti.sh which is forseti cron script fails because the python command does not have an alias. (In fact, the startup-script runs python3 directly )

$ forseti model get ${MODEL_NAME} | python -c \"import sys, json; print json.load(sys.stdin)['status']\"

python: command not found

The second issue.

the syntax does not support python3.

run_forseti.sh uses the python2 syntax, so even if run it with python3, get a syntax error. python3 print method requires (). （this is a problem on the forseti side, not on the terraform side, so I will propose an Issue to forseti repository）

$ forseti model get ${MODEL_NAME} | python3 -c \"import sys, json; print json.load(sys.stdin)['status']\"

syntax error

[e-koma]

I suggested that run_forseti.sh should use python3 directly. because docker environment uses python3 directly

If the proposal is accepted, this Issue will be closed.

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 15 days if no further activity occurs. Thank you for your contributions.