Comment 10 for bug 1260265

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/99026
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=656884d60b90c28981659747fd2795d1a509e4ac
Submitter: Jenkins
Branch: master

commit 656884d60b90c28981659747fd2795d1a509e4ac
Author: Hans Lindgren <email address hidden>
Date: Tue Jun 10 12:38:54 2014 +0200

    Update IronicHostManager tests to better match how code works

    This is the first in a series of patches to prepare Ironic for the
    removal of capabilities in Nova HostManager.

    IronicHostManager used to rely on capabilities info to decide which
    host state class to use, VM or baremetal. This has been broken since
    host capabilities reporting were removed in Nova, resulting in Nova
    HostState class always being used.

    IronicBareMetal tests make use of an internal Nova HostManager variable
    in a way that hides the fact that IronicNodeState class is no longer
    used. This patch rewrites tests to better reflect how Ironic/Nova code
    now works and prepares for further changes that needs to land in Ironic
    before Nova HostManager capabilities can be removed.

    Change-Id: I795eaed58f6587585d89366d8f1e521588866a58
    Partial-Bug: #1260265