Setting WEBROOT does not change static paths

Bug #1434276 reported by Adam Young
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
High
David Lyle

Bug Description

setting WEBROOT as per commit 2a772d64800070689478a49b6aa26a5893b2067b only changes the paths for the dynamic pages.

I tried just adding STATIC_URL = WEBROOT + 'static/' in
/opt/stack/horizon/openstack_dashboard/local/local_settings.py But apache remoes the /dashboard/ portion and the result was 404.

In order to make the static pages (such as CSS) appear, I had to add the following to /etc/httpd/conf.d/horizon.conf

    Alias /dashboard/static /opt/stack/horizon/static

David Lyle (david-lyle)
Changed in horizon:
status: New → Confirmed
importance: Undecided → High
milestone: none → kilo-rc1
assignee: nobody → David Lyle (david-lyle)
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/166857

Changed in horizon:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/166857
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=18dddd97da19af894a0974e3560197036ea8969a
Submitter: Jenkins
Branch: master

commit 18dddd97da19af894a0974e3560197036ea8969a
Author: David Lyle <email address hidden>
Date: Mon Mar 23 08:26:01 2015 -0600

    Fixing changing WEBROOT from /

    There were a few items missed with the patch enabling changing
    WEBROOT patch.

    First the XStatic files were being unpacked in the wrong directoy
    because root_url was not being passed in. This caused 404 errors
    when running with a different web root.

    This change allows root_url to be passed in on XStatic handling.

    Second, STATIC_URL and MEDIA_URL were not using WEBROOT.
    These now are built with WEBROOT.

    Third, the fa-font-path variable was hard-coded in _variables.scss
    to use "/" as the web root.

    A webroot variable has been added that can be overridden. Additionally,
    a new theme has also been added to change the web root.

    Closes-Bug: #1434276
    Change-Id: Ifff1c60b3405ce2136d039b8d74da08752e7215c

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: kilo-rc1 → 2015.1.0
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.