AVAILABLE_REGIONS setting name gets confused with regions within Keystone service catalog

Bug #1494251 reported by Timur Sufiev
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
In Progress
Low
Timur Sufiev

Bug Description

Currently we have 2 different kinds of regions: the service regions inside Keystone catalog and AVAILABLE_REGIONS setting inside
Horizon, yet use the same name 'regions' for both of them. That creates a lot of confusion when solving some region-related issues at the Horizon/Keystone junction, even explaining what is exactly being broken poses a serious challenge when our common language has such a flaw!

I propose to invent 2 distinct terms for these entities, so at least we won't be terminologically challenged when fixing the related bugs.

According to the discussion in ML (see http://lists.openstack.org/pipermail/openstack-dev/2015-July/068917.html), the name 'AVAILABLE_KEYSTONE_ENDPOINTS' was chosen as a proper replacement.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

Changed in horizon:
assignee: nobody → Timur Sufiev (tsufiev-x)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by Timur Sufiev (<email address hidden>) on branch: master
Review: https://review.openstack.org/222131
Reason: Change-id should not be the same as dependency commit

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to horizon (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/224759

Timur Sufiev (tsufiev-x)
Changed in horizon:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by Timur Sufiev (<email address hidden>) on branch: master
Review: https://review.openstack.org/222132
Reason: Moved into https://review.openstack.org/#/c/224759/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by David Lyle (<email address hidden>) on branch: master
Review: https://review.openstack.org/224759
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Rob Cresswell (<email address hidden>) on branch: master
Review: https://review.openstack.org/224759
Reason: Due to inactivity and merge conflict

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

Other bug subscribers