rhtconsulting / rhc-ose

OpenShift Automation and Utilities by Red Hat Consulting
42 stars 34 forks source link

Fixes Issue #163 if rhsm_password is not defined #218

Closed vvaldez closed 8 years ago

vvaldez commented 8 years ago

What does this PR do?

Changes the set_fact task in subscription-manager pre_task that sets the rhsm_password. Previously this task was assuming if the password was not being used it was being set to rhsm_password=''. This PR changes the check to default to None if no password is set.

How should this be manually tested?

Run a provision without rhsm_password set

Is there a relevant Issue open for this?

Fixes Issue #163.

Who would you like to review this?

/cc @etsauer

etsauer commented 8 years ago

LGTM. Merging.