Bug 681344

Summary: Change repo groupid field to be a map instead of an array of strings
Product: [Retired] Pulp Reporter: Mike McCune <mmccune>
Component: z_otherAssignee: Pradeep Kilambi <pkilambi>
Status: CLOSED CURRENTRELEASE QA Contact: Preethi Thomas <pthomas>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: pkilambi, skarmark
Target Milestone: ---Keywords: Triaged
Target Release: Sprint 22   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-08-16 12:06:39 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 647488, 684061    

Description Mike McCune 2011-03-01 21:19:29 UTC
Currently users of the groupid field are adding multiple values to the field and are being forced to do string parsing to store different types of groups.

Example:

["product:rhel-5-vt", "product:rhel-5-tools", "env:qa"]


this would be easier to manage if groupid was changed to be a map of values:

 {"product" : ["rhel-5-vt", "rhel-5-tools"], "env" : ["qa"]}

then we could find specific fields on the repo in an easier manner.

Comment 1 Pradeep Kilambi 2011-03-18 21:33:53 UTC
commit 3cce1985db2e426d9506d387f68c2ec7eda64bfc

Comment 2 Jeff Ortel 2011-03-18 22:12:30 UTC
Build: 0.154

Comment 3 Pradeep Kilambi 2011-03-21 13:30:42 UTC
This is still in a sprint22 branch. So it did not make it into 154 build. Moving back to modified.

Comment 4 Pradeep Kilambi 2011-03-22 18:55:12 UTC
commit de3decce84e763d066706c679f6f0df5d349a15c
Author: Pradeep Kilambi <pkilambi>
Date:   Tue Mar 22 14:49:15 2011 -0400

    681344 - repo group is back to an array of strings until we see it necessary to be a hash

Comment 5 Jeff Ortel 2011-03-29 17:32:33 UTC
Build: 0.158

Comment 6 Preethi Thomas 2011-03-31 19:52:37 UTC
verified
groupid is working as before


[root@preethi ~]# pulp-admin -u admin -p admin repo update --id=f14 --addgroup=test
Successfully updated repository [ f14 ]

Comment 7 Preethi Thomas 2011-08-16 12:06:39 UTC
Closing with Community Release 15

pulp-0.0.223-4.

Comment 8 Preethi Thomas 2011-08-16 12:20:41 UTC
Closing with Community Release 15

pulp-0.0.223-4.