Turned out that COMMON_MYSQL_MIGRATE_PASS|bool is resolved to false even if COMMON_MYSQL_MIGRATE_PASS|length is greater than zero!!
bool filter is a conversion of True string into a boolean, not a boolean check as we would intuitively assume. See here
Make sure that the following steps are done before merging:
[ ] A DevOps team member has approved the PR.
[ ] Are you adding any new default values that need to be overridden when this change goes live? If so:
[ ] Update the appropriate internal repo (be sure to update for all our environments)
[ ] If you are updating a secure value rather than an internal one, file a DEVOPS ticket with details.
[ ] Add an entry to the CHANGELOG.
[ ] If you are making a complicated change, have you performed the proper testing specified on the Ops Ansible Testing Checklist? Adding a new variable does not require the full list (although testing on a sandbox is a great idea to ensure it links with your downstream code changes).
shadinaif
commented
2 years ago
Fix for https://appsembler.atlassian.net/browse/RED-3045
Turned out that
COMMON_MYSQL_MIGRATE_PASS|boolis resolved tofalseeven ifCOMMON_MYSQL_MIGRATE_PASS|lengthis greater than zero!!boolfilter is a conversion ofTruestring into a boolean, not a boolean check as we would intuitively assume. See hereMake sure that the following steps are done before merging: