Cannot override compute_driver config

Bug #1472445 reported by Mathieu Gagné
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-nova
Fix Released
High
Mathieu Gagné
Kilo
Fix Released
High
Mathieu Gagné

Bug Description

All nova::compute::* manifests enforce an hardcoded value for the compute_driver config and do not allow the user to override it.

A user should be able to override the compute_driver by its own. Common use case is to use a local or derivative version of the compute driver which adds features and/or bugfixes.

Mathieu Gagné (mgagne)
affects: nova → puppet-nova
Mathieu Gagné (mgagne)
Changed in puppet-nova:
assignee: nobody → Mathieu Gagné (mgagne)
Changed in puppet-nova:
status: New → In Progress
Changed in puppet-nova:
status: In Progress → Confirmed
status: Confirmed → In Progress
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-nova (master)

Reviewed: https://review.openstack.org/199360
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=7a7051ab4885e8a7aaed50f6e90dfcdedfb57773
Submitter: Jenkins
Branch: master

commit 7a7051ab4885e8a7aaed50f6e90dfcdedfb57773
Author: Mathieu Gagné <email address hidden>
Date: Tue Jul 7 20:53:02 2015 -0400

    Add ability to override compute_driver

    This change introduces a new compute_driver parameter
    for all compute driver manifests. This will allow a user to override
    the compute_driver config value by its own instead of forking or
    monkey-patching the manifests.

    It also changes the compute_driver config value of Ironic
    for ironic.IronicDriver which also works and better fit
    the config value style already used in the other manifests.

    Common use case for this parameter is to allow the use of a local
    or derivative version of a driver which adds features and/or bug fixes.

    Closes-bug: #1472445
    Change-Id: I4cd211b389303c22f4c2aa6db7592cc9861d4f40

Changed in puppet-nova:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-nova (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/200543

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-nova (stable/kilo)

Reviewed: https://review.openstack.org/200543
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=32a51c7d44545c79e166eea57f159f9395b47b72
Submitter: Jenkins
Branch: stable/kilo

commit 32a51c7d44545c79e166eea57f159f9395b47b72
Author: Mathieu Gagné <email address hidden>
Date: Tue Jul 7 20:53:02 2015 -0400

    Add ability to override compute_driver

    This change introduces a new compute_driver parameter
    for all compute driver manifests. This will allow a user to override
    the compute_driver config value by its own instead of forking or
    monkey-patching the manifests.

    It also changes the compute_driver config value of Ironic
    for ironic.IronicDriver which also works and better fit
    the config value style already used in the other manifests.

    Common use case for this parameter is to allow the use of a local
    or derivative version of a driver which adds features and/or bug fixes.

    Closes-bug: #1472445
    Change-Id: I4cd211b389303c22f4c2aa6db7592cc9861d4f40
    (cherry picked from commit 7a7051ab4885e8a7aaed50f6e90dfcdedfb57773)

tags: added: in-stable-kilo
Mathieu Gagné (mgagne)
no longer affects: puppet-nova/kilo
Changed in puppet-nova:
milestone: none → 7.0.0
Cody Herriges (ody-cat)
Changed in puppet-nova:
status: Fix Committed → Fix Released
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.