Deprecate the libvirt volume_drivers config parameter

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

Bug Description

In this thread the topic of deprecating and removing config parameters related to extension points for non-public APIs was discussed. The consensus was that such extension points should not be exposed and that instead people wishing to develop extensions should be doing so on a nova branch, instead of entirely separate repository.

  https://<email address hidden>/msg30206.html

The vif_drivers parameter is now removed, and this bug is to track deprecation & removal of the volume_drivers parameter since that serves an identical purpose

It will be deprecated in Kilo and deleted in Lxxxx

Tags: libvirt
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to nova (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/117252

Tracy Jones (tjones-i)
tags: added: libvirt
Sean Dague (sdague)
Changed in nova:
status: New → Confirmed
importance: Undecided → High
milestone: none → juno-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to nova (master)

Reviewed: https://review.openstack.org/117252
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=79a7029d5f54c82d82dfaff979e62560dd090050
Submitter: Jenkins
Branch: master

commit 79a7029d5f54c82d82dfaff979e62560dd090050
Author: Daniel P. Berrange <email address hidden>
Date: Wed Aug 27 14:47:28 2014 +0100

    libvirt: deprecated volume_drivers config parameter

    In Icehouse the 'vif_driver' parameter was deprecated

      commit 9f6070e194504cc2ca2b7f2a2aabbf91c6b81897
      Author: Lars Kellogg-Stedman <email address hidden>
      Date: Fri Apr 4 14:58:12 2014 -0400

        mark vif_driver as deprecated and log warning

    and then in Juno removed

      commit 7561c8ded211d53e8745d1420a73b82bd0fc35cf
      Author: Daniel P. Berrange <email address hidden>
      Date: Thu Jul 24 11:53:47 2014 +0100

        libvirt: remove 'vif_driver' config parameter

    The 'volumes_drivers' parameter does the same job as the
    'vif_driver' parameter, but for volumes instead of vifs.
    The reasons for deprecating it, in preparation for its
    removal in Lxxxx are thus the same.

    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.

    DocImpact: libvirt.volume_drivers config param for
               nova.conf is deprecated, to be removed in
               the Lxxxx release
    Related-bug: #1362191
    Change-Id: I1f884f36e24fe478922671ca3eaf6b7c21fd8a5a

Matt Riedemann (mriedem)
Changed in nova:
milestone: juno-rc1 → none
summary: - Deprecated & remove the libvirt volume_drivers config parameter
+ Deprecated the libvirt volume_drivers config parameter
summary: - Deprecated the libvirt volume_drivers config parameter
+ Deprecate the libvirt volume_drivers config parameter
Changed in nova:
status: Confirmed → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → juno-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: juno-rc1 → 2014.2
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.