Many \n in 'warning' notification

Bug #1467893 reported by Anastasia Palkina
46
This bug affects 7 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Alexander Gubanov

Bug Description

1. Install ISO #13 for 7.0
2. Open Fuel Dashboard
3. Open Notifications (button "View all")
4. There are many \n in 'warning' notification (see screen)

"build_id": "2015-06-21_11-05-15", "build_number": "13", "release_versions": {"2014.2.2-7.0": {"VERSION": {"build_id": "2015-06-21_11-05-15", "build_number": "13", "api": "1.0", "fuel-library_sha": "f1fed1738972f77c6d7bac61f4211be5dba170e8", "nailgun_sha": "20eca6f5c003c990d2ace8f12d5b84cc89eba584", "feature_groups": ["mirantis"], "openstack_version": "2014.2.2-7.0", "production": "docker", "python-fuelclient_sha": "912f2a72cd279c54c52bb3c86cadc19575f711bc", "astute_sha": "776157f722b13aff5f59bc098cf948793e6498ef", "fuel-ostf_sha": "8614b6d8dfa594fa24802bb2273aa84f392587ef", "release": "7.0", "fuelmain_sha": "3b866d2ff3091a60362327028085fa62fd16c5a0"}}}, "auth_required": true, "api": "1.0", "fuel-library_sha": "f1fed1738972f77c6d7bac61f4211be5dba170e8", "nailgun_sha": "20eca6f5c003c990d2ace8f12d5b84cc89eba584", "feature_groups": ["mirantis"], "openstack_version": "2014.2.2-7.0", "production": "docker", "python-fuelclient_sha": "912f2a72cd279c54c52bb3c86cadc19575f711bc", "astute_sha": "776157f722b13aff5f59bc098cf948793e6498ef", "fuel-ostf_sha": "8614b6d8dfa594fa24802bb2273aa84f392587ef", "release": "7.0", "fuelmain_sha": "3b866d2ff3091a60362327028085fa62fd16c5a0"

Tags: area-qa
Revision history for this message
Anastasia Palkina (apalkina) wrote :
Changed in fuel:
status: New → Confirmed
Revision history for this message
Vitaly Kramskikh (vkramskikh) wrote :

Fuel UI handles newlines correctly. Seems to be nailgun issue

Changed in fuel:
assignee: Fuel UI Team (fuel-ui) → Fuel Python Team (fuel-python)
Revision history for this message
Vitaly Kramskikh (vkramskikh) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-fuelclient (master)

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

Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Alexander Gubanov (ogubanov)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-main (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on python-fuelclient (master)

Change abandoned by Alexander Gubanov (<email address hidden>) on branch: master
Review: https://review.openstack.org/200272
Reason: https://review.openstack.org/#/c/200414/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-main (master)

Reviewed: https://review.openstack.org/200414
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=0a8dfbd80b0f5464854c6eb3c692298864326f1b
Submitter: Jenkins
Branch: master

commit 0a8dfbd80b0f5464854c6eb3c692298864326f1b
Author: Alexander Gubanov <email address hidden>
Date: Fri Jul 10 11:06:24 2015 +0300

    Fix display of new line in notifications for Fuel UI

    The root cause is excess backslash in bash scripts,
    which is interpreted as escape character.

    Change-Id: I5a03853a273c6c7413d4c77c1ee122286387bd9e
    Closes-Bug: #1467893

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Verified on ISO #43

"build_id": "2015-07-16_04-50-59", "build_number": "43", "release_versions": {"2014.2.2-7.0": {"VERSION": {"build_id": "2015-07-16_04-50-59", "build_number": "43", "api": "1.0", "fuel-library_sha": "9796e2535307a5d9967e83cc7592c54078b11edb", "nailgun_sha": "bb362555dba04f461bb570aa414ab628e753d57c", "feature_groups": ["mirantis"], "openstack_version": "2014.2.2-7.0", "fuel-agent_sha": "ce4bfd0037f538cf929a92290fd211950e5cdf7d", "production": "docker", "python-fuelclient_sha": "fe14335efc3c800162b72d1e333e5179314e2ff4", "astute_sha": "3cc1bb6b2a7cc2b437c6c6ffc6366653b08749bf", "fuel-ostf_sha": "5b5ec07f15b1d15e452366e838c4c830010ee795", "release": "7.0", "fuelmain_sha": "533d1722016da9774fb406f86d35942e5870ed73"}}}, "auth_required": true, "api": "1.0", "fuel-library_sha": "9796e2535307a5d9967e83cc7592c54078b11edb", "nailgun_sha": "bb362555dba04f461bb570aa414ab628e753d57c", "feature_groups": ["mirantis"], "openstack_version": "2014.2.2-7.0", "fuel-agent_sha": "ce4bfd0037f538cf929a92290fd211950e5cdf7d", "production": "docker", "python-fuelclient_sha": "fe14335efc3c800162b72d1e333e5179314e2ff4", "astute_sha": "3cc1bb6b2a7cc2b437c6c6ffc6366653b08749bf", "fuel-ostf_sha": "5b5ec07f15b1d15e452366e838c4c830010ee795", "release": "7.0", "fuelmain_sha": "533d1722016da9774fb406f86d35942e5870ed73"

Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
Anastasia Palkina (apalkina) wrote :

I think we need to remove control symbols too because now text is not stretched across the length of the string
http://postimg.org/image/jttmwrm3d/

Changed in fuel:
status: Fix Released → Confirmed
Revision history for this message
Alexander Gubanov (ogubanov) wrote :
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Alexander, I think that the first variant is better because other long messages in notifications are stretched across the length of the string.

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

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

Changed in fuel:
status: Confirmed → In Progress
tags: removed: ui
Changed in fuel:
assignee: Alexander Gubanov (ogubanov) → Aleksey Zvyagintsev (azvyagintsev)
Changed in fuel:
assignee: Aleksey Zvyagintsev (azvyagintsev) → nobody
Changed in fuel:
assignee: nobody → Alexander Gubanov (ogubanov)
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Move to 8.0 because:

1) it is medium bug
2
) issue was partially fixed

Changed in fuel:
milestone: 7.0 → 8.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-main (master)

Reviewed: https://review.openstack.org/202774
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=7c2dcb0b8bafff7b53e6f497c10f8f7c967ad535
Submitter: Jenkins
Branch: master

commit 7c2dcb0b8bafff7b53e6f497c10f8f7c967ad535
Author: Alexander Gubanov <email address hidden>
Date: Thu Jul 16 23:31:19 2015 +0300

    Fix display of new line in notifications for Fuel UI

    Closes-Bug: #1467893
    Change-Id: If2188fad7c5790af804edd3227b36815e5e56c53

Changed in fuel:
status: In Progress → Fix Committed
Dmitry Pyzhov (dpyzhov)
tags: added: area-qa
Changed in fuel:
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.