Bug 1536681

Summary: Unwanted comma in disk type string for Azure instances
Product: Red Hat CloudForms Management Engine Reporter: Nandini Chandra <nachandr>
Component: UI - OPSAssignee: Greg Piatigorski <gpiatigo>
Status: CLOSED CURRENTRELEASE QA Contact: Nandini Chandra <nachandr>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.8.0CC: cpelland, gblomqui, hkataria, jfrey, jhardy, lavenel, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.10.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.10.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1552886 1552887 (view as bug list) Environment:
Last Closed: 2019-02-11 14:01:12 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1552886, 1552887    
Attachments:
Description Flags
screen shot of Azure instance disk info none

Description Nandini Chandra 2018-01-19 22:28:40 UTC
Created attachment 1383617 [details]
screen shot of Azure instance disk info

Description of problem:
-----------------------
I noticed that there was an unwanted comma in disk size string for Azure instances while verifying https://bugzilla.redhat.com/show_bug.cgi?id=1505951

See attached screen shot.

Version-Release number of selected component (if applicable):
----------------------
5.8.3.1
5.9.0.17


How reproducible:
----------------
Always


Steps to Reproduce:
------------------
1.
2.
3.


Actual results:
--------------


Expected results:
-----------------


Additional info:
---------------

Comment 2 Nandini Chandra 2018-01-19 22:42:56 UTC
Decsription should read:

I noticed that there was an unwanted comma in the disk *type* string for Azure instances.

Comment 6 CFME Bot 2018-02-08 11:43:52 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/bb55ae034f721aad0ca7d0a99c1e4af3b8da295f

commit bb55ae034f721aad0ca7d0a99c1e4af3b8da295f
Author:     Greg Piatigorski <gpiatigo>
AuthorDate: Mon Feb 5 14:52:06 2018 -0800
Commit:     Greg Piatigorski <gpiatigo>
CommitDate: Wed Feb 7 14:52:45 2018 -0800

    Fixes Azure Instance display file name and mode descriptions
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1536681

 app/helpers/textual_mixins/textual_devices.rb      | 25 ++++++++++--------
 .../helpers/textual_mixins/textual_devices_spec.rb | 30 +++++++++++++++++++++-
 2 files changed, 44 insertions(+), 11 deletions(-)

Comment 7 Greg Piatigorski 2018-02-08 17:53:14 UTC
Note: Code refactored to display hard disk in the right hand cell, redundancy of provider name eliminated (only displayed on top of page, and all parameters not set/missing data will now show "N/A".

Comment 10 Nandini Chandra 2018-07-16 22:09:28 UTC
verified in 5.10.0.4