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

Tenant admin can create groups for other tenants #134

Closed
evertmulder opened this issue Jan 11, 2017 · 2 comments
Closed

Tenant admin can create groups for other tenants #134

evertmulder opened this issue Jan 11, 2017 · 2 comments
Assignees
@martinpovolny

Comments

@evertmulder
Copy link
Contributor

evertmulder commented Jan 11, 2017 •
edited
Loading

When logging in as tenant admin (role "EVM Tenant admin" in tenant test1)

2017-01-11 22_28_26-microsoft excel - book1

It is posible to create groups for te root tenant and for other tenants (test2). (The API handles this correctly, rejecting these actions)
@evertmulder evertmulder mentioned this issue Jan 11, 2017
Closed
@martinpovolny martinpovolny self-assigned this Jan 12, 2017
@martinpovolny martinpovolny mentioned this issue Jan 13, 2017
Merged
martinpovolny added a commit to martinpovolny/manageiq-ui-classic that referenced this issue Jan 13, 2017
@martinpovolny
Tenant admin should not be able to create groups in other tenants.
93c02d8 
ManageIQ#134

This is only a part of the fix. The 2nd part needs fixing on the
manageiq core side.
@martinpovolny martinpovolny mentioned this issue Jan 13, 2017
Merged
martinpovolny added a commit to martinpovolny/manageiq-ui-classic that referenced this issue Jan 13, 2017
@martinpovolny
Tenant admin should not be able to create groups in other tenants.
cec2dcf 
ManageIQ#134

This is only a part of the fix. The 2nd part needs fixing on the
manageiq core side.
martinpovolny added a commit to martinpovolny/manageiq-ui-classic that referenced this issue Jan 13, 2017
@martinpovolny
Tenant admin should not be able to create groups in other tenants.
ad59f26 
ManageIQ#134

This is only a part of the fix. The 2nd part needs fixing on the
manageiq core side.
martinpovolny added a commit to martinpovolny/manageiq that referenced this issue Jan 14, 2017
@martinpovolny
Tenant admin should not be able to create groups in other tenants.
48d0fd6 
ManageIQ/manageiq-ui-classic#134

This is only a part of the fix. The 2nd part needs fixing on the
manageiq-ui-classic side.
martinpovolny added a commit to martinpovolny/manageiq-ui-classic that referenced this issue Jan 17, 2017
@martinpovolny
Tenant admin should not be able to create groups in other tenants.
e46ed97 
ManageIQ#134

This is only a part of the fix. The 2nd part needs fixing on the
manageiq core side.
rodneyhbrown7 pushed a commit to rodneyhbrown7/manageiq-ui-classic that referenced this issue Feb 2, 2017
@martinpovolny
Tenant admin should not be able to create groups in other tenants.
a070aa2 
ManageIQ#134

This is only a part of the fix. The 2nd part needs fixing on the
manageiq core side.
@martinpovolny
Copy link
Member

@evertmulder : can we close this issue?

@evertmulder
Copy link
Contributor Author

Yes, I also tested this. Everything works as expected now. 👍👍
Thanks.

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

@martinpovolny martinpovolny

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@martinpovolny @evertmulder