Bug 1444398 - [Ansible Embedded] - Editing Name of Amazon Credentials is not possible without filling keys again
Summary: [Ansible Embedded] - Editing Name of Amazon Credentials is not possible witho...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.8.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.9.0
Assignee: Marcel Hild
QA Contact: Pavol Kotvan
URL:
Whiteboard: ansible_embed:ui
Depends On:
Blocks: 1445389
TreeView+ depends on / blocked
 
Reported: 2017-04-21 09:32 UTC by Pavol Kotvan
Modified: 2018-03-27 14:38 UTC (History)
12 users (show)

Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1445389 (view as bug list)
Environment:
Last Closed: 2018-03-06 15:28:29 UTC
Category: Bug
Cloudforms Team: Ansible
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Comment 2 Milan Zázrivec 2017-04-24 14:40:41 UTC
The error shown in the attachment of this report is really a backend problem. Reassigning.

Comment 3 James Wong 2017-04-24 16:42:58 UTC
Pavol,

I think the update did succeed.  Can you do a refresh and confirm that?

thanks,
James

Comment 4 Marcel Hild 2017-04-25 04:58:30 UTC
fixed in ansible_tower_client https://github.com/ansible/ansible_tower_client_ruby/pull/92

Comment 5 Marcel Hild 2017-04-25 04:59:21 UTC
and release via 0.12.1 https://github.com/ansible/ansible_tower_client_ruby/commit/e33587b38644f6a1a6f2124ead2722c3e9df9637

so it should make it into the next build

Comment 9 Marcel Hild 2017-04-25 13:23:06 UTC
Milan, can you have a look at Pavols screenshots? Looks like the ec2 credentials should be prefilled with ****, like the other.

Maybe the error in the attached log is unrelated (but fixed anyway)

Comment 10 Milan Zázrivec 2017-04-25 13:59:21 UTC
The problem here is that the UI form we're looking at works with our
rest API. And that API does not return this piece of information
for the given credential resource. I'm not even sure if we store this
info in ManageIQ DB or whether it's contained just in Ansible.

Nevertheless, we're not rendering anything in the UI, since there's
basically nothing to render.

Comment 11 James Wong 2017-04-25 14:15:12 UTC
Milan,

The API_ATTRIBUTES give you some hint by :type => :password

:security_token => {
      :type       => :password,
      :label      => N_('STS Token'),
      :help_text  => N_('Security Token Service(STS) Token for this credential'),
      :max_length => 1024
    }

See if you can make use of that.

James

Comment 14 Milan Zázrivec 2017-04-26 06:08:04 UTC
(In reply to James Wong from comment #11)
> Milan,
> 
> The API_ATTRIBUTES give you some hint by :type => :password
> 
> :security_token => {
>       :type       => :password,
>       :label      => N_('STS Token'),
>       :help_text  => N_('Security Token Service(STS) Token for this
> credential'),
>       :max_length => 1024
>     }
> 
> See if you can make use of that.

Yes. I am making use of that. Though this is unrelated to the problem
we're discussing.

Comment 16 James Wong 2017-04-26 13:03:34 UTC
Jillian,

Can you check with Alberto to see if API can unpack the options field from credential and send along to UI?

Thanks,
James

Comment 17 James Wong 2017-04-26 13:18:02 UTC
Milan,

So API does send you the 'options' attribute of the credential(authentication). It's a hash containing these extra attributes there.  Can you see UI can make use of that?

thanks,
James

Pavol, so this BZ is marked a POST with Marcel's fix. Do you mind create another bug tract this 'original' issue you are reporting?

Comment 18 Milan Zázrivec 2017-04-26 13:23:37 UTC
(In reply to James Wong from comment #17)
> Milan,
> 
> So API does send you the 'options' attribute of the
> credential(authentication). It's a hash containing these extra attributes
> there.  Can you see UI can make use of that?

UI is making use of those options.

Comment 19 James Wong 2017-04-26 13:47:00 UTC
Now I see the problem:

The value of `usename` (described in API_ATTRIBUTES) is not stored in `options`, but in native attribute userid and UI has no info to link the two.


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