Copy logs on test failure

Bug #1542390 reported by hongbin
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
Critical
Dimitry Ushakov

Bug Description

Quoted from @Egor:

I think we should do more investigation before applying any "hot" patches. E.g. I look at several failures today and honestly there is no way to find out reasons.
I believe we are not copying logs (https://github.com/openstack/magnum/blob/master/magnum/tests/functional/python_client_base.py#L163) during test failure,
we register handler at setUp (https://github.com/openstack/magnum/blob/master/magnum/tests/functional/python_client_base.py#L244), but Swarm tests, create
bay in setUpClass (https://github.com/openstack/magnum/blob/master/magnum/tests/functional/swarm/test_swarm_python_client.py#L48) which called before setUp.
So there is no way to see any logs from vm.

hongbin (hongbin034)
Changed in magnum:
importance: Undecided → High
Adrian Otto (aotto)
Changed in magnum:
status: New → Triaged
importance: High → Critical
Dimitry Ushakov (dimalg)
Changed in magnum:
assignee: nobody → Dimitry Ushakov (dimalg)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to magnum (master)

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

Changed in magnum:
status: Triaged → In Progress
Changed in magnum:
assignee: Dimitry Ushakov (dimalg) → Corey O'Brien (coreypobrien)
Changed in magnum:
assignee: Corey O'Brien (coreypobrien) → Dimitry Ushakov (dimalg)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

Reviewed: https://review.openstack.org/278127
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=7d524491c8c6a4be3ece80bfe20e0e26d5ebd143
Submitter: Jenkins
Branch: master

commit 7d524491c8c6a4be3ece80bfe20e0e26d5ebd143
Author: dimtruck <email address hidden>
Date: Tue Feb 9 16:25:27 2016 -0600

    Copy logs on test failure

    This patch will copy logs from nova servers to local environment
    on test failures.
    Don't delete bays until teardown so log gathering will work.
    Give functional test types a common base.

    Added timeout to docker ps; else, the command would time out,
    failing to run the rest of the tests.

    Closes-Bug: 1542390
    Change-Id: I016ed8fe311bede407f57b7982126c4b77749bec
    Co-Authored-By: Corey O'Brien <email address hidden>

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

This issue was fixed in the openstack/magnum 2.0.0 release.

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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