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

Pass manageiq-smartstate the Resource Group Name not the Object #155

Merged
merged 1 commit into from
Nov 6, 2017

Conversation

jerryk55
Copy link
Member

@jerryk55 jerryk55 commented Nov 2, 2017

A previous fix that was backported to the FINE branch fixed this
issue caused by the resource group being changed from a string
to an object a while ago but the fix never went into the
MASTER branch. This will fix in Master and should be back-ported
to Gaprindashvili. It should not go into the FINE branch.

https://bugzilla.redhat.com/show_bug.cgi?id=1507977 is the bug for this PR.

@roliveri @hsong-rh @bronaghs @djberg96 please review and merge.

@jerryk55
Copy link
Member Author

jerryk55 commented Nov 2, 2017

@miq-bot add_label gaprindashvili/yes

@jerryk55
Copy link
Member Author

jerryk55 commented Nov 2, 2017

@miq-bot add_label fine/no

@jerryk55
Copy link
Member Author

jerryk55 commented Nov 2, 2017

@miq-bot add_label bug

@miq-bot miq-bot added the bug label Nov 2, 2017
@miq-bot
Copy link
Member

miq-bot commented Nov 3, 2017

This pull request is not mergeable. Please rebase and repush.

A previous fix that was backported to the FINE branch fixed this
issue caused by the resource group being changed from a string
to an object a while ago but the fix never went into the
MASTER branch.  This will fix in Master and should be back-ported
to Gaprindashvili
@jerryk55 jerryk55 force-pushed the another_resource_group_name_fix branch from 3057c7e to d71660f Compare November 3, 2017 18:49
@miq-bot
Copy link
Member

miq-bot commented Nov 3, 2017

Checked commit jerryk55@d71660f with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks fine. 🍪

@juliancheal
Copy link
Member

LGTM 👍

@bronaghs bronaghs merged commit b0271f2 into ManageIQ:master Nov 6, 2017
@bronaghs bronaghs added this to the Sprint 73 Ending Nov 13, 2017 milestone Nov 6, 2017
simaishi pushed a commit that referenced this pull request Nov 7, 2017
Pass manageiq-smartstate the Resource Group Name not the Object
(cherry picked from commit b0271f2)

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

simaishi commented Nov 7, 2017

Gaprindashvili backport details:

$ git log -1
commit 8be9bdd3dc05da2d72a88d844e49ee07d8d2e528
Author: Bronagh Sorota <bsorota@redhat.com>
Date:   Mon Nov 6 09:59:12 2017 -0500

    Merge pull request #155 from jerryk55/another_resource_group_name_fix
    
    Pass manageiq-smartstate the Resource Group Name not the Object
    (cherry picked from commit b0271f20fe6894d9808c9e957d923285d2f2066f)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1510072

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