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

Apply RBAC filtering to cloud networks when creating floating ips #3531

Merged
merged 1 commit into from Mar 8, 2018

Conversation

mansam
Copy link
Contributor

@mansam mansam commented Mar 7, 2018

@mansam
Copy link
Contributor Author

mansam commented Mar 7, 2018

@miq-bot add_label gaprindashvili/yes
@miq-bot add_label fine/yes

Fine bug: https://bugzilla.redhat.com/show_bug.cgi?id=1552780

@h-kataria
Copy link
Contributor

looks good, restarted failing tests, ready to merge after tests are green.

@dclarizio dclarizio self-assigned this Mar 8, 2018
@aufi
Copy link
Member

aufi commented Mar 8, 2018

Looks good to me 👍

@dclarizio dclarizio added the bug label Mar 8, 2018
@h-kataria
Copy link
Contributor

@mansam can you rebase/repush your branch, lets see if that fixes unrelated travis test failures.

@mansam mansam force-pushed the filter-networks-for-floating-ips branch from 96b5e2f to 7ddf1eb Compare March 8, 2018 17:30
@miq-bot
Copy link
Member

miq-bot commented Mar 8, 2018

Checked commit mansam@7ddf1eb with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. ⭐

@dclarizio
Copy link

@h-kataria and I agree the failing tests are unrelated to this change . . . merging

@dclarizio dclarizio merged commit 4773d1f into ManageIQ:master Mar 8, 2018
@dclarizio dclarizio added this to the Sprint 81 Ending Mar 12, 2018 milestone Mar 8, 2018
simaishi pushed a commit that referenced this pull request Mar 9, 2018
Apply RBAC filtering to cloud networks when creating floating ips
(cherry picked from commit 4773d1f)

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

simaishi commented Mar 9, 2018

Gaprindashvili backport details:

$ git log -1
commit e87c6baa217b5d5384d96e930e2d11140ea3f5be
Author: Dan Clarizio <dclarizi@redhat.com>
Date:   Thu Mar 8 11:01:33 2018 -0800

    Merge pull request #3531 from mansam/filter-networks-for-floating-ips
    
    Apply RBAC filtering to cloud networks when creating floating ips
    (cherry picked from commit 4773d1fae2f9b367d982c9f339d185f615a2f2e9)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1553787

simaishi pushed a commit that referenced this pull request Apr 10, 2018
Apply RBAC filtering to cloud networks when creating floating ips
(cherry picked from commit 4773d1f)

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

Fine backport details:

$ git log -1
commit 0fbf5a2fa25d0165c1f965ad8b58a906dc6b0ba9
Author: Dan Clarizio <dclarizi@redhat.com>
Date:   Thu Mar 8 11:01:33 2018 -0800

    Merge pull request #3531 from mansam/filter-networks-for-floating-ips
    
    Apply RBAC filtering to cloud networks when creating floating ips
    (cherry picked from commit 4773d1fae2f9b367d982c9f339d185f615a2f2e9)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1552780

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.

None yet

7 participants