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

Test az_to_cn method from super #203

Merged
merged 1 commit into from
Jan 19, 2018

Conversation

d-m-u
Copy link
Contributor

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

PR ManageIQ/manageiq#16811 adds RBAC filtering to allowed_cloud_network base class and openstack needed specs around the fact that the cloud network list is NOT dependent on availability zone choice unlike azure and amazon. (Per ManageIQ/manageiq#16824 (comment))

Related bz links:
https://bugzilla.redhat.com/show_bug.cgi?id=1533277
https://bugzilla.redhat.com/show_bug.cgi?id=1535189

@d-m-u
Copy link
Contributor Author

d-m-u commented Jan 19, 2018

@miq-bot assign @mansam
@miq-bot add_label test

@d-m-u d-m-u force-pushed the add_tests_for_allowed_cloud_networks branch from 254719b to 4583169 Compare January 19, 2018 19:03
@miq-bot
Copy link
Member

miq-bot commented Jan 19, 2018

Checked commit d-m-u@4583169 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. 🍪

@mansam
Copy link
Contributor

mansam commented Jan 19, 2018

test failure is unrelated, merging.

@mansam mansam merged commit 591e223 into ManageIQ:master Jan 19, 2018
@d-m-u d-m-u deleted the add_tests_for_allowed_cloud_networks branch January 19, 2018 19:52
@d-m-u
Copy link
Contributor Author

d-m-u commented Jan 19, 2018

thanks @mansam!!

@aufi aufi added this to the Sprint 78 Ending Jan 29, 2018 milestone Jan 30, 2018
simaishi pushed a commit that referenced this pull request Mar 20, 2018
Test az_to_cn method from super
(cherry picked from commit 591e223)
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit bf83e81778a4201b988931556ca8abce7ce72313
Author: Samuel Lucidi <[email protected]>
Date:   Fri Jan 19 14:51:48 2018 -0500

    Merge pull request #203 from d-m-u/add_tests_for_allowed_cloud_networks
    
    Test az_to_cn method from super
    (cherry picked from commit 591e223695233e341d132e44ada9e89e113892a9)

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