Comment 1 for bug 1420971

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

Reviewed: https://review.openstack.org/153743
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=3b8a2cf781d43d0b6937ec66aff5f37657f625ed
Submitter: Jenkins
Branch: master

commit 3b8a2cf781d43d0b6937ec66aff5f37657f625ed
Author: Anish Bhatt <email address hidden>
Date: Fri Feb 6 16:40:36 2015 -0800

    libvirt : Fix slightly misleading parameter name, validate param

    We actually need transport_iface, not transport as the param
    (iscsi_tcp & iser are exceptions to this, where transport and
    transport_iface are one and the same), so change it so that user is
    not confused as to what must be provided. Also fixes a typo in
    param help section.

    Also added code to ensure that non-existing transport_iface is not
    provided, in which case we we log a warning and fall back to the
    default instead of failing as would have previously happened.

    There is no change in functionality, this will just ensure that
    documentation is not misleading.

    Originally added in commit 554647a4deee6ece221eb79fc93551de72b17ae3.

    DocImpact - due to the renamed config option
    Closes-Bug: #1420971
    Change-Id: I6c190328803e8efcde34d522e2c1814ef6afc220