Bug 755653
Summary: | updating a group's name or description causes the group to be removed from any roles with which it is associated | ||
---|---|---|---|
Product: | [Other] RHQ Project | Reporter: | Ian Springer <ian.springer> |
Component: | Core UI | Assignee: | Lukas Krejci <lkrejci> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Mike Foley <mfoley> |
Severity: | urgent | Docs Contact: | |
Priority: | high | ||
Version: | 4.2 | CC: | ccrouch, hrupp, lkrejci, mazz, skondkar |
Target Milestone: | --- | ||
Target Release: | JON 3.0.0, RHQ 4.3.0 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | 4.3.0 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2012-02-07 19:21:00 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 752488 |
Description
Ian Springer
2011-11-21 17:37:08 UTC
not so comfortable with last minute changes to security. setting priority to MEDIUM. lukas ... put this only in master (not jon 3 branch). additionally 1) document your perception of the regression risk with security, 2) document the tests that you have performed to verify this 3) document who else in development has code reviewed your change (10:25:01 AM) lkrejci: mfoley: wrt https://bugzilla.redhat.com/show_bug.cgi?id=755653. is this not a regression from JON 2.4.1? (i didn't check yet) (10:27:51 AM) mfoley: looking (10:28:23 AM) mfoley: i don't know if it is a regression or not (10:34:00 AM) lkrejci: mfoley: it's a regression from jon 2.4.1, i'm going to work on it anyway but I think it *should* go into the release This is only in master right now. http://git.fedorahosted.org/git/?p=rhq/rhq.git;a=commitdiff;h=90081e71f34681d194767d00705d485a393ee133 commit 90081e71f34681d194767d00705d485a393ee133 Author: Lukas Krejci <lkrejci> Date: Thu Nov 24 12:56:28 2011 +0100 [BZ 755653] - Make sure role membership is not updated through the updateResourceGroup() methods. Verified on master build#790 (Version: 4.3.0-SNAPSHOT Build Number: b9ef6c8) Created a group and a role. Associated the group to the role. Changed the group name and verified that the role is associated with the group. It displays the changed group name in 'Assigned Resource Groups' table. assigning to dev. has this change been code reviewed by another dev? It looks good to me. then please push to jon 3 branch code looks good. pushed to release_jon3.x branch: e4eac1f verified changing status of VERIFIED BZs for JON 2.4.2 and JON 3.0 to CLOSED/CURRENTRELEASE |