Neutron should retry http connections when notifying e.g. nova

Bug #1883712 reported by Slawek Kaplonski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Wishlist
Brian Haley

Bug Description

In case of failure during sending notification from neutron to nova (or ironic), neutron should retry such connection few times.
We can add new config option to allow to tune how many times it should be retried.

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

Fix proposed to branch: master
Review: https://review.opendev.org/736026

Changed in neutron:
status: Confirmed → In Progress
Changed in neutron:
assignee: Slawek Kaplonski (slaweq) → Brian Haley (brian-haley)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.opendev.org/736026
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=e94511cd251299cee84b8b67d404972988ae28d3
Submitter: Zuul
Branch: master

commit e94511cd251299cee84b8b67d404972988ae28d3
Author: Slawek Kaplonski <email address hidden>
Date: Tue Jun 16 22:39:01 2020 +0200

    Add config option ``http_retries``

    This option allows to configure Number of times nova or ironic client
    should retry on any failed http call.
    Default value for this new option is "3".

    Change-Id: I795ee7ca729646be0411a1232bf218015c65010f
    Closes-Bug: #1883712

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/ussuri)

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/740214

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/train)

Fix proposed to branch: stable/train
Review: https://review.opendev.org/740215

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/stein)

Fix proposed to branch: stable/stein
Review: https://review.opendev.org/740216

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/740220

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.opendev.org/740222

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/rocky)

Reviewed: https://review.opendev.org/740220
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=6bd6847a796c10742ddc6dad166df2c0bb5de636
Submitter: Zuul
Branch: stable/rocky

commit 6bd6847a796c10742ddc6dad166df2c0bb5de636
Author: Slawek Kaplonski <email address hidden>
Date: Tue Jun 16 22:39:01 2020 +0200

    Add config option ``http_retries``

    This option allows to configure Number of times nova or ironic client
    should retry on any failed http call.
    Default value for this new option is "3".

    Conflicts:
        neutron/notifiers/ironic.py
        neutron/notifiers/nova.py
        neutron/tests/unit/notifiers/test_nova.py

    Change-Id: I795ee7ca729646be0411a1232bf218015c65010f
    Closes-Bug: #1883712
    (cherry picked from commit e94511cd251299cee84b8b67d404972988ae28d3)

tags: added: in-stable-rocky
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/train)

Reviewed: https://review.opendev.org/740215
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=76e651005b62888090dfe097b7fe03c62cdd32f1
Submitter: Zuul
Branch: stable/train

commit 76e651005b62888090dfe097b7fe03c62cdd32f1
Author: Slawek Kaplonski <email address hidden>
Date: Tue Jun 16 22:39:01 2020 +0200

    Add config option ``http_retries``

    This option allows to configure Number of times nova or ironic client
    should retry on any failed http call.
    Default value for this new option is "3".

    Change-Id: I795ee7ca729646be0411a1232bf218015c65010f
    Closes-Bug: #1883712
    (cherry picked from commit e94511cd251299cee84b8b67d404972988ae28d3)

tags: added: in-stable-train
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/stein)

Reviewed: https://review.opendev.org/740216
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=5d06db0989a46608eb699f401c1b02aaf070c623
Submitter: Zuul
Branch: stable/stein

commit 5d06db0989a46608eb699f401c1b02aaf070c623
Author: Slawek Kaplonski <email address hidden>
Date: Tue Jun 16 22:39:01 2020 +0200

    Add config option ``http_retries``

    This option allows to configure Number of times nova or ironic client
    should retry on any failed http call.
    Default value for this new option is "3".

    Conflicts:
        neutron/notifiers/ironic.py

    Change-Id: I795ee7ca729646be0411a1232bf218015c65010f
    Closes-Bug: #1883712
    (cherry picked from commit e94511cd251299cee84b8b67d404972988ae28d3)

tags: added: in-stable-stein
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/ussuri)

Reviewed: https://review.opendev.org/740214
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=34d4a8bb9907a41bc3aee8398c68a54f0e91fe1d
Submitter: Zuul
Branch: stable/ussuri

commit 34d4a8bb9907a41bc3aee8398c68a54f0e91fe1d
Author: Slawek Kaplonski <email address hidden>
Date: Tue Jun 16 22:39:01 2020 +0200

    Add config option ``http_retries``

    This option allows to configure Number of times nova or ironic client
    should retry on any failed http call.
    Default value for this new option is "3".

    Change-Id: I795ee7ca729646be0411a1232bf218015c65010f
    Closes-Bug: #1883712
    (cherry picked from commit e94511cd251299cee84b8b67d404972988ae28d3)

tags: added: in-stable-ussuri
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/queens)

Reviewed: https://review.opendev.org/740222
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=290c3a9885c5c02e9dd3d6505473fc3f08207991
Submitter: Zuul
Branch: stable/queens

commit 290c3a9885c5c02e9dd3d6505473fc3f08207991
Author: Slawek Kaplonski <email address hidden>
Date: Tue Jun 16 22:39:01 2020 +0200

    Add config option ``http_retries``

    This option allows to configure Number of times nova or ironic client
    should retry on any failed http call.
    Default value for this new option is "3".

    Conflicts:
        neutron/notifiers/ironic.py
        neutron/notifiers/nova.py
        neutron/tests/unit/notifiers/test_nova.py

    Change-Id: I795ee7ca729646be0411a1232bf218015c65010f
    Closes-Bug: #1883712
    (cherry picked from commit e94511cd251299cee84b8b67d404972988ae28d3)

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron queens-eol

This issue was fixed in the openstack/neutron queens-eol release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron rocky-eol

This issue was fixed in the openstack/neutron rocky-eol release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.