Define constants for connector factory protocol
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
os-brick |
Fix Released
|
Undecided
|
Ryan Rossiter |
Bug Description
While reviewing the os-brick integration change in nova:
https:/
I brought up that it would be better to specify a constant for the protocol parameter passed from nova to os-brick's InitiatorConnec
http://
The values passed in to os-brick right now are hard-coded. The code is case insensitive, which is good, but we should go a step further and have constants defined in os-brick that we can then use in nova and cinder so we don't have to hard-code the values.
Long-term it might be nicer yet to define enums using the enum field types in oslo.versionedo
Changed in os-brick: | |
status: | New → Confirmed |
Changed in os-brick: | |
assignee: | nobody → Cale Rath (ctrath) |
Changed in os-brick: | |
assignee: | Cale Rath (ctrath) → nobody |
Changed in os-brick: | |
assignee: | nobody → Ryan Rossiter (rlrossit) |
status: | Confirmed → In Progress |
Changed in os-brick: | |
milestone: | none → 0.4.0 |
status: | Fix Committed → Fix Released |
This is the patch: https:/ /review. openstack. org/#/c/ 207090/