Not works correctly hyperlink on the horizon if it is in the browser cache

Bug #1385133 reported by okosse
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Sergii Golovatiuk
6.1.x
Won't Fix
Medium
Vladimir Kuklin
7.0.x
Won't Fix
Medium
Sergii Golovatiuk
8.0.x
Fix Released
Medium
Sergii Golovatiuk
puppet-horizon
Fix Released
Undecided
Alex Schultz

Bug Description

After deployment of openstack with fuel, we get the hyperlink to access the horizon in the form of http://ipaddress . But the full address for the horizon is different. And this is different for Ubuntu (/horizon) and CentOS(/dashboard). When the first time the firefox cache gets full reference and further uses it. If I change the type of deployment, I will get a 404 error. I think a good solution would be dynamic code full address of the horizon, for example for CentOS:
<a href="http://10.20.0.3/dashboard" target="_blank">
http://10.20.0.3/
</a>

Changed in fuel:
importance: Undecided → Medium
milestone: none → 6.1
milestone: 6.1 → 6.0
assignee: nobody → Fuel UI Team (fuel-ui)
Revision history for this message
Julia Aranovich (jkirnosova) wrote :

No ui code changes required to change the link to "http://<ip>/dashboard" or "http://<ip>/horizon". UI simply displays deployment task message.

Revision history for this message
Vitaly Kramskikh (vkramskikh) wrote :

I think this issue should be solved by reconfiguring nginx for horizon to prevent caching or redirect using 302 code instead of 301 (if it is 301)

Changed in fuel:
assignee: Fuel UI Team (fuel-ui) → Fuel Library Team (fuel-library)
Changed in fuel:
status: New → Triaged
tags: added: low-hanging-fruit
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Przemyslaw Kaminski (pkaminski)
status: Triaged → In Progress
Revision history for this message
Łukasz Oleś (loles) wrote :

medium bug moved to 6.1

Changed in fuel:
milestone: 6.0 → 6.1
Revision history for this message
Przemyslaw Kaminski (pkaminski) wrote :

Moving to fuel-library team since I don't have time for it right now and my patschet was broken by new Puppet manifests for new Horizon.

Changed in fuel:
assignee: Przemyslaw Kaminski (pkaminski) → Fuel Library Team (fuel-library)
Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Vladimir Kuklin (vkuklin)
Changed in fuel:
milestone: 6.1 → 7.0
tags: removed: ui
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-library (master)

Change abandoned by Igor Shishkin (<email address hidden>) on branch: master
Review: https://review.openstack.org/132543
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.

Changed in fuel:
assignee: Vladimir Kuklin (vkuklin) → Bogdan Dobrelya (bogdando)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-horizon (master)

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

Changed in puppet-horizon:
assignee: nobody → Sergii Golovatiuk (sgolovatiuk)
status: New → In Progress
Changed in puppet-horizon:
assignee: Sergii Golovatiuk (sgolovatiuk) → Alex Schultz (alex-schultz)
Changed in fuel:
assignee: Sergii Golovatiuk (sgolovatiuk) → Alex Schultz (alex-schultz)
Revision history for this message
Bartłomiej Piotrowski (bpiotrowski) wrote :

Moving to 8.0 due to SCF.

Changed in puppet-horizon:
assignee: Alex Schultz (alex-schultz) → Sergii Golovatiuk (sgolovatiuk)
Changed in puppet-horizon:
assignee: Sergii Golovatiuk (sgolovatiuk) → Alex Schultz (alex-schultz)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-horizon (master)

Reviewed: https://review.openstack.org/208462
Committed: https://git.openstack.org/cgit/openstack/puppet-horizon/commit/?id=8de1b6100bd325e09b7964ba910183237b36b779
Submitter: Jenkins
Branch: master

commit 8de1b6100bd325e09b7964ba910183237b36b779
Author: Sergii Golovatiuk <email address hidden>
Date: Mon Aug 3 14:30:27 2015 +0200

    Make redirect type configurable

    This change exposes a redirect type configuration variable so that a
    user can specify if they want to use 'temp' or 'permanent' redirects.

    This change allows a user to switch to use a 'temp' redirect type
    because Mozilla Firefox and Google Chrome cache the responses of '301
    Moved Permanently'. Some users have multi tenant environments that have
    different URLs for Horizon dashboards (Ubuntu - /horizon, CentOS -
    dashboard). '302 Found' is not cacheable response which solves the issue
    for multitenant users.

    Change-Id: I81694d0ddeebafec7824f5b275d5fb504cd302b1
    Closes-Bug: 1385133
    Co-Authored-By: Alex Schultz <email address hidden>

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

Reviewed: https://review.openstack.org/132543
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=11e87de0c8dcb529480c1766b08df8f6e22497eb
Submitter: Jenkins
Branch: master

commit 11e87de0c8dcb529480c1766b08df8f6e22497eb
Author: Przemyslaw Kaminski <email address hidden>
Date: Mon Nov 3 11:56:02 2014 +0100

    Make redirect type configurable and set to temp

    This change exposes a redirect type configuration variable so that a
    user can specify if they want to use 'temp' or 'permanent' redirects.

    This change allows a user to switch to use a 'temp' redirect type
    because Mozilla Firefox and Google Chrome cache the responses of '301
    Moved Permanently'. Some users have multi tenant environments that have
    different URLs for Horizon dashboards (Ubuntu - /horizon, CentOS -
    dashboard). '302 Found' is not cacheable response which solves the issue
    for multitenant users.

    Upstream Change-Id: I81694d0ddeebafec7824f5b275d5fb504cd302b1

    Change-Id: Ib92685283e64100351af60f18eea393850665787
    Closes-Bug: #1385133
    Signed-off-by: Sergii Golovatiuk <email address hidden>
    Co-Authored-By: Alex Schultz <email address hidden>
    Signed-off-by: Sergii Golovatiuk <email address hidden>

Changed in fuel:
status: In Progress → Fix Committed
Dmitry Pyzhov (dpyzhov)
tags: added: area-library
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 7.0 → 8.0
tags: added: on-verification
Revision history for this message
Nastya Urlapova (aurlapova) wrote :

Verified on 8.0 - #478.

Matt Fischer (mfisch)
Changed in puppet-horizon:
status: Fix Committed → Fix Released
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.