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.

Sign up for GitHub

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

Jump to bottom

An Ansible Tower "Inventory" is a ManageIQ "InventoryRootGroup" #14716

Merged
merged 1 commit into from
Apr 10, 2017
Merged

An Ansible Tower "Inventory" is a ManageIQ "InventoryRootGroup" #14716

merged 1 commit into from
Apr 10, 2017

Conversation

bdunne
Copy link
Member

@bdunne bdunne commented Apr 10, 2017

Related to #14691

@bdunne bdunne requested review from jameswnl and lgalis April 10, 2017 17:44
@miq-bot
Copy link
Member

miq-bot commented Apr 10, 2017

Checked commit bdunne@c4b96dc with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks good. 🍪

@Fryguy Fryguy merged commit feb614e into ManageIQ:master Apr 10, 2017
@Fryguy Fryguy added this to the Sprint 58 Ending Apr 10, 2017 milestone Apr 10, 2017
@bdunne bdunne deleted the fix_embedded_ansible_inventory_inheritence branch April 10, 2017 20:51
simaishi pushed a commit that referenced this pull request Apr 11, 2017
@Fryguy @simaishi
Merge pull request #14716 from bdunne/fix_embedded_ansible_inventory_…
c752a84 
…inheritence

An Ansible Tower "Inventory" is a ManageIQ "InventoryRootGroup"
(cherry picked from commit feb614e)
@simaishi
Copy link
Contributor

Fine backport details:

$ git log -1
commit c752a84a84df355c75a614369ae1707fef61521b
Author: Jason Frey <fryguy9@gmail.com>
Date:   Mon Apr 10 16:50:25 2017 -0400

    Merge pull request #14716 from bdunne/fix_embedded_ansible_inventory_inheritence
    
    An Ansible Tower "Inventory" is a ManageIQ "InventoryRootGroup"
    (cherry picked from commit feb614e342c44450e251edf5d22db3444a622aa5)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jameswnl jameswnl jameswnl approved these changes

@lgalis lgalis Awaiting requested review from lgalis

Assignees

@Fryguy Fryguy

Labels
bug fine/backported providers/automation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@bdunne @miq-bot @simaishi @jameswnl @Fryguy