Merge "keystone: ensure bool for two parameters"

This commit is contained in:
Zuul 2023-09-18 12:26:24 +00:00 committed by Gerrit Code Review
commit 65079b4dc1
4 changed files with 10 additions and 3 deletions

View File

@ -82,7 +82,7 @@ connection_string = {{ osprofiler_backend_connection_string }}
allowed_origin = {{ grafana_public_endpoint }}
{% endif %}
{% if enable_keystone_federation %}
{% if enable_keystone_federation | bool %}
[federation]
{% for dashboard in keystone_trusted_dashboards %}
trusted_dashboard = {{ dashboard }}

View File

@ -78,7 +78,7 @@
{
"path": "/var/log/kolla/keystone/keystone.log",
"owner": "keystone:keystone"
},{% if keystone_enable_federation_openid %}
},{% if keystone_enable_federation_openid | bool %}
{
"path": "{{ keystone_container_federation_oidc_metadata_folder }}",
"owner": "{{ apache_user }}:{{ apache_user }}",

View File

@ -62,7 +62,7 @@ LogLevel info
SSLCertificateKeyFile /etc/keystone/certs/keystone-key.pem
{% endif -%}
{% if keystone_enable_federation_openid %}
{% if keystone_enable_federation_openid | bool %}
OIDCClaimPrefix "OIDC-"
OIDCClaimDelimiter ";"
OIDCResponseType "{{ keystone_federation_oidc_response_type }}"

View File

@ -0,0 +1,7 @@
---
fixes:
- |
``enable_keystone_federation`` and ``keystone_enable_federation_openid``
have not been explicitly handled as bool in various templates in the
keystone role so far.
`LP#2036390 <https://bugs.launchpad.net/kolla-ansible/+bug/2036390>`__