Bug 1458928 - Blank fields in ansible credential summary screen
Summary: Blank fields in ansible credential summary screen
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.8.0
Hardware: All
OS: All
medium
medium
Target Milestone: GA
: 5.8.3
Assignee: Milan Zázrivec
QA Contact: Dmitry Misharov
URL:
Whiteboard: ui:ansible
Depends On: 1439797
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-06-05 20:57 UTC by Satoe Imaishi
Modified: 2017-12-14 15:17 UTC (History)
9 users (show)

Fixed In Version: 5.8.1.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1439797
Environment:
Last Closed: 2017-12-14 15:17:09 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Empty "password" field (66.57 KB, image/png)
2017-06-28 14:51 UTC, Dmitry Misharov
no flags Details

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.


Note You need to log in before you can comment on or make changes to this bug.