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

Seed tenant product features in Spec::Support::API::Helpers #508

Merged
merged 1 commit into from
Nov 9, 2018

Conversation

lpichler
Copy link
Contributor

@lpichler lpichler commented Nov 9, 2018

This fixes [test failures in API repo](url
https://travis-ci.org/ManageIQ/manageiq-api/jobs/452479021) because ManageIQ/manageiq#18102 requires seeded product tenant features.

Note:
in case that tenant product features are not seeded correctly - this fix is "safe point" ManageIQ/manageiq#18179 (it skips tenant product feature authorisation)

@miq-bot add_label bug, spec

@miq-bot assign @gtanzillo

@miq-bot
Copy link
Member

miq-bot commented Nov 9, 2018

@lpichler Cannot apply the following label because they are not recognized: spec

@gtanzillo gtanzillo added this to the Sprint 99 Ending Nov 19, 2018 milestone Nov 9, 2018
@gtanzillo gtanzillo merged commit 7a024f7 into ManageIQ:master Nov 9, 2018
@lpichler lpichler deleted the seed_tenant_product_features branch November 9, 2018 16:56
@lpichler
Copy link
Contributor Author

@miq-bot add_label hammer/yes

simaishi pushed a commit that referenced this pull request Nov 14, 2018
Seed tenant product features in Spec::Support::API::Helpers

(cherry picked from commit 7a024f7)

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

Hammer backport details:

$ git log -1
commit c0202c10dc9e13207c45caf35901b2b49ac19b6a
Author: Gregg Tanzillo <gtanzill@redhat.com>
Date:   Fri Nov 9 11:51:11 2018 -0500

    Merge pull request #508 from lpichler/seed_tenant_product_features
    
    Seed tenant product features in Spec::Support::API::Helpers
    
    (cherry picked from commit 7a024f7488f39daa9bb208dd0745d941c53e0bd0)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1468795

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