NSX v3 retries conf property used for multiple semantics

Bug #1540885 reported by Boden R
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
vmware-nsx
Fix Released
Undecided
Boden R

Bug Description

The 'retries' property in the nsx_v3 group of the nsx.ini is currently used in 2 contexts within the nsx v3 plugin:
(a) As the number of times to retry a request IFF the request fails due to StaleRevision exception.
(b) The number of HTTP retries used in the nsx v3 client logic.

We need to not only clarify this usage in the conf, but also introduce a separate conf prop for HTTP retries.

Boden R (boden)
Changed in vmware-nsx:
assignee: nobody → Boden R (boden)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to vmware-nsx (master)

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

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

Reviewed: https://review.openstack.org/275178
Committed: https://git.openstack.org/cgit/openstack/vmware-nsx/commit/?id=111415d06ef8bda609bfd6a06f472f6c6bb905ef
Submitter: Jenkins
Branch: master

commit 111415d06ef8bda609bfd6a06f472f6c6bb905ef
Author: Boden R <email address hidden>
Date: Tue Feb 2 05:59:05 2016 -0700

    NSX-v3 HTTP retries conf property

    This patch adds a new conf property to the nsx.ini under the
    nsx_v3 group called http_retries. This value is used for the
    max retries used on HTTP requests. Additionally this patch
    clarifies the use of http_retries vs retries in the conf properties.
    A unit test is also included.

    backport: liberty

    Change-Id: Ifb0d1aaa6a11163f6520a6472ef743e3eab3ce38
    Closes-Bug: #1540885

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

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/277904

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

Reviewed: https://review.openstack.org/277904
Committed: https://git.openstack.org/cgit/openstack/vmware-nsx/commit/?id=8753ef96a3b30e23e640d554996969a6383b7084
Submitter: Jenkins
Branch: stable/liberty

commit 8753ef96a3b30e23e640d554996969a6383b7084
Author: Boden R <email address hidden>
Date: Tue Feb 2 05:59:05 2016 -0700

    NSX-v3 HTTP retries conf property

    This patch adds a new conf property to the nsx.ini under the
    nsx_v3 group called http_retries. This value is used for the
    max retries used on HTTP requests. Additionally this patch
    clarifies the use of http_retries vs retries in the conf properties.
    A unit test is also included.

    backport: liberty

    Change-Id: Ifb0d1aaa6a11163f6520a6472ef743e3eab3ce38
    Closes-Bug: #1540885
    (cherry picked from commit 111415d06ef8bda609bfd6a06f472f6c6bb905ef)

tags: added: in-stable-liberty
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.