Bug 681344 - Change repo groupid field to be a map instead of an array of strings
Summary: Change repo groupid field to be a map instead of an array of strings
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: z_other
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: Sprint 22
Assignee: Pradeep Kilambi
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On:
Blocks: verified-to-close 684061
TreeView+ depends on / blocked
 
Reported: 2011-03-01 21:19 UTC by Mike McCune
Modified: 2011-08-16 12:20 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2011-08-16 12:06:39 UTC
Embargoed:


Attachments (Terms of Use)

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.


Note You need to log in before you can comment on or make changes to this bug.