Varaible for WEBSSO_INITIAL_CHOICE missing

Bug #1660322 reported by Ingemar Fällman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
Low
Duong Ha-Quang

Bug Description

WEBSSO_INITIAL_CHOICE should be set by a variable like horizon_websso_initial_choice that defaults to "credentials" in https://github.com/openstack/openstack-ansible-os_horizon/blob/master/templates/horizon_local_settings.py.j2#L178 and a new default value should be set in https://github.com/openstack/openstack-ansible-os_horizon/blob/master/defaults/main.yml

--- /etc/ansible/roles/os_horizon/templates/horizon_local_settings.py.j2.orig 2017-01-30 13:29:54.368304450 +0100
+++ /etc/ansible/roles/os_horizon/templates/horizon_local_settings.py.j2 2017-01-30 13:30:48.981640641 +0100
@@ -175,7 +175,7 @@
 WEBSSO_ENABLED = True

 # Determines which authentication choice to show as default.
-WEBSSO_INITIAL_CHOICE = "credentials"
+WEBSSO_INITIAL_CHOICE = "{{ horizon_websso_initial_choice }}"

 # The list of authentication mechanisms which include keystone
 # federation protocols and identity provider/federation protocol

--- /etc/ansible/roles/os_horizon/defaults/main.yml.orig 2017-01-30 13:32:07.805795654 +0100
+++ /etc/ansible/roles/os_horizon/defaults/main.yml 2017-01-30 13:34:56.349573132 +0100
@@ -288,3 +288,6 @@

 horizon_keystone_admin_roles:
   - admin
+
+# Set the "credentials" authentication choice to show as default.
+horizon_websso_initial_choice: "credentials"

tags: added: low-hanging-fruit
Changed in openstack-ansible:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible-os_horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/468845

Changed in openstack-ansible:
assignee: nobody → Duong Ha-Quang (duonghq)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible-os_horizon (master)

Reviewed: https://review.openstack.org/468845
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-os_horizon/commit/?id=1283febbb18f994d329ed83738ec7bd20aabd329
Submitter: Jenkins
Branch: master

commit 1283febbb18f994d329ed83738ec7bd20aabd329
Author: Duong Ha-Quang <email address hidden>
Date: Mon May 29 17:22:28 2017 +0700

    Move WEBSSO_INITIAL_CHOICE setting to variable

    Currently, WEBSSO_INITIAL_CHOICE is hard-coded in template, this patchset
    convert WEBSSO_INITIAL_CHOICE setting to variable horizon_websso_initial_choice
    in main.yml

    Change-Id: I7e18ec981f6a575ee81da1429545f8b46a771fb8
    Closes-bug: #1660322

Changed in openstack-ansible:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible-os_horizon 16.0.0.0b2

This issue was fixed in the openstack/openstack-ansible-os_horizon 16.0.0.0b2 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.