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

Fix physical server and topology access rights for EvmRole-operator #16958

Merged
merged 1 commit into from
Feb 6, 2018

Conversation

Sanmoo
Copy link

@Sanmoo Sanmoo commented Feb 6, 2018

Gives EvmRole-operator the rights to access physical server and physical infra topology pages.

Fixes issues:

@saulotoledo
Copy link
Member

I think this is ok. These lines are missing in my previous PR.

@h-kataria @rodneyhbrown7 Who can take a look on this?

@miq-bot
Copy link
Member

miq-bot commented Feb 6, 2018

Checked commit Sanmoo@06e887e with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 👍

@h-kataria h-kataria self-assigned this Feb 6, 2018
@h-kataria h-kataria added this to the Sprint 79 Ending Feb 12, 2018 milestone Feb 6, 2018
@h-kataria h-kataria merged commit 03f19ed into ManageIQ:master Feb 6, 2018
@Sanmoo Sanmoo deleted the fix_1505618 branch February 6, 2018 14:33
simaishi pushed a commit that referenced this pull request Mar 7, 2018
Fix physical server and topology access rights for EvmRole-operator
(cherry picked from commit 03f19ed)

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

simaishi commented Mar 7, 2018

Gaprindashvili backport details:

$ git log -1
commit e1d30e1a544e79f1888c2f9701142582a76e7953
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Tue Feb 6 09:29:53 2018 -0500

    Merge pull request #16958 from Sanmoo/fix_1505618
    
    Fix physical server and topology access rights for EvmRole-operator
    (cherry picked from commit 03f19ed7b30d4e7256826acc883614b9b98f481c)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1530680

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