Bug 1439632 - Ansible Credentials type should have proper format
Summary: Ansible Credentials type should have proper format
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - Service
Version: 5.8.0
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: GA
: 5.9.0
Assignee: Allen W
QA Contact: Satyajit Bulage
URL:
Whiteboard: ui:ssui
Depends On:
Blocks: 1444871
TreeView+ depends on / blocked
 
Reported: 2017-04-06 10:45 UTC by Satyajit Bulage
Modified: 2018-03-06 14:42 UTC (History)
6 users (show)

Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1444871 (view as bug list)
Environment:
Last Closed: 2018-03-06 14:42:34 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Screenshot showing actual results (71.86 KB, image/png)
2017-04-06 10:45 UTC, Satyajit Bulage
no flags Details

Description Satyajit Bulage 2017-04-06 10:45:30 UTC
Created attachment 1269327 [details]
Screenshot showing actual results

Description of problem: On provisioned service summary page, Credentials Type is showing as "ManageIQ::Providers::EmbeddedAnsible::AutomationManager::MachineCredential" rather "Machine".

Version-Release number of selected component (if applicable):5.8.0.9-alpha2.20170404195944_1d7ece4


How reproducible:100%


Steps to Reproduce:
1. Login to SSUI
2. Navigate to Services
3. Click on successfully provisioned service.
4. On summary page see under provision sub-tab.

Actual results: Credentials Type is showing as "ManageIQ::Providers::EmbeddedAnsible::AutomationManager::MachineCredential"


Expected results: It should show as "Machine" for machine credentials.


Additional info:

Comment 2 Chris Kacerguis 2017-04-06 16:20:55 UTC
We are just displaying what we get, so going to send this to providers.  Greg, feel free to kick it back if I am wrong :)

Comment 3 James Wong 2017-04-17 17:02:41 UTC
Chris,

There's API_OPTIONS in the model class and can be more user friendly.

For e.g ManageIQ::Providers::EmbeddedAnsible::AutomationManager::MachineCredential,
API_OPTIONS = {
    :label      => N_('Machine'),
    :type       => 'machine',
    :attributes => API_ATTRIBUTES
  }.freeze


For e.g ManageIQ::Providers::EmbeddedAnsible::AutomationManager::AmazonCredential,
API_OPTIONS = {
    :type       => 'cloud',
    :label      => N_('Amazon'),
    :attributes => API_ATTRIBUTES
  }.freeze
  TOWER_KIND = 'aws'.freeze

Comment 4 Chris Kacerguis 2017-04-19 17:20:11 UTC
Currently, blocked by: https://github.com/ManageIQ/manageiq/pull/14783

Hi Alberto - Any update on when this PR will get in?

Comment 9 Satyajit Bulage 2018-01-22 19:31:02 UTC
Able to see Credentials Type as Machine instead of long name.

Verified Version: 5.9.0.17.20180116225234_ac8b6f5


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