Skip to content

Commit 1cf4d0f

Browse files
author
Dominique Quatravaux
committed
[hotfix] We want site_has_own_route to be a Boolean
We are not entirely sure why this was working before. But if a `>` multi-line indicator is used, the final newline in the quoted multi-line string is kept; resulting in something that Ansible is well within its rights to refuse to interpret as a Boolean. As a consequence, the `when: site_has_own_route` guard in `ansible/roles/wordpress-instance/tasks/routes.yml` was always true, resulting in spurious attempts to apply the `oc` command (which doesn't - yet? - exist in the ansible-runner image).
1 parent 9bb62ca commit 1cf4d0f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

ansible/roles/wordpress-instance/vars/routes-vars.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ route_is_root_url: '{{ lookup("wpveritas", "url") is match("https?://[^/]+$") }}
33
route_hostname: '{{ lookup("wpveritas", "url") | hostname_of_url }}'
44
route_name: '{{ lookup("wpveritas", "url") | route_name(route_wp_veritas_env) }}'
55
site_exists: '{{ not lookup("wpveritas", "isDeleted") }}'
6-
site_has_own_route: >
6+
site_has_own_route: >-
77
{{ route_wp_veritas_env.startswith("unm-")
88
or route_wp_veritas_env.startswith("subdomains")
99
or (openshift_namespace == "wwp-test" and route_is_root_url) }}

0 commit comments

Comments
 (0)