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

Fix cleanup state machine path build #392

Merged

Conversation

ghost
Copy link

@ghost ghost commented Aug 3, 2018

The current build method for the cleanup state machine path leaves a leading / in the namespace name, which leads to path resolving failure. This PR removes it.

Associated RHBZ: https://bugzilla.redhat.com/show_bug.cgi?id=1599997

@ghost
Copy link
Author

ghost commented Aug 3, 2018

@miq-bot add-labels transformation, gaprindashvili/yes, bug, blocker

@miq-bot
Copy link
Member

miq-bot commented Aug 3, 2018

Checked commit fabiendupont@56494df with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🍰

@mkanoor mkanoor added this to the Sprint 92 Ending Aug 13, 2018 milestone Aug 7, 2018
@mkanoor mkanoor merged commit c4023ec into ManageIQ:master Aug 7, 2018
simaishi pushed a commit that referenced this pull request Aug 8, 2018
@simaishi
Copy link
Contributor

simaishi commented Aug 8, 2018

Gaprindashvili backport details:

$ git log -1
commit 1452ad84fa68470f55b2dabd0ef37405bc936188
Author: Madhu Kanoor <mkanoor@redhat.com>
Date:   Tue Aug 7 12:08:48 2018 -0400

    Merge pull request #392 from fdupont-redhat/v2v_fix_cleanup_sm_path_build
    
    Fix cleanup state machine path build
    (cherry picked from commit c4023ec37733f5c74a8748d9beb3885aaa523a34)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1608758

@ghost ghost deleted the v2v_fix_cleanup_sm_path_build branch August 14, 2018 15:58
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

4 participants