tempest should log req-id for failed tests to make debugging easier

Bug #1210639 reported by Joe Gordon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tempest
Fix Released
Medium
Matthew Treinish

Bug Description

even though nova returns the req-id of each call, tempest doesn't record them. These req-id are returned under 'x-compute-request-id' and can be used to search the nova logs to find the logs related to that request.

If the req-ids are printed to stdout testr will record that information and only print stdout for failed tests.

Joe Gordon (jogo)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tempest (master)

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

Changed in tempest:
assignee: nobody → Matthew Treinish (treinish)
status: New → In Progress
Changed in tempest:
importance: Undecided → Medium
Revision history for this message
Attila Fazekas (afazekas) wrote :

We recently switched to oslo logging. I do not really follow why to use print if we have loggers.

Normally python programs are using loggers for printing.

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

Reviewed: https://review.openstack.org/41934
Committed: http://github.com/openstack/tempest/commit/e5423911e117bf760fc702c03fd6a40af4d60048
Submitter: Jenkins
Branch: master

commit e5423911e117bf760fc702c03fd6a40af4d60048
Author: Matthew Treinish <email address hidden>
Date: Tue Aug 13 18:07:31 2013 -0400

    Log request ids from glance and nova

    This commit adds an option a log message at the info level to
    print the req-id returned from nova and glance. This can be useful
    when trying to debug test failures because it provides you with a
    unique identifier in the nova and glance logs for request
    information. This is done at the info level so that it will be
    included in the subunit stream to be printed out during a test
    failure to help with debugging.

    Fixes bug 1210639

    Change-Id: I819f94c538e6a15732dbb4b5c2dafdc270e0b2b4

Changed in tempest:
status: In Progress → Fix Committed
Sean Dague (sdague)
Changed in tempest:
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.