The creation fip does not endure restarting of l3-agent

[QingchuanHao]

when creating the first floating ip of a router, the veth-pair(fpr and rfp) will be created. But if I3-agent is restarted accidentally before assigning locally allocated ip for the fpr or rfp, gateway will not be added successfully with and error raised

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/302340

[Carl Baldwin (carl-baldwin)]

It is helpful to have the details of the error raised. But, I'll go look at the fix. Maybe it will be apparent.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/309591

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/309592

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/309591
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=b9bec4be6ba06087ebfa20007dcbc2cb646a5345
Submitter: Jenkins
Branch: master

commit b9bec4be6ba06087ebfa20007dcbc2cb646a5345
Author: Brian Haley <email address hidden>
Date: Fri Apr 22 15:43:24 2016 -0400

    DVR: Use IPDevice class consistently

    create_rtr_2_fip_link() only creates IPDevice class instances
    after it determines the VETH pair does not exist. Change to
    always create them as instead of using int_dev[0]. This is
    also setup for a follow-on patch.

    Partial-Bug: #1566383
    Change-Id: I58e242f05ffb9a33bda5aeb93344861769845d2e

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/309592
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=17cfffbbfa57ccec54c7e06518b5e7e8a00e0c51
Submitter: Jenkins
Branch: master

commit 17cfffbbfa57ccec54c7e06518b5e7e8a00e0c51
Author: Brian Haley <email address hidden>
Date: Fri Apr 22 16:09:07 2016 -0400

    DVR: Use existing IPDevice to add address on FIP VETH

    create_rtr_2_fip_link() has IPDevice instances for each
    end of the VETH pair it created, just use them to add the
    link-local IP to each one. Because this is a point-to-point
    link, we don't add the broadcast address when we assign the IP,
    so add a flag to IpAddrCommand() class to accomodate us.
    This is also setup for a follow-on patch.

    Partial-Bug: #1566383
    Change-Id: I66d70afaf3c52861e0887d3f535d5327ea46b128

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/302340
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=a47d22942a46cf12589cee6353dc2b6deb58f178
Submitter: Jenkins
Branch: master

commit a47d22942a46cf12589cee6353dc2b6deb58f178
Author: Brian Haley <email address hidden>
Date: Thu May 5 10:07:36 2016 -0400

    DVR: Ensure fpr and rfp devices are configured correctly

    Restarting the l3-agent when creating the fip namespace may
    leave rfp and fpr devices with no IP addresses, since we will
    not try and configure them if the devices already exist.

    Fix this by always trying to configure the IP addresses,
    even if the devices already exist.

    Closes-Bug: #1566383
    Change-Id: Idad3bb4808ec66b9a764bb621dfb04a10eb7ed6

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/314722

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/314762

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/314767

[OpenStack Infra (hudson-openstack)]

Change abandoned by Swaminathan Vasudevan (<email address hidden>) on branch: stable/mitaka
Review: https://review.openstack.org/314767

[OpenStack Infra (hudson-openstack)]

Change abandoned by Swaminathan Vasudevan (<email address hidden>) on branch: stable/mitaka
Review: https://review.openstack.org/314762
Reason: abandon

[OpenStack Infra (hudson-openstack)]

Change abandoned by Swaminathan Vasudevan (<email address hidden>) on branch: stable/mitaka
Review: https://review.openstack.org/314722
Reason: adandon

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/314790

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/314801

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/314804

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/320098

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/320101

[OpenStack Infra (hudson-openstack)]

Change abandoned by Swaminathan Vasudevan (<email address hidden>) on branch: stable/mitaka
Review: https://review.openstack.org/314790
Reason: Alternate patch with correct change-id uploaded.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Swaminathan Vasudevan (<email address hidden>) on branch: stable/mitaka
Review: https://review.openstack.org/314801
Reason: Alternate patch with correct change-id uploaded.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/320098
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=5d57c7c89e3407cbb17a1e30e67dc76d5168867d
Submitter: Jenkins
Branch: stable/mitaka

commit 5d57c7c89e3407cbb17a1e30e67dc76d5168867d
Author: Brian Haley <email address hidden>
Date: Fri Apr 22 15:43:24 2016 -0400

    DVR: Use IPDevice class consistently

    create_rtr_2_fip_link() only creates IPDevice class instances
    after it determines the VETH pair does not exist. Change to
    always create them as instead of using int_dev[0]. This is
    also setup for a follow-on patch.

    Partial-Bug: #1566383
    Change-Id: I58e242f05ffb9a33bda5aeb93344861769845d2e
    (cherry picked from commit b9bec4be6ba06087ebfa20007dcbc2cb646a5345)
    Conflicts:
     neutron/agent/l3/dvr_fip_ns.py

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/320101
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=b6d99239cf12291169305b7f79dda21e88c670ee
Submitter: Jenkins
Branch: stable/mitaka

commit b6d99239cf12291169305b7f79dda21e88c670ee
Author: Brian Haley <email address hidden>
Date: Fri Apr 22 16:09:07 2016 -0400

    DVR: Use existing IPDevice to add address on FIP VETH

    create_rtr_2_fip_link() has IPDevice instances for each
    end of the VETH pair it created, just use them to add the
    link-local IP to each one. Because this is a point-to-point
    link, we don't add the broadcast address when we assign the IP,
    so add a flag to IpAddrCommand() class to accomodate us.
    This is also setup for a follow-on patch.

    Partial-Bug: #1566383
    Change-Id: I66d70afaf3c52861e0887d3f535d5327ea46b128
    (cherry picked from commit 17cfffbbfa57ccec54c7e06518b5e7e8a00e0c51)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/314804
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=18bb187632511191cb8c84133462cc838c6fc67c
Submitter: Jenkins
Branch: stable/mitaka

commit 18bb187632511191cb8c84133462cc838c6fc67c
Author: Brian Haley <email address hidden>
Date: Thu May 5 10:07:36 2016 -0400

    DVR: Ensure fpr and rfp devices are configured correctly

    Restarting the l3-agent when creating the fip namespace may
    leave rfp and fpr devices with no IP addresses, since we will
    not try and configure them if the devices already exist.

    Fix this by always trying to configure the IP addresses,
    even if the devices already exist.

    Closes-Bug: #1566383
    (cherry picked from commit a47d22942a46cf12589cee6353dc2b6deb58f178)

    Change-Id: Idad3bb4808ec66b9a764bb621dfb04a10eb7ed6

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/neutron 9.0.0.0b1 development milestone.

[Davanum Srinivas (DIMS) (dims-v)]

This issue was fixed in the openstack/neutron 8.1.2 release.