Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove satellite6 credential type from embedded ansible space #16663

Merged
merged 1 commit into from
Dec 19, 2017

Conversation

jameswnl
Copy link
Contributor

@jameswnl jameswnl commented Dec 13, 2017

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1525172

Depends on ManageIQ/manageiq-providers-ansible_tower#46

After deploying both PR, I added an external Tower and refresh will still inventory the Satellite6Credential as follow db output,

vmdb_production=# select name, type, resource_type, resource_id from authentications where type like '%Satellite%';
      name      |                                    type                                    |    resource_type    | resource_id
----------------+----------------------------------------------------------------------------+---------------------+-------------
 hello_sat_cred | ManageIQ::Providers::AnsibleTower::AutomationManager::Satellite6Credential | ExtManagementSystem |           5
 sat6_cred      | ManageIQ::Providers::AnsibleTower::AutomationManager::Satellite6Credential | ExtManagementSystem |           5
(2 rows)

And at the same time, Satellite6Credential type is gone from embedded one as shown here
screen shot 2017-12-13 at 6 06 35 pm

@jameswnl
Copy link
Contributor Author

@miq-bot add_labels bug, blocker, providers/ansible_tower

@blomquisg
Copy link
Member

@jameswnl can you add a screenshot that shows the Sat6 credential removed from the list for embedded ansible?

@blomquisg
Copy link
Member

LGTM ... the tower pr 46 is still being reviewed, but I think this one is good once that's merged.

@gmcculloug
Do you concur

@blomquisg blomquisg requested a review from gmcculloug December 13, 2017 23:43
@jameswnl
Copy link
Contributor Author

too graphic this review has become...

@gmcculloug
Copy link
Member

@blomquisg It's everything I dreamed it would be. 👍

Is the failing test because of the related PR or a change required here?

@blomquisg
Copy link
Member

Is the failing test because of the related PR or a change required here?

Yep. Once that's merged, I believe this should pass.

@miq-bot
Copy link
Member

miq-bot commented Dec 14, 2017

Checked commit jameswnl@07f6bd3 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🏆

@agrare agrare closed this Dec 19, 2017
@agrare agrare reopened this Dec 19, 2017
@agrare agrare merged commit a649e48 into ManageIQ:master Dec 19, 2017
@agrare agrare added this to the Sprint 76 Ending Jan 1, 2018 milestone Dec 19, 2017
simaishi pushed a commit that referenced this pull request Dec 19, 2017
remove satellite6 credential type from embedded ansible space
(cherry picked from commit a649e48)

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1527583
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit e3dea29b5b31cfda2dd1c1f5e810ed37c30095d7
Author: Adam Grare <[email protected]>
Date:   Tue Dec 19 09:35:21 2017 -0500

    Merge pull request #16663 from jameswnl/hide-sat6
    
    remove satellite6 credential type from embedded ansible space
    (cherry picked from commit a649e48674a1b0302732c6dbe653ab77209163c8)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1527583

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants