Change in vdsm[master]: automation: Keep xunit report

dcaroest at redhat.com dcaroest at redhat.com
Wed Oct 7 14:19:17 UTC 2015


David Caro has posted comments on this change.

Change subject: automation: Keep xunit report
......................................................................


Patch Set 3:

(1 comment)

https://gerrit.ovirt.org/#/c/46643/3/automation/build-artifacts.sh
File automation/build-artifacts.sh:

Line 15: make PYFLAKES=true PEP8=true NOSE_EXCLUDE=.* rpm
Line 16: 
Line 17: find "$BUILDS" \
Line 18:     -iname \*.rpm \
Line 19:     -exec mv {} "$EXPORTED_ARTIFACTS/" \;
his will not work, as build-artifacts.sh will be run independently too (the fact that check-patch runs it is to avoid duplicating the code).

Any of the three main scripts (check-patch, check-merged and build-artifacts) should be able to run without any parameters or expecting any env vars (aside from posix ones, $PWD $HOME and such)
Line 20: find "$PWD" \
Line 21:     -maxdepth 1 \
Line 22:     -iname vdsm\*.tar.gz \


-- 
To view, visit https://gerrit.ovirt.org/46643
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia6eaed1a42e67059e457969d785e70b08efb5f2f
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list