DeferredVersionTestCase failure

Bug #1087575 reported by Mark McLoughlin
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo-incubator
Fix Released
High
Davanum Srinivas (DIMS)
Grizzly
Fix Released
High
Davanum Srinivas (DIMS)

Bug Description

See e.g. http://logs.openstack.org/17605/1/gate/gate-oslo-incubator-python26/244/console.html

 06:41:16 FAIL: test_print_deferred_version (tests.unit.test_version.DeferredVersionTestCase)
 06:41:16 ----------------------------------------------------------------------
 06:41:16 Traceback (most recent call last):
 06:41:16 File "/home/jenkins/workspace/gate-oslo-incubator-python26/tests/unit/test_version.py", line 68, in test_print_deferred_version
 06:41:16 self.assertEquals("5.5.5.5", sys.stderr.getvalue().strip())
 06:41:16 AssertionError: '5.5.5.5' != '5.5.5.'
 06:41:16 >> raise self.failureException, \
 06:41:16 (None or '%r != %r' % ('5.5.5.5', '5.5.5.'))

Revision history for this message
Mark McLoughlin (markmc) wrote :

Here's where those tests got merged: https://review.openstack.org/17439

Interesting that Jenkins didn't report any failures there

Changed in oslo:
assignee: nobody → Davanum Srinivas (dims-v)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo-incubator (master)

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

Changed in oslo:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

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

Reviewed: https://review.openstack.org/17887
Committed: http://github.com/openstack/oslo-incubator/commit/46da5746b0af23640e7b6a9a7f90494916a8aadc
Submitter: Jenkins
Branch: master

commit 46da5746b0af23640e7b6a9a7f90494916a8aadc
Author: Davanum Srinivas <email address hidden>
Date: Tue Dec 11 18:05:03 2012 -0500

    fix intermittent failures with deferred version test

    Need to get rid of the deferred string approach as
    the problem was overriding string methods is hard and
    flaky. There is just no other documented
    argparse hook to allow deferring of the version
    calculation as version passed in has to be a str
    or buffer. Added a test to try longer string and even
    run it in a loop just to be sure.

    Fixes LP #1087575

    Change-Id: Icf83944d67b8ce5d3230627f3de10501da3276f2

Changed in oslo:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in oslo:
milestone: none → grizzly-2
status: Fix Committed → Fix Released
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.