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

Tests cloud_networks with RBAC filtering from super #394

Merged
merged 1 commit into from
Mar 9, 2018

Conversation

d-m-u
Copy link
Contributor

@d-m-u d-m-u commented Jan 16, 2018

Amazon's allowed_cloud_networks needs test for RBAC filtering from PR here

@d-m-u
Copy link
Contributor Author

d-m-u commented Jan 16, 2018

@miq-bot add_label test

@miq-bot miq-bot added the test label Jan 16, 2018
@d-m-u
Copy link
Contributor Author

d-m-u commented Jan 16, 2018

@syncrou can you review please?

@d-m-u
Copy link
Contributor Author

d-m-u commented Jan 16, 2018

@Ladas does this work for ManageIQ/manageiq#16824 (comment)?

@miq-bot miq-bot added the wip label Jan 16, 2018
@d-m-u d-m-u closed this Jan 17, 2018
@d-m-u d-m-u reopened this Jan 17, 2018
@d-m-u d-m-u changed the title [WIP] Tests cloud_networks with RBAC filtering from super Tests cloud_networks with RBAC filtering from super Jan 17, 2018
@miq-bot miq-bot removed the wip label Jan 17, 2018
@bronaghs
Copy link

@d-m-u Once you rebase, the tests should pass.

@gmcculloug gmcculloug self-assigned this Feb 13, 2018
allow(User).to receive_messages(:server_timezone => "UTC")
FactoryGirl.create(:ems_amazon)
end

Choose a reason for hiding this comment

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

@d-m-u - can you use:ems instead of creating :provider?

@d-m-u d-m-u force-pushed the adding_test_for_allowed_ci branch from 27ba999 to 64ad835 Compare March 1, 2018 13:16
@miq-bot
Copy link
Member

miq-bot commented Mar 1, 2018

Checked commit d-m-u@64ad835 with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 👍

@bronaghs
Copy link

bronaghs commented Mar 7, 2018

@gmcculloug - bump

Copy link
Member

@gmcculloug gmcculloug left a comment

Choose a reason for hiding this comment

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

Sorry this took so long to get back to. Tests look good. 👍

@gmcculloug gmcculloug merged commit 6f051ff into ManageIQ:master Mar 9, 2018
@gmcculloug gmcculloug added this to the Sprint 81 Ending Mar 12, 2018 milestone Mar 9, 2018
@d-m-u
Copy link
Contributor Author

d-m-u commented Mar 20, 2018

@miq-bot add_label gaprindashvili/yes

simaishi pushed a commit that referenced this pull request Mar 22, 2018
Tests cloud_networks with RBAC filtering from super
(cherry picked from commit 6f051ff)
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 850cb12ec628e2ed1e5f38f548b19efc582136a1
Author: Greg McCullough <gmccullo@redhat.com>
Date:   Fri Mar 9 06:39:50 2018 -0500

    Merge pull request #394 from d-m-u/adding_test_for_allowed_ci
    
    Tests cloud_networks with RBAC filtering from super
    (cherry picked from commit 6f051ff77df34c4eaed236c837132dac1c694c84)

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