Bug 1364945
| Summary: | Multi-tenancy - tenant name not renamed in Set group ownership dropdown menu | |||
|---|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Pavol Kotvan <pakotvan> | |
| Component: | UI - OPS | Assignee: | Brian McLaughlin <bmclaugh> | |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Pavol Kotvan <pakotvan> | |
| Severity: | high | Docs Contact: | ||
| Priority: | medium | |||
| Version: | 5.6.0 | CC: | cpelland, hkataria, jhardy, mpovolny, obarenbo, pakotvan, simaishi | |
| Target Milestone: | GA | Keywords: | Regression, TestOnly, ZStream | |
| Target Release: | 5.7.0 | |||
| Hardware: | Unspecified | |||
| OS: | Unspecified | |||
| Whiteboard: | tenant_cfme:ui | |||
| Fixed In Version: | 5.7.0.1 | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | ||
| Clone Of: | ||||
| : | 1374695 (view as bug list) | Environment: | ||
| Last Closed: | 2017-01-11 20:14:41 UTC | Type: | Bug | |
| Regression: | --- | Mount Type: | --- | |
| Documentation: | --- | CRM: | ||
| Verified Versions: | Category: | Bug | ||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
| Cloudforms Team: | --- | Target Upstream Version: | ||
| Embargoed: | ||||
| Bug Depends On: | ||||
| Bug Blocks: | 1374695 | |||
|
Comment 2
Harpreet Kataria
2016-08-08 20:15:45 UTC
Brian, On ownership edit screen in the Group drop down we should only show non tenant groups. You can refer to the code that build Access control tree in OPS to see how built in Tenant Groups are eliminated from the tree. Let me know if you have questions. Thanks, ~Harpreet New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/832b50af2f94ab73cc078061cc6cb533e7621f17 commit 832b50af2f94ab73cc078061cc6cb533e7621f17 Author: Brian McLaughlin <bmclaugh> AuthorDate: Tue Aug 23 21:08:51 2016 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Tue Aug 23 21:10:12 2016 -0400 Spec tests https://bugzilla.redhat.com/show_bug.cgi?id=1364945 .../application_controller/ci_processing_spec.rb | 14 ++++++++++++++ 1 file changed, 14 insertions(+) New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/98f1bec732e5ec57c95f1548fce3c44ff0d57aad commit 98f1bec732e5ec57c95f1548fce3c44ff0d57aad Author: Brian McLaughlin <bmclaugh> AuthorDate: Tue Aug 23 21:05:56 2016 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Tue Aug 23 21:10:12 2016 -0400 Show non-tenant groups in Set Ownership group dropdown https://bugzilla.redhat.com/show_bug.cgi?id=1364945 app/controllers/application_controller/ci_processing.rb | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/e920d4036fe2789f66521291608e86e986cc2c4a commit e920d4036fe2789f66521291608e86e986cc2c4a Author: Brian McLaughlin <bmclaugh> AuthorDate: Tue Aug 23 21:56:04 2016 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Tue Aug 23 21:56:04 2016 -0400 address miq-bot feedback https://bugzilla.redhat.com/show_bug.cgi?id=1364945 app/controllers/application_controller/ci_processing.rb | 11 ++++++----- spec/controllers/application_controller/ci_processing_spec.rb | 1 - 2 files changed, 6 insertions(+), 6 deletions(-) New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/2cc9de56e058a255217b428ed270bb7d57c8733f commit 2cc9de56e058a255217b428ed270bb7d57c8733f Author: Brian McLaughlin <bmclaugh> AuthorDate: Tue Aug 30 15:50:56 2016 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Tue Aug 30 15:50:56 2016 -0400 Refactor solution to remove restriction to vm_cloud Removed test for Tenant descendant strategy due to this method now returning non-tenant groups. https://bugzilla.redhat.com/show_bug.cgi?id=1364945 .../application_controller/ci_processing.rb | 8 +------ .../application_controller/ci_processing_spec.rb | 28 ++-------------------- 2 files changed, 3 insertions(+), 33 deletions(-) |