Caching the client can cause the int-tests to fail on slow nodes

Bug #1647002 reported by Peter Stachowski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack DBaaS (Trove)
Fix Released
Undecided
Peter Stachowski

Bug Description

If a node is really slow then a single test can take longer than an hour. It will then fail as client it has cached will now have an expired token. To alleviate this, the caching of the clients should be removed (since the time saved on caching the clients is probably negligible anyway).

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

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

Changed in trove:
assignee: nobody → Peter Stachowski (peterstac)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to trove (master)

Reviewed: https://review.openstack.org/406503
Committed: https://git.openstack.org/cgit/openstack/trove/commit/?id=d558e5c2e0b02a9f3f52e1370bdfe5b1495a4b31
Submitter: Jenkins
Branch: master

commit d558e5c2e0b02a9f3f52e1370bdfe5b1495a4b31
Author: Peter Stachowski <email address hidden>
Date: Sat Dec 3 18:25:19 2016 +0000

    Stop caching client to fix long-running test fails

    If a node is really slow then a single test can take longer than an
    hour. It will then fail as the client it has cached will now have an
    expired token. To alleviate this, the caching of the clients was removed
    (since the time saved on caching the clients is probably negligible
    anyway).

    Checking the client code was also cleaned up for a number of tests.

    The cluster tests were modified, however the code was reverted as
    there is no way to test it at present (they will not run as they
    haven't been modified to work with the new neutron set up). This
    will be fixed in a separate changereq (since they are currently not
    being tested in the gate, this should not hold up development).

    Change-Id: I63d5b78084486b806bff38068260e42128c0d0ed
    Closes-Bug: 1647002

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

This issue was fixed in the openstack/trove 7.0.0.0b2 development milestone.

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.