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

[V2V] Add applying_right_sizing state to InfraConversionJob #19238

Merged

Conversation

ghost
Copy link

@ghost ghost commented Sep 2, 2019

For IMS 1.3 we're moving state machine handling from automate into core. For ease of writing and review, we're breaking this down into (hopefully) easily digestible bits, one state at a time. Each PR will ultimately have a corresponding PR in manageiq-content that deletes the relevant bit of code from automate.

Original automate code:
https://github.com/ManageIQ/manageiq-content/blob/master/content/automate/ManageIQ/Transformation/Infrastructure/VM/Common.class/__methods__/applyrightsize.rb

This PR adds applying_right_sizing state to the state machine. The state machine is changed to allow transition from waiting_for_inventory_refresh.

RHBZ: https://bugzilla.redhat.com/show_bug.cgi?id=1747927
Depends on #19149, #19150, #19154, #19177, #19200, #19216, #19222, #19230
Built on #19230

@ghost
Copy link
Author

ghost commented Sep 2, 2019

@miq-bot add-label transformation, refactoring, ivanchuk/yes
@miq-bot add-reviewer @agrare

@miq-bot
Copy link
Member

miq-bot commented Sep 2, 2019

Checked commits fabiendupont/manageiq@6be48fb~...5d6ae53 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
4 files checked, 0 offenses detected
Everything looks fine. 🍰

@ghost
Copy link
Author

ghost commented Sep 3, 2019

@miq-bot add-reviewer @djberg96

Copy link
Contributor

@djberg96 djberg96 left a comment

Choose a reason for hiding this comment

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

LGTM!

@djberg96
Copy link
Contributor

@agrare You ok with this one?

@agrare agrare self-assigned this Sep 10, 2019
@agrare
Copy link
Member

agrare commented Sep 10, 2019

Hey sorry @fdupont-redhat this one slipped through I didn't notice it

@agrare agrare merged commit 7a820ad into ManageIQ:master Sep 10, 2019
@agrare agrare added this to the Sprint 120 Ending Sep 16, 2019 milestone Sep 10, 2019
@ghost ghost deleted the v2v_state_machine_apply_right_size branch September 10, 2019 15:14
simaishi pushed a commit that referenced this pull request Nov 4, 2019
…_right_size

[V2V] Add applying_right_sizing state to InfraConversionJob

(cherry picked from commit 7a820ad)

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

simaishi commented Nov 4, 2019

Ivanchuk backport details:

$ git log -1
commit bf502643dd759b65da14163a86f5cda96771e521
Author: Adam Grare <agrare@redhat.com>
Date:   Tue Sep 10 09:44:09 2019 -0400

    Merge pull request #19238 from fdupont-redhat/v2v_state_machine_apply_right_size
    
    [V2V] Add applying_right_sizing state to InfraConversionJob
    
    (cherry picked from commit 7a820adbf8db1e22d8a64db4cca2cd6b8f0aa97a)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1768522

@simaishi
Copy link
Contributor

simaishi commented Dec 3, 2019

Reverted the backport

commit 50dde45a0ab3b2b8c91b43fa72dda4b1bf4c2858
Author: Satoe Imaishi <simaishi@redhat.com>
Date:   Tue Dec 3 12:44:51 2019 -0500

    Revert "Merge pull request #19238 from fdupont-redhat/v2v_state_machine_apply_right_size"

    This reverts commit bf502643dd759b65da14163a86f5cda96771e521.

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

simaishi pushed a commit that referenced this pull request Feb 21, 2020
…_right_size

[V2V] Add applying_right_sizing state to InfraConversionJob

(cherry picked from commit 7a820ad)

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

Ivanchuk backport details:

$ git log -1
commit 94c02b544a2d3c02babb3a608afb75a3ae9b7daf
Author: Adam Grare <agrare@redhat.com>
Date:   Tue Sep 10 09:44:09 2019 -0400

    Merge pull request #19238 from fdupont-redhat/v2v_state_machine_apply_right_size

    [V2V] Add applying_right_sizing state to InfraConversionJob

    (cherry picked from commit 7a820adbf8db1e22d8a64db4cca2cd6b8f0aa97a)

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

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