78251b0304
We need to specify either external_openstack_tenant_name or
external_openstack_tenant_id. Those values were checked by seeing they
are defined or they have actual values separately.
However those values are always defined because of the following code
of openstack/defaults/main.yml:
external_openstack_tenant_id: "{{ lookup('env','OS_TENANT_ID')| default(lookup('env','OS_PROJECT_ID'),true) }}"
external_openstack_tenant_name: "{{ lookup('env','OS_TENANT_NAME')| default(lookup('env','OS_PROJECT_NAME'),true) }}"
So even if not specifying both values, those checks could not detect
the misconfiguration. This fixes this to detect the misconfiguration.
27 lines
1.3 KiB
YAML
27 lines
1.3 KiB
YAML
---
|
|
- name: External OpenStack Cloud Controller | check external_openstack_auth_url value
|
|
fail:
|
|
msg: "external_openstack_auth_url is missing"
|
|
when: external_openstack_auth_url is not defined or not external_openstack_auth_url
|
|
|
|
- name: External OpenStack Cloud Controller | check external_openstack_username value
|
|
fail:
|
|
msg: "external_openstack_username is missing"
|
|
when: external_openstack_username is not defined or not external_openstack_username
|
|
|
|
- name: External OpenStack Cloud Controller | check external_openstack_password value
|
|
fail:
|
|
msg: "external_openstack_password is missing"
|
|
when: external_openstack_password is not defined or not external_openstack_password
|
|
|
|
- name: External OpenStack Cloud Controller | check external_openstack_region value
|
|
fail:
|
|
msg: "external_openstack_region is missing"
|
|
when: external_openstack_region is not defined or not external_openstack_region
|
|
|
|
- name: External OpenStack Cloud Controller | check external_openstack_tenant_id value
|
|
fail:
|
|
msg: "one of external_openstack_tenant_id or external_openstack_tenant_name must be specified"
|
|
when:
|
|
- (external_openstack_tenant_id is not defined or not external_openstack_tenant_id) and
|
|
(external_openstack_tenant_name is not defined or not external_openstack_tenant_name)
|