openshift / openshift-ansible-contrib

Additional roles and playbooks for OpenShift installation and management
https://www.openshift.com
Apache License 2.0
283 stars 372 forks source link

Updates for AWS ref arch #1063

Closed mazzystr closed 6 years ago

mazzystr commented 6 years ago

What does this PR do?

Fixes a problem with the condition on the NOTICE! debug/fail task. When an ami is not not specified in the vars file the Cloud Access ami search task is triggered. When the ami search task can't find an ami the fail task "NOTICE! Red Hat Cloud Access machine image not found" task is supposed to be triggered. The NOTICE! task will fail instead of executing debug output then fail.

Force aws describe-images command to json to ensure jq can consume and process output.

Fix to IAM policy statement

How should this be manually tested?

See OCP 3.9 on AWS reference architecture.

Is there a relevant Issue open for this?

Who would you like to review this?

@cooktheryan