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

catalog item accepts remove_resources option #14328

Merged
merged 1 commit into from
Mar 15, 2017

Conversation

bzwei
Copy link
Contributor

@bzwei bzwei commented Mar 14, 2017

Choosing retirement entry is based on :remove_resources option in the config_info

https://www.pivotaltracker.com/story/show/141500781

@miq-bot assign @gmcculloug
@miq-bot add_label providers/ansible_tower
@miq-bot add_label enhancement
@miq-bot add_label service
@miq-bot add_label euwe/no

@tinaafitz please review

@miq-bot
Copy link
Member

miq-bot commented Mar 14, 2017

@bzwei Cannot apply the following label because they are not recognized: service

@tinaafitz
Copy link
Member

@bzwei Looks good.

@miq-bot
Copy link
Member

miq-bot commented Mar 15, 2017

Checked commit bzwei@5183ef0 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks good. 🍰

@gmcculloug
Copy link
Member

Look good.

@gmcculloug gmcculloug merged commit 4d88fff into ManageIQ:master Mar 15, 2017
@gmcculloug gmcculloug added this to the Sprint 57 Ending Mar 27, 2017 milestone Mar 15, 2017
@bzwei bzwei deleted the multiple_retirement_eps branch March 15, 2017 15:39
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