Comment 3 for bug 942713

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

Reviewed: https://review.openstack.org/4647
Committed: http://github.com/openstack/quantum/commit/c029777dd7523c5216fee5d48dd9f9ca4bf3b84d
Submitter: Jenkins
Branch: master

commit c029777dd7523c5216fee5d48dd9f9ca4bf3b84d
Author: Dan Wendlandt <email address hidden>
Date: Tue Feb 28 12:34:49 2012 -0800

    Fix some plugins that don't check that nets + ports are owned by tenant

    bug 942713. This bug confuses the validate_networks() method of
    QuantumManager in Nova, causing it to believe that it is valid for a
    tenant to plug into a particular network when in fact that network is not
    owned by the tenant, nor the "provider".

    The patch also adds unit tests to confirm correct plugin behavior.

    This patch fixes the issue for the Sample Plugin, the OVS plugin,
    the Linux Bridge plugin, and the Ryu plugin, all of which has the
    same DB model. Validated the fix with the unit tests.

    I couldn't run the unit tests for the NVP plugin standalone, but by
    inspection, the code seems to handle this case. I wasn't able to run
    the Cisco plugin unit tests, and that code uses its own DB model, so I
    am uncertain whether this issue exists in that plugin.

    Change-Id: I8c4a5f3eb151b91a1076821dc1916842510dfb90