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: ---------------
Decsription should read: I noticed that there was an unwanted comma in the disk *type* string for Azure instances.
https://github.com/ManageIQ/manageiq-ui-classic/pull/3373
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(-)
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".
verified in 5.10.0.4