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

VM Retirement Request - clear values from vmretire_request_starting instance. #532

Merged

Conversation

tinaafitz
Copy link
Member

This instance used to call into automate to initiate retirement.
With retirement as a request, retirement is initiated when the tasks are
created. We need to clear the values of this instance because the
request task processing will create the request tasks only if the
request_starting event succeeds.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1706208
Also requires: ManageIQ/manageiq#18738

With retirement as a request, retirement is initiated when the tasks are
created. We need to clear the values of this instance because the
request task processing will create the request tasks only if the
request_starting event succeeds.
@tinaafitz
Copy link
Member Author

@miq-bot add_label bug, hammer/yes

@miq-bot
Copy link
Member

miq-bot commented May 6, 2019

Checked commit tinaafitz@c714094 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. ⭐

@coveralls
Copy link

coveralls commented May 6, 2019

Pull Request Test Coverage Report for Build 3038

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 96.909%

Totals Coverage Status
Change from base Build 3010: 0.0%
Covered Lines: 2884
Relevant Lines: 2976

💛 - Coveralls

@mkanoor mkanoor merged commit a2eb75a into ManageIQ:master May 7, 2019
simaishi pushed a commit that referenced this pull request May 7, 2019
…instance

VM Retirement Request - clear values from vmretire_request_starting instance.

(cherry picked from commit a2eb75a)

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

simaishi commented May 7, 2019

Hammer backport details:

$ git log -1
commit d42e03ffb35d1803877003c6be53f1f3c2a72370
Author: Madhu Kanoor <mkanoor@redhat.com>
Date:   Tue May 7 10:48:08 2019 -0400

    Merge pull request #532 from tinaafitz/fix_vmretire_request_starting_instance
    
    VM Retirement Request - clear values from vmretire_request_starting instance.
    
    (cherry picked from commit a2eb75a4a744165f2919ccf810ba6da64061d744)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1707456

@tinaafitz tinaafitz added this to the Sprint 111 Ending May 13, 2019 milestone May 14, 2019
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.

None yet

5 participants