Bug 640762

Summary:	Groups - dynagroup: Flipping the Recursive toggle sets in the UI but does not actually get applied to group.
Product:	[Other] RHQ Project	Reporter:	Corey Welton <cwelton>
Component:	Core UI	Assignee:	RHQ Project Maintainer <rhq-maint>
Status:	CLOSED NOTABUG	QA Contact:	Mike Foley <mfoley>
Severity:	medium	Docs Contact:
Priority:	low
Version:	unspecified	CC:	jshaughn
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2011-05-26 13:23:54 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:	585306, 638727

Description Corey Welton 2010-10-06 18:57:33 UTC

Description of problem:
User can set and re-set the Recursive bit on a dynagroup, and it remains set correctly in UI, but it does not actually get applied to the group.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.  Create and populate a dynagroup, "Test Dynagroup".  Assure it is NOT Recursive; use the following Expression:

resource.type.plugin = Platforms
resource.type.name = Linux
  
2. Recalculate your dynagroup
3. Inventory > Dynagroup Manager > double-click "Test Dynagroup" to edit.  
4. View value currently set for the Recursive bit; flip it; save.  
5. Inventory > Compatible Groups > "Test Dynagroup" - View the value for "Recursive?:" field.
6. Repeat steps 3-5 multiple times.

  
Actual results:
Each time we go to set the Recursive bit, we can change it, and subsequent returns to the edit page indicate that the value is being saved.  However, the compatible group view for the dynagroup never changes, it always reads "Recursive? : no"  (or quite possibly "yes" if that's the way you set it initially.)

Expected results:
Flipping the bit is correctly reflected in the compatible group view for the dynagroup.

Additional info:

Comment 1 Jay Shaughnessy 2011-02-17 19:52:19 UTC

I think the idea is that you need to save the group def recursive flag but then you need to recalculate the groups.  Doing this seems to work.

Comment 2 Mike Foley 2011-05-26 13:23:54 UTC

reviewed Dynagroups.  I can see why the bug was logged (sort of a usability induced confusion over expectations) ... and I understand jshaughn's response as well.  no code change here, so nothing to verify.  there is the "save and recalculate" button clearly labelled ... eh ... if i can think of some way to make the UI's intentions more clear i will advocate for it, but will close this bug for now as no code change.