[Horizon] Volumes names displays with escaped symbols

Bug #1597751 reported by Sergii Turivnyi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Status tracked in 10.0.x
10.0.x
Confirmed
Medium
MOS Horizon
9.x
Fix Released
Medium
Timur Sufiev
OpenStack Dashboard (Horizon)
Fix Released
Low
Unassigned

Bug Description

Detailed bug description:
Volumes names displays with escaped symbols on Volume Snapshots tab

Steps to reproduce:
1) Deploy MOS ISO #405 http://paste.openstack.org/show/524258/
2) Creare volume with name """'"
3) Create volume snapshot with name """'"
4) Navigate to the Volume Snapshots tab

Expected results:
Volume name is """'"

Actual result:
Volume name is """'"

Reproducibility:
100%

Workaround:
Do not use names like this=)

Description of the environment:
http://paste.openstack.org/show/524258/

Additional information:
See attachment

Tags: area-horizon
Revision history for this message
Sergii Turivnyi (sturivnyi) wrote :
Changed in mos:
assignee: nobody → MOS Horizon (mos-horizon)
Changed in mos:
status: New → Confirmed
Timur Sufiev (tsufiev-x)
summary: - [Hirozon] Volumes names displays with escaped symbols
+ [Horizon] Volumes names displays with escaped symbols
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/horizon (9.0/mitaka)

Fix proposed to branch: 9.0/mitaka
Change author: Timur Sufiev <email address hidden>
Review: https://review.fuel-infra.org/23678

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/horizon (9.0/mitaka)

Reviewed: https://review.fuel-infra.org/23678
Submitter: Pkgs Jenkins <email address hidden>
Branch: 9.0/mitaka

Commit: 9acc56e0635d3bec1684717f6c6db376c7bf2054
Author: Timur Sufiev <email address hidden>
Date: Tue Jul 26 12:56:04 2016

Remove unnecessary escaping of volume name

Variable contents in Django templates are escaped by default, no need
to do it explicitly.

Change-Id: Icdcdd403f4f401adc6f0326b4241a0d9346a8541
Closes-Bug: #1597751

tags: added: on-verification
Revision history for this message
Ekaterina Shutova (eshutova) wrote :

Volume name is """'"
Verified on:
cat /etc/fuel_build_id:
 495
cat /etc/fuel_build_number:
 495
cat /etc/fuel_release:
 9.0
cat /etc/fuel_openstack_version:
 mitaka-9.0
rpm -qa | egrep 'fuel|astute|network-checker|nailgun|packetary|shotgun':
 fuel-release-9.0.0-1.mos6359.noarch
 fuel-bootstrap-cli-9.0.0-1.mos291.noarch
 fuel-notify-9.0.0-1.mos8642.noarch
 fuel-nailgun-9.0.0-1.mos8911.noarch
 network-checker-9.0.0-1.mos77.x86_64
 fuel-ostf-9.0.0-1.mos947.noarch
 fuel-ui-9.0.0-1.mos2842.noarch
 shotgun-9.0.0-1.mos90.noarch
 fuel-utils-9.0.0-1.mos8642.noarch
 python-packetary-9.0.0-1.mos157.noarch
 fuel-library9.0-9.0.0-1.mos8642.noarch
 fuel-migrate-9.0.0-1.mos8642.noarch
 python-fuelclient-9.0.0-1.mos363.noarch
 fuelmenu-9.0.0-1.mos276.noarch
 fuel-mirror-9.0.0-1.mos157.noarch
 fuel-provisioning-scripts-9.0.0-1.mos8911.noarch
 fuel-openstack-metadata-9.0.0-1.mos8911.noarch
 rubygem-astute-9.0.0-1.mos782.noarch
 fuel-9.0.0-1.mos6359.noarch
 fuel-agent-9.0.0-1.mos291.noarch
 fuel-misc-9.0.0-1.mos8642.noarch
 nailgun-mcagents-9.0.0-1.mos782.noarch
 fuel-setup-9.0.0-1.mos6359.noarch

tags: removed: on-verification
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/horizon (mcp/newton)

Fix proposed to branch: mcp/newton
Change author: Timur Sufiev <email address hidden>
Review: https://review.fuel-infra.org/33880

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/horizon (11.0/ocata)

Fix proposed to branch: 11.0/ocata
Change author: Timur Sufiev <email address hidden>
Review: https://review.fuel-infra.org/33916

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/horizon (mcp/ocata)

Fix proposed to branch: mcp/ocata
Change author: Timur Sufiev <email address hidden>
Review: https://review.fuel-infra.org/34707

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/horizon (11.0/ocata)

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: 11.0/ocata
Review: https://review.fuel-infra.org/33916
Reason: 11.0/ocata is obsolete. We use mcp/ocata instead.

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/horizon (mcp/ocata)

Reviewed: https://review.fuel-infra.org/34707
Submitter: Pkgs Jenkins <email address hidden>
Branch: mcp/ocata

Commit: 91d05145773e6dbb8dced7fe04cf68da629d0667
Author: Timur Sufiev <email address hidden>
Date: Wed Apr 26 15:44:18 2017

Remove unnecessary escaping of volume name

Variable contents in Django templates are escaped by default, no need
to do it explicitly.

Change-Id: Icdcdd403f4f401adc6f0326b4241a0d9346a8541
Closes-Bug: #1597751

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/horizon (mcp/newton)

Reviewed: https://review.fuel-infra.org/33880
Submitter: Pkgs Jenkins <email address hidden>
Branch: mcp/newton

Commit: be19c5052ad6b2a7eb8a2acfe4bc18a3c5c469ce
Author: Timur Sufiev <email address hidden>
Date: Mon May 8 12:23:36 2017

Remove unnecessary escaping of volume name

Variable contents in Django templates are escaped by default, no need
to do it explicitly.

Change-Id: Icdcdd403f4f401adc6f0326b4241a0d9346a8541
Closes-Bug: #1597751

Changed in horizon:
milestone: none → pike-2
status: New → Fix Released
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

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

commit d2b1a9cbcc10e0b34f4715472bc58189cfe4013e
Author: Timur Sufiev <email address hidden>
Date: Tue Jul 26 15:41:22 2016 +0300

    Remove unnecessary escaping of volume name

    Variable contents in Django templates are escaped by default, no need
    to do it explicitly.

    Change-Id: I0b9619b7b1605bcdb17fe3b7204bf6e6c2520454
    Closes-Bug: #1597751

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/horizon 12.0.0.0b2

This issue was fixed in the openstack/horizon 12.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.