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.

Sign up for GitHub

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

Jump to bottom

Admin Role does not have permission to modify node role when a role is specified in Allowed Node Roles field #3217

Closed
atran-inverse opened this issue May 3, 2018 · 3 comments
Closed

Admin Role does not have permission to modify node role when a role is specified in Allowed Node Roles field #3217

atran-inverse opened this issue May 3, 2018 · 3 comments
Assignees
@jrouzierinverse
Labels
Priority: Medium Type: Bug
Milestone
+1 (patch release)

Comments

@atran-inverse
Copy link
Contributor

In the web admin, Configuration -> System Configuration -> Admin Access, a new admin role was created with a role specified under Allowed node roles:
image

When a user belonging in this admin role tries to assign the role to a node, PF gives an error when clicking on Save:
image

All actions for Nodes are defined:
image
@atran-inverse atran-inverse added this to the +1 (patch release) milestone May 3, 2018
@julsemaan
Copy link
Collaborator

This should go in the maintenance 7 + 8

@julsemaan julsemaan reopened this May 9, 2018
jrouzierinverse added a commit that referenced this issue May 9, 2018
@jrouzierinverse
If the node does not have a role allow it to be modified
a37740a 
Fixes #3217
jrouzierinverse added a commit that referenced this issue May 9, 2018
@jrouzierinverse
If the node does not have a role allow it to be modified
5ee109d 
Fixes #3217
jrouzierinverse added a commit that referenced this issue May 9, 2018
@jrouzierinverse
If the node does not have a role allow it to be modified
8792975 
Fixes #3217
jrouzierinverse added a commit that referenced this issue May 9, 2018
@jrouzierinverse
If the node does not have a role allow it to be modified
e4a6240 
Fixes #3217
jrouzierinverse added a commit that referenced this issue May 9, 2018
@jrouzierinverse
If the node does not have a role allow it to be modified
846563e 
Fixes #3217
jrouzierinverse added a commit that referenced this issue May 9, 2018
@jrouzierinverse
If the node does not have a role allow it to be modified
e7b63c7 
Fixes #3217
@julsemaan
Copy link
Collaborator

@atran-inverse said this is still an issue currently on 7.2 with the commit that was done.

Is there another commit ?

@jrouzierinverse
Copy link
Member

yup and it is pushed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jrouzierinverse jrouzierinverse

Labels
Priority: Medium Type: Bug
Projects
None yet
Milestone
+1 (patch release)
Development

No branches or pull requests
3 participants
@jrouzierinverse @julsemaan @atran-inverse