Delete the libvirt volume_drivers config parameter

Bug #1371175 reported by Daniel Berrange
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
jichenjc

Bug Description

A followup from

  https://bugs.launchpad.net/nova/+bug/1362191

This bug is to track the actual deletion of the volume_drivers config parameter.

Changed in nova:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
jichenjc (jichenjc) wrote :

To be done in L release

Changed in nova:
assignee: nobody → jichenjc (jichenjc)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/168511

Changed in nova:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/168511
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=3bcaf68357d361100cb3ac35b3a5504627db8f0e
Submitter: Jenkins
Branch: master

commit 3bcaf68357d361100cb3ac35b3a5504627db8f0e
Author: jichenjc <email address hidden>
Date: Sun Mar 22 23:23:04 2015 +0800

    libvirt: remove volume_drivers config param

    Copied commit message from
    commit I1f884f36e24fe478922671ca3eaf6b7c21fd8a5a

    The 'volumes_drivers' parameter does the same job as the
    'vif_driver' parameter, but for volumes instead of vifs.

    The API for the LibvirtBaseVolumeDriver class is not
    considered to be a public extension point. As such it
    is not required or expected to remain compatible across
    releases.

    The main reason for its existance is for people who are
    developing new cinder drivers, to be able to write out
    of tree nova volume drivers. The recommended practice
    is to instead work in-tree to nova on a branch. This is
    an approach that would be required in order to submit
    the work for review and eventual merge in any case.

    commit I1f884f36e24fe478922671ca3eaf6b7c21fd8a5a
    deprecated it and during the discussion the actually target
    is Kilo. But it's K-3 so more reasonable target is
    Liberty.

    DocImpact: libvirt.volume_drivers config param is removed.

    Closes-Bug: #1371175

    Change-Id: I832820499ec3304132379ad9b9d1ee92c5a75b61

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → kilo-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: kilo-rc1 → 2015.1.0
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.