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

Enable scm_credential type in refresh #14471

Merged
merged 1 commit into from
Mar 23, 2017

Conversation

jameswnl
Copy link
Contributor

The scm_credential was not being inventoried with most accurate class type.

Also fixing a test shared_examples being directed to wrong file.

@jameswnl
Copy link
Contributor Author

@miq-bot add_labels wip, enhancement, providers/ansible_tower

@miq-bot
Copy link
Member

miq-bot commented Mar 23, 2017

Checked commit jameswnl@fcea79e with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
5 files checked, 1 offense detected

app/models/manageiq/providers/ansible_tower/shared/inventory/parser/automation_manager.rb

@jameswnl jameswnl changed the title [WIP] Enable scm_credential type in refresh Enable scm_credential type in refresh Mar 23, 2017
@jameswnl
Copy link
Contributor Author

@miq-bot remove_label wip

@jameswnl
Copy link
Contributor Author

@durandom help take a quick look. UI is blocked on this.

Copy link
Member

@durandom durandom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 Looks great

@@ -1,7 +1,7 @@
require 'support/ansible_shared/automation_manager/refresher'
require 'support/ansible_shared/automation_manager/refresher_v2'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wow... i.e. we never tested the _v2 refresh?!

Copy link
Contributor Author

@jameswnl jameswnl Mar 23, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we did at one point 😬

@blomquisg blomquisg merged commit 348fc2f into ManageIQ:master Mar 23, 2017
@blomquisg blomquisg added this to the Sprint 57 Ending Mar 27, 2017 milestone Mar 23, 2017
@jameswnl jameswnl deleted the scm-cred-enabled branch March 24, 2017 15:56
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.

5 participants