convert MultiStrOpt ini file params to ListOpt

Bug #1532622 reported by John Stanford
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Confirmed
Wishlist
Unassigned

Bug Description

The following references to MultiStrOpt should be converted to ListOpt. The only disadvantage of ListOpt is that it is comma-separated, but none of the identified parameters seem to have values that would contain commas:

The configs in question would be:

nova/api/openstack/compute/legacy_v2/extensions.py: osapi_compute_extension
nova/compute/monitors/__init__.py: compute_available_monitors
nova/network/ldapdns.py: ldap_dns_servers
nova/network/linux_net.py: dhcpbridge_flagfile
nova/network/linux_net.py: dns_server
nova/network/linux_net.py: force_snat_range
nova/network/linux_net.py: forward_bridge_interface
nova/openstack/common/policy.py: policy_dirs
nova/pci/request.py: pci_alias
nova/pci/whitelist.py:pci_passthrough_whitelist
nova/scheduler/host_manager.py: scheduler_available_filters
nova/virt/disk/api.py: virt_mkfs

There is an email thread on the topic here:

http://markmail.org/message/5ut4rdjivpw6a6a6#query:+page:1+mid:5ut4rdjivpw6a6a6+state:results

Tags: oslo
tags: added: oslo
Changed in nova:
importance: Undecided → Wishlist
status: New → Confirmed
Changed in nova:
assignee: nobody → Stephen Finucane (sfinucan)
assignee: Stephen Finucane (sfinucan) → nobody
assignee: nobody → Stephen Finucane (sfinucan)
Revision history for this message
Sarafraj Singh (sarafraj-singh) wrote :

Stephen,
Are you working on the fix? Please change status to Inprogress if you are, otherwise remove yourself as assignee so someone else can pick it up.

Revision history for this message
Stephen Finucane (stephenfinucane) wrote :

These options have been moved around, meaning only a few of them are still worth keeping. These options have been removed:

nova/api/openstack/compute/legacy_v2/extensions.py: osapi_compute_extension
nova/openstack/common/policy.py: policy_dirs

These options are going to be/have been deprecated, and shouldn't be moved:

nova/conf/network.py: dhcpbridge_flagfile
nova/conf/network.py: dns_server
nova/conf/network.py: force_snat_range
nova/conf/network.py: forward_bridge_interface

These options (with updated paths) are still worth moving:

nova/conf/compute.py compute_available_monitors
nova/conf/network.py ldap_dns_servers
nova/conf/pci.py: pci_alias
nova/conf/pci.py:pci_passthrough_whitelist
nova/conf/scheduler.py: scheduler.available_filters
nova/conf/virt.py: virt_mkfs

Sean Dague (sdague)
Changed in nova:
assignee: Stephen Finucane (stephenfinucane) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.