Comment 3 for bug 1554717

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to networking-cisco (stable/liberty)

Reviewed: https://review.openstack.org/290708
Committed: https://git.openstack.org/cgit/openstack/networking-cisco/commit/?id=61937780c7586d22f0a1708afe996ee62fac0de8
Submitter: Jenkins
Branch: stable/liberty

commit 61937780c7586d22f0a1708afe996ee62fac0de8
Author: Paul Michali <email address hidden>
Date: Wed Mar 9 10:33:56 2016 -0500

    Use public method execute() for client extension

    This fixes breakage on Liberty, due to updated neutronclient.

    The client extension was using take_action(), which is really a
    private method (and caused problems recently, when it changed from
    get_data to take_action).

    Instead, this commit uses execute(), which is the public method for
    client extension purposes. This will prevent issues from occurring,
    if/when the neutronclient changes.

    Cherry-pick of I48ef59f10e741aa95a1f75e569ff9660e3ec5819 to Liberty.
    Had to manually perform, because, on Liberty, the method used is
    get_data() and not take_action().

    Also, needed to cherry-pick I7f94bf62710af51291f36a23c3ff0ea4feb52591
    so that tests pass.

    Change-Id: I2c3daba7a72a5d81774e58e62bc962494c1342f5
    Closes-Bug: #1554717
    Partial-Bug: #1546138