Neutron ansible module not plugging subnet into router

[Matt Thompson]

There seems to be an issue w/ running os-tempest.yml where all networks are created, as is the router, but the private subnet is not getting plugged into the router. I'm not sure if this is failing 100% of the time, but this did not appear to be an issue w/ juno.

More details to come.

[Ian Cordasco (icordasc)]

Can you provide some more details how to reproduce accurately?

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/170046
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=8bd358d2c25f9bfb6011f751ef5342e012fbb6d8
Submitter: Jenkins
Branch: master

commit 8bd358d2c25f9bfb6011f751ef5342e012fbb6d8
Author: Matt Thompson <email address hidden>
Date: Thu Apr 2 09:21:16 2015 +0100

    Fix bug in playbooks/library/neutron

    This change updates _add_router_interface to list ports for the router
    device only, and not for all devices. As we select specific device
    ports only, we no longer need to skip over other device's ports, and
    this indirectly fixes a bug where we were using 'next' instead of
    'continue' to skip to the next iteration of the for loop.

    Change-Id: I892dc0f58f1d8bbb7ebd46f7790c5420c4c8b94b
    Closes-Bug: #1439295

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: juno
Review: https://review.openstack.org/171489

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: icehouse
Review: https://review.openstack.org/171494

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/171494
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=1f3dc1f7f3ce33d353f88c40cbe96248eaa1c174
Submitter: Jenkins
Branch: icehouse

commit 1f3dc1f7f3ce33d353f88c40cbe96248eaa1c174
Author: Matt Thompson <email address hidden>
Date: Thu Apr 2 09:21:16 2015 +0100

    Fix bug in playbooks/library/neutron

    This change updates _add_router_interface to list ports for the router
    device only, and not for all devices. As we select specific device
    ports only, we no longer need to skip over other device's ports, and
    this indirectly fixes a bug where we were using 'next' instead of
    'continue' to skip to the next iteration of the for loop.

    Change-Id: I892dc0f58f1d8bbb7ebd46f7790c5420c4c8b94b
    Closes-Bug: #1439295
    (cherry picked from commit 8bd358d2c25f9bfb6011f751ef5342e012fbb6d8)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/171489
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=34fcd737c1de3e3039ec48e4f2261ca61331b8ba
Submitter: Jenkins
Branch: juno

commit 34fcd737c1de3e3039ec48e4f2261ca61331b8ba
Author: Matt Thompson <email address hidden>
Date: Thu Apr 2 09:21:16 2015 +0100

    Fix bug in playbooks/library/neutron

    This change updates _add_router_interface to list ports for the router
    device only, and not for all devices. As we select specific device
    ports only, we no longer need to skip over other device's ports, and
    this indirectly fixes a bug where we were using 'next' instead of
    'continue' to skip to the next iteration of the for loop.

    Change-Id: I892dc0f58f1d8bbb7ebd46f7790c5420c4c8b94b
    Closes-Bug: #1439295
    (cherry picked from commit 8bd358d2c25f9bfb6011f751ef5342e012fbb6d8)