UTAH

Merge lp://qastaging/~canonical-platform-qa/utah/print-test-status into lp://qastaging/utah

  1. print-test-status
  2. Merge into dev
Proposed by Sergio Cazzolato
Status: Merged
Merged at revision: 1096
Proposed branch: lp://qastaging/~canonical-platform-qa/utah/print-test-status
Merge into: lp://qastaging/utah
Diff against target: 252 lines (+180/-13)
3 files modified
utah/client/const.py (+25/-0)
utah/client/result.py (+19/-13)
utah/yaml2xunit.py (+136/-0)
To merge this branch: bzr merge lp://qastaging/~canonical-platform-qa/utah/print-test-status
Reviewer Review Type Date Requested Status
Max Brustkern Pending
Canonical CI Engineering Pending
Review via email: mp+317702@code.qastaging.launchpad.net

Commit message

change to print test status and export yaml test results to junit format.

Description of the change

This change is adding a new dependency, to install it just run:
> sudo pip install junit_xml

To post a comment you must log in.
lp://qastaging/~canonical-platform-qa/utah/print-test-status updated
1094. By Sergio Cazzolato

Adding yaml to xunit converted

1095. By Sergio Cazzolato

Fixing flake8 errors

Revision history for this message
Max Brustkern (nuclearbob) wrote :

If fetch or setup steps fail, we may have a yaml file with no testcase_test commands. Will that pose a problem for grafana? Also, in the case where provisioning fails, we won't have a yaml file, so it would be useful to know what we want the junit xml to look like in that case, although that may end up landing in lp:qa-jenkins-jobs. Let me know if you want me to propose a patch for the script packaging, but with that and the typo fix in place, we should be good.

lp://qastaging/~canonical-platform-qa/utah/print-test-status updated
1096. By Sergio Cazzolato

Adding +x to python file, fixing typo and considering the scenario when the yaml file does not exist

Revision history for this message
Sergio Cazzolato (sergio-j-cazzolato) wrote :

I have added a change to support the scenario when the yaml file does not exist. So in that case and when there are tnot any testcase_test comman, the xml file will be like empty (<testsuites />). So then when we process that for grafana, currently it wont sending any information, but in the future we can indicate that this build is failed.

Revision history for this message
Max Brustkern (nuclearbob) wrote :

If you apply this patch, we should get the script installed as part of the package:
=== modified file 'debian/utah.install'
--- debian/utah.install 2013-02-20 18:22:37 +0000
+++ debian/utah.install 2017-02-21 20:39:58 +0000
@@ -6,3 +6,4 @@
 utah-common_*_all.deb usr/share/utah
 python-jsonschema_*_all.deb usr/share/utah
 templates usr/share/utah
+utah/yaml2xunit.py usr/bin

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