David Caro has posted comments on this change.
Change subject: automation: Enable xunit report ......................................................................
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/46640/4/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/" ;
How about:
you can do whatever you want in the scripts, if you want to have the common.sh for common stuff go for it, as long as the ci can run the script without params or custom env vars it's perfectly fine Line 20: find "$PWD" \ Line 21: -maxdepth 1 \ Line 22: -iname vdsm*.tar.gz \