Relocating panel to a different dashboard via Pluggable Settings causes TemplateDoesNotExist
Bug #1533910 reported by
Alan Fung
This bug affects 3 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| OpenStack Dashboard (Horizon) |
In Progress
|
Medium
|
Vishal Manchanda | ||
Bug Description
Steps to reproduce:
1. Move users panel to be under admin dashboard by modifying _3030_identity_
2. Set PANEL_GROUP = 'admin'
3. Set PANEL_DASHBOARD = 'admin'
Expected:
1. Users panel to be under Admin panel group in the navigation.
2. Users panel to load.
Actual:
1. Users panel under Admin panel group in the navigation.
2. Users panel would generate a TemplateDoesNot
Observations:
This issue is due to Dashboard.register populates the panel_template_dirs for horizon template loader with the key "admin/users" for the Identity Users panel. However, TemplateLoader.
| description: | updated |
Revision history for this message
| Alan Fung (alanfung) wrote | #1 |
| Changed in horizon: | |
| status: | New → Confirmed |
| importance: | Undecided → High |
| milestone: | none → next |
| Changed in horizon: | |
| assignee: | nobody → Alan Fung (alanfung) |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix proposed to horizon (master) | #2 |
| Changed in horizon: | |
| status: | Confirmed → In Progress |
Revision history for this message
| ST9527 (st9527-cs229) wrote | #3 |
| Changed in horizon: | |
| assignee: | Alan Fung (alanfung) → Richard Jones (r1chardj0n3s) |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Change abandoned on horizon (master) | #4 |
| description: | updated |
Revision history for this message
| Akihiro Motoki (amotoki) wrote | #5 |
| Changed in horizon: | |
| assignee: | Richard Jones (r1chardj0n3s) → nobody |
| Changed in horizon: | |
| status: | In Progress → Confirmed |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix proposed to horizon (master) | #6 |
| Changed in horizon: | |
| assignee: | nobody → Vishal Manchanda (vishalmanchanda) |
| status: | Confirmed → In Progress |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Change abandoned on horizon (master) | #7 |
Revision history for this message
| Akihiro Motoki (amotoki) wrote | #8 |
| Changed in horizon: | |
| importance: | High → Medium |
To post a comment you must log in.
As a suggestion to address this, we can modify Dashboard.register to generate the key from panel_dir (directory of the panel) instead of appending cls.slug and panel.slug.
Both approaches to generate the key make assumptions about the file layout, but the suggested is less fallible and compatible with (what I presume) to be a function of Pluggable Settings, which is the flexibility of moving panel to any dashboard.