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

Make cloud_tenants and flavors subcollections consistent with others #321

Merged
merged 1 commit into from
Feb 14, 2018

Conversation

jntullo
Copy link

@jntullo jntullo commented Feb 14, 2018

Infra providers do not have cloud_tenants or flavors, causing a 500 error to be returned when they are requested. This makes the behavior consistent with other subcollections, checking to see if the object responds to the subcollection.

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

@jntullo jntullo added the bug label Feb 14, 2018
@abellotti abellotti self-assigned this Feb 14, 2018
@abellotti
Copy link
Member

That should do it 👍 Thanks @jntullo

@miq-bot
Copy link
Member

miq-bot commented Feb 14, 2018

Checked commit jntullo@6fc044e with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
4 files checked, 0 offenses detected
Everything looks fine. ⭐

@abellotti abellotti added this to the Sprint 80 Ending Feb 26, 2018 milestone Feb 14, 2018
@abellotti abellotti merged commit b3a2bf8 into ManageIQ:master Feb 14, 2018
@jntullo jntullo deleted the invalid_subcollections branch February 14, 2018 19:32
@simaishi
Copy link
Contributor

@abellotti Can this be gaprindashvili/yes ?

@abellotti
Copy link
Member

yes, should be safe for gaprindashvili. Thanks.

simaishi pushed a commit that referenced this pull request Sep 20, 2018
Make cloud_tenants and flavors subcollections consistent with others

(cherry picked from commit b3a2bf8)

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

Gaprindashvili backport details:

$ git log -1
commit 55432b631acfa970c6ac4b5e5d7f31af121d8e86
Author: Alberto Bellotti <abellotti@users.noreply.github.com>
Date:   Wed Feb 14 09:40:24 2018 -0500

    Merge pull request #321 from jntullo/invalid_subcollections
    
    Make cloud_tenants and flavors subcollections consistent with others
    
    (cherry picked from commit b3a2bf8fa4b125b6f5d699942a2f20cf5b3b2a1f)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1612905

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.

4 participants