Merge lp://qastaging/~canonical-platform-qa/utah/add-status-to-yaml into lp://qastaging/utah

Proposed by Max Brustkern
Status: Merged
Approved by: Max Brustkern
Approved revision: 1101
Merged at revision: 1101
Proposed branch: lp://qastaging/~canonical-platform-qa/utah/add-status-to-yaml
Merge into: lp://qastaging/utah
Diff against target: 35 lines (+11/-0)
1 file modified
utah/client/common.py (+11/-0)
To merge this branch: bzr merge lp://qastaging/~canonical-platform-qa/utah/add-status-to-yaml
Reviewer Review Type Date Requested Status
Sergio Cazzolato (community) Approve
Canonical CI Engineering Pending
Review via email: mp+320415@code.qastaging.launchpad.net

Commit message

Add status to run_cmd data.

Description of the change

This branch adds a status element to the return of run_cmd. It is PASS for retcode 0, FAIL for retcode nonzero if the cmd_type is testcase_test, and ERROR for retcode nonzero for other cmd_types.

To post a comment you must log in.
Revision history for this message
Sergio Cazzolato (sergio-j-cazzolato) wrote :

code lgtm, approving

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches