From 947aac114fa6b674688ed0ab86ef40057caac523 Mon Sep 17 00:00:00 2001
From: Mohammed Naser <mnaser@vexxhost.com>
Date: Fri, 25 Jun 2021 13:14:49 -0400
Subject: [PATCH] Fix multidomain support selector

At the moment, the multidomain support selector is broken because
it always puts the value of a boolean inside a string which always
evaluates to true, which means setting it to false does nothing.

This patch drops the quotes around the templated configuration,
that way, it is taken for the literal boolean value.

Change-Id: I02c0a0ece680ecb55e83f3da5a992398c3ab6390
---
 horizon/Chart.yaml              | 2 +-
 horizon/values.yaml             | 2 +-
 releasenotes/notes/horizon.yaml | 1 +
 3 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/horizon/Chart.yaml b/horizon/Chart.yaml
index 81a60b44ce..0defb8e879 100644
--- a/horizon/Chart.yaml
+++ b/horizon/Chart.yaml
@@ -14,7 +14,7 @@ apiVersion: v1
 appVersion: v1.0.0
 description: OpenStack-Helm Horizon
 name: horizon
-version: 0.2.3
+version: 0.2.4
 home: https://docs.openstack.org/horizon/latest/
 icon: https://www.openstack.org/themes/openstack/images/project-mascots/Horizon/OpenStack_Project_Horizon_vertical.png
 sources:
diff --git a/horizon/values.yaml b/horizon/values.yaml
index ddb9de5c06..da824060cc 100644
--- a/horizon/values.yaml
+++ b/horizon/values.yaml
@@ -300,7 +300,7 @@ conf:
 
         # Set this to True if running on multi-domain model. When this is enabled, it
         # will require user to enter the Domain name in addition to username for login.
-        OPENSTACK_KEYSTONE_MULTIDOMAIN_SUPPORT = '{{ .Values.conf.horizon.local_settings.config.keystone_multidomain_support }}'
+        OPENSTACK_KEYSTONE_MULTIDOMAIN_SUPPORT = {{ .Values.conf.horizon.local_settings.config.keystone_multidomain_support }}
 
         # Overrides the default domain used when running on single-domain model
         # with Keystone V3. All entities will be created in the default domain.
diff --git a/releasenotes/notes/horizon.yaml b/releasenotes/notes/horizon.yaml
index ccc442a521..0aefb50d79 100644
--- a/releasenotes/notes/horizon.yaml
+++ b/releasenotes/notes/horizon.yaml
@@ -13,4 +13,5 @@ horizon:
   - 0.2.1 Make python script PEP8 compliant
   - 0.2.2 Use policies in yaml format
   - 0.2.3 Add openstack_enable_password_retrieve variable in value
+  - 0.2.4 Fix OPENSTACK_KEYSTONE_MULTIDOMAIN_SUPPORT value
 ...