Bug 1458928

Summary: Blank fields in ansible credential summary screen
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: UI - OPSAssignee: Milan Zázrivec <mzazrivec>
Status: CLOSED CURRENTRELEASE QA Contact: Dmitry Misharov <dmisharo>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.8.0CC: agrare, cpelland, hkataria, jfrey, jhardy, mpovolny, obarenbo, pakotvan, simaishi
Target Milestone: GAKeywords: ZStream
Target Release: 5.8.3   
Hardware: All   
OS: All   
Whiteboard: ui:ansible
Fixed In Version: 5.8.1.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1439797 Environment:
Last Closed: 2017-12-14 15:17:09 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1439797    
Bug Blocks:    
Attachments:
Description Flags
Empty "password" field none

Comment 2 CFME Bot 2017-06-05 20:58:30 UTC
New commit detected on ManageIQ/manageiq-ui-classic/fine:
https://github.com/ManageIQ/manageiq-ui-classic/commit/4addda796e63a2d145ab918312c5997be857aff4

commit 4addda796e63a2d145ab918312c5997be857aff4
Author:     Martin Povolny <mpovolny>
AuthorDate: Wed May 3 22:06:47 2017 +0200
Commit:     Satoe Imaishi <simaishi>
CommitDate: Mon Jun 5 16:57:08 2017 -0400

    Merge pull request #1068 from mzazrivec/dont_display_username_and_passwords_twice
    
    Credentials summary: don't display username & password twice
    (cherry picked from commit 3577152dd3d2115007141362e11b899ad885c2a2)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1458928

 app/helpers/ansible_credential_helper/textual_summary.rb | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

Comment 3 Dmitry Misharov 2017-06-28 14:51:00 UTC
Created attachment 1292668 [details]
Empty "password" field

Redundant "User" field removed, but "Password" field still is always empty. It's not clear is "password" field filled or not.

Comment 5 Milan Zázrivec 2017-10-30 14:32:30 UTC
Regarding the password field not being set -- this is how things are done on the backend (model)
side. Even if the password for that particular credential resource is set, the model won't
show it. So as far as UI goes, there's not much we can do about it here. Not unless
backend provides some way for the UI to see that the password has been set (please
feel free to file a bz).

I'm moving this back to ON_QA.

Comment 6 Dmitry Misharov 2017-10-31 14:54:46 UTC
Verified in 5.8.2.3.20171016155816_aaec796.