NSX: Correct default timeout params

Bug #1352074 reported by OpenStack Infra
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-manuals
Fix Released
Medium
Gauvain Pocentek

Bug Description

https://review.openstack.org/106494
commit 0b7a7687309e8c740ccf79fbbbcabbc64b6f5fd1
Author: Aaron Rosen <email address hidden>
Date: Fri Jul 11 13:56:15 2014 -0700

    NSX: Correct default timeout params

    Previously, req_timeout and http_timeout were set to the same value
    which is not correct. req_timeout is the total time limit for a cluster
    request and http_timeout is the time allowed before aborting a request on
    an unresponsive controller. Since the default configuration allows 2
    retries req_timeout should be double that of http_timeout because of this
    this patch goes ahead and removes req_timeout as this should just be
    http_timeout * retries.

    Because prevouly req_timeout and http_timeout were the same this exposed
    a corner case that when the nsx controller returned a 307 we would issue
    the request against the redirected controller but in the case where the
    session cookie had expire when the request was issued we would get a 401
    response back and never retry the request. Now that the default values are
    corrected this issue should no longer occur as the next time time we issue
    the request we'll fetch a new auth cookie for the redirected controller.

    This patch also bumps the timeout values to be higher. We've seen
    more and more timeouts occur in our CI system largely because our
    cloud is overloaded so increasing the default timeouts will *hopefully*
    help reduce test failures.

    DocImpact

    Closes-bug: 1340969
    Closes-bug: 1338846

    Change-Id: Id7244cd4d9316931f4f7df1c3b41b3a894f2909a

Revision history for this message
Andreas Jaeger (jaegerandi) wrote :

will be changed with the next import of our tables

Changed in openstack-manuals:
status: New → Confirmed
importance: Undecided → Medium
tags: added: autogenerate-config-docs config-reference
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-manuals (master)

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

Changed in openstack-manuals:
assignee: nobody → Gauvain Pocentek (gpocentek)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-manuals (master)

Reviewed: https://review.openstack.org/115256
Committed: https://git.openstack.org/cgit/openstack/openstack-manuals/commit/?id=8b2016b5d035b3475fe197b2132a8e1e8abff350
Submitter: Jenkins
Branch: master

commit 8b2016b5d035b3475fe197b2132a8e1e8abff350
Author: Gauvain Pocentek <email address hidden>
Date: Tue Aug 19 11:22:22 2014 +0200

    Update the config reference tables

    Include the new tables in the documentation.

    Fix the cinder-prophetstor_dpl.xml filename and the xiv category.

    Manually fix those files for niceness test:
    - glance-logging.xml (lines 34, 38), fixed in oslo-incubator
    - trove-logging.xml (lines 26, 30), fixed in oslo-incubator
    - neutron-openvswitch_agent.xml (line 45), to be fixed in neutron

    Closes-Bug: #1340858
    Closes-Bug: #1344231
    Closes-Bug: #1345956
    Closes-Bug: #1346711
    Closes-Bug: #1347978
    Partial-Bug: #1348329
    Closes-Bug: #1352074
    Partial-Bug: #1353417
    Closes-Bug: #1354622
    Closes-Bug: #1339754
    Closes-Bug: #1358598
    Closes-Bug: #1358259
    Closes-Bug: #1357865
    Partial-Bug: #1357457
    Closes-Bug: #1357421
    Change-Id: Id2da7d7762ca954bd552dbf89a9ff28b144efb68

Changed in openstack-manuals:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-manuals 15.0.0

This issue was fixed in the openstack/openstack-manuals 15.0.0 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.