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

Ansible Service: skip dialog options for retirement #14602

Merged
merged 1 commit into from
Apr 3, 2017

Conversation

bzwei
Copy link
Contributor

@bzwei bzwei commented Mar 31, 2017

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

Although there is no dialog at retirement the old code would still read the dialog options from the provisioning time. The fix is to skip such options for retirement.

@miq-bot add_label bug, providers/ansible_tower, services, fine/yes
@miq-bot assign @gmcculloug

@miq-bot
Copy link
Member

miq-bot commented Mar 31, 2017

Checked commit bzwei@77fef96 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 gmcculloug merged commit 961e29d into ManageIQ:master Apr 3, 2017
@gmcculloug gmcculloug added this to the Sprint 58 Ending Apr 10, 2017 milestone Apr 3, 2017
simaishi pushed a commit that referenced this pull request Apr 3, 2017
Ansible Service: skip dialog options for retirement
(cherry picked from commit 961e29d)
@simaishi
Copy link
Contributor

simaishi commented Apr 3, 2017

Fine backport details:

$ git log -1
commit 6ff82bc7aaa9bd8ffdcb4a9f1e12abce2982ce53
Author: Greg McCullough <gmccullo@redhat.com>
Date:   Mon Apr 3 14:01:41 2017 -0400

    Merge pull request #14602 from bzwei/retirement_hosts
    
    Ansible Service: skip dialog options for retirement
    (cherry picked from commit 961e29d4a42ddc8317da6817021ce4959e44f605)

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