manila-tempest-dsvm-scenario job does not fail if individual tests fail

Bug #1770183 reported by Goutham Pacha Ravi
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Shared File Systems Service (Manila)
Fix Released
High
Goutham Pacha Ravi

Bug Description

A failure that was reported as a successful job run:

Job Results: https://review.openstack.org/#/c/544519/12
Failures: http://logs.openstack.org/19/544519/12/check/manila-tempest-dsvm-scenario/0c5b584/logs/testr_results.html.gz (gz of all logs attached below)

Revision history for this message
Goutham Pacha Ravi (gouthamr) wrote :
Changed in manila:
assignee: nobody → Goutham Pacha Ravi (gouthamr)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to manila (master)

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

Changed in manila:
status: New → In Progress
Revision history for this message
Goutham Pacha Ravi (gouthamr) wrote :

Another instance of this issue:

All tempest API test jobs are meant to fail on https://review.openstack.org/#/c/568393/1 because the code change that it tests hasn't merged in manila yet (https://review.openstack.org/#/c/568364/). However, some of the jobs posted "SUCCESS", despite test failures.

Example:
http://logs.openstack.org/93/568393/1/check/manila-tempest-dsvm-postgres-container/366dc5c/job-output.txt.gz

The RCA:
- The issue occurs because we don't seem to be saving the execution result of the tempest test invocation correctly. Please see https://review.openstack.org/568650 for the fix.

Changed in manila:
importance: Undecided → High
milestone: none → rocky-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to manila (master)

Reviewed: https://review.openstack.org/568650
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=a0947ab42d9f37236c1f0784bd39346b04f8b6a8
Submitter: Zuul
Branch: master

commit a0947ab42d9f37236c1f0784bd39346b04f8b6a8
Author: Goutham Pacha Ravi <email address hidden>
Date: Tue May 15 10:36:08 2018 -0700

    Fix post-execution for tempest tests

    Manila's post_test_hook runs tempest after
    populating tempest.conf with the requisite
    options, the results from tempest are saved
    prior to performing some post tasks, including
    deciding to run tests again with a different
    configuration if necessary
    (Ex: manila-tempest-minimal-dsvm-dummy). The
    result is then replayed at the end of the gate
    job so it can be reported to gerrit as a
    SUCCESS or a FAILURE.

    Fix the mis-aligned command that saves the
    test execution status to replay later.

    Change-Id: I1578760697aeae6e20bf567c60fbcdee0a642bee
    Closes-Bug: 1770183

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to manila (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/578772

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to manila (stable/queens)

Reviewed: https://review.openstack.org/578772
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=a6f4e23d26b7e7e054ad29ea572a0f23f83083a5
Submitter: Zuul
Branch: stable/queens

commit a6f4e23d26b7e7e054ad29ea572a0f23f83083a5
Author: Goutham Pacha Ravi <email address hidden>
Date: Tue May 15 10:36:08 2018 -0700

    Fix post-execution for tempest tests

    Manila's post_test_hook runs tempest after
    populating tempest.conf with the requisite
    options, the results from tempest are saved
    prior to performing some post tasks, including
    deciding to run tests again with a different
    configuration if necessary
    (Ex: manila-tempest-minimal-dsvm-dummy). The
    result is then replayed at the end of the gate
    job so it can be reported to gerrit as a
    SUCCESS or a FAILURE.

    Fix the mis-aligned command that saves the
    test execution status to replay later.

    Change-Id: I1578760697aeae6e20bf567c60fbcdee0a642bee
    Closes-Bug: 1770183
    (cherry picked from commit a0947ab42d9f37236c1f0784bd39346b04f8b6a8)

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/manila 6.0.2

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