Need resource.prop resource list class
Bug #1423620 reported by
Terry Howe
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| OpenStack SDK |
Confirmed
|
High
|
Unassigned | ||
Bug Description
SecurityGroup has a list of SecurityGroupRule. There is a kludge in the __init__ to convert the dicts into SecurityGroupRules. If there was a special resource list property with type=SecurityGr
Revision history for this message
| Brian Curtin (brian.curtin) wrote | #1 |
| Changed in python-openstacksdk: | |
| status: | New → Confirmed |
| Changed in python-openstacksdk: | |
| importance: | Undecided → High |
| milestone: | none → 1.0 |
Revision history for this message
| Terry Howe (thowe-g) wrote | #2 |
| tags: | added: resource |
| Changed in python-openstacksdk: | |
| milestone: | 1.0 → 1.1 |
To post a comment you must log in.
Take a look at what's done in https:/
We're going to have to do the type conversion somewhere, and I think it makes the most sense to unserialize as early as possible when that type is going to come into play, which would be an overridden get call. I think that approach also lines up with how https:/
This is also how you'd do something like unicode/bytes conversion - as close as possible to the boundaries of your application.
I don't think we should have any special type to signify the list itself is special. It's Python so a list is a list and we should just document that it contains values we created of a certain type but not go any further. It really only matters on the receiving side of things that we're creating the right types for the list members.