Agent HardwareManager loading always loads GenericHardwareManager

Bug #1327923 reported by Russell Haering
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Medium
Russell Haering

Bug Description

The HardwareManager loading logic in the Ironic Python Agent attempts to load the most suitable implementation found by stevedore, but sorts them in the wrong order so always ends up loading the GenericHardwareManager.

Tags: agent
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic-python-agent (master)

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

Changed in ironic:
assignee: nobody → Russell Haering (russellhaering)
status: New → In Progress
Dmitry Tantsur (divius)
Changed in ironic:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic-python-agent (master)

Reviewed: https://review.openstack.org/98678
Committed: https://git.openstack.org/cgit/openstack/ironic-python-agent/commit/?id=5d5d443d167f38322e543689c51d60a41f1c254b
Submitter: Jenkins
Branch: master

commit 5d5d443d167f38322e543689c51d60a41f1c254b
Author: Russell Haering <email address hidden>
Date: Sun Jun 8 18:45:40 2014 -0700

    Fix HardwareManager loading order bug

    The agent attempts to sort the HardwareManager implementations found
    by stevedore in order of suitability. This patch fixes a bug in the
    sorting order that caused the GenericHardwareManager to always be
    loaded.

    Fixes bug 1327923

    Change-Id: I1cc20e5351a704325f0cf41dab205c9164799f6a

Changed in ironic:
status: In Progress → Fix Committed
Changed in ironic:
milestone: none → juno-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ironic:
milestone: juno-2 → 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.