PciPassthroughFilter always added to filter list

Bug #1730369 reported by Edward Hope-Morley
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Nova Cloud Controller Charm
Fix Released
Medium
Edward Hope-Morley

Bug Description

Currently the nova-cloud-controller will always add PciPassthroughFilter to scheduler_default_filters in nova.conf. This is because the code is reading the enable-sriov setting as boolean on the neutron-api relation when in fact booleans are always converted to strings meaning that this will always be true (e.g. if set to "False").

Tags: openstack sts
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-nova-cloud-controller (master)

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

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

Reviewed: https://review.openstack.org/517934
Committed: https://git.openstack.org/cgit/openstack/charm-nova-cloud-controller/commit/?id=05ea3a13f1bf14938069238902008edbef69b4b3
Submitter: Zuul
Branch: master

commit 05ea3a13f1bf14938069238902008edbef69b4b3
Author: Edward Hope-Morley <email address hidden>
Date: Mon Nov 6 12:03:27 2017 +0000

    Only use PciPassthroughFilter when sriov enabled

    Fixes code that inadvertently interprets strings as
    booleans resulting in them always being True.
    PciPassthroughFilter will now only be added to
    scheduler_default_filters if sriov is enabled.

    Change-Id: Ie2c79e98d7880e8172ac7e77febb51af04732bec
    Closes-Bug: 1730369

Changed in charm-nova-cloud-controller:
status: In Progress → Fix Committed
James Page (james-page)
Changed in charm-nova-cloud-controller:
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.