Dan Kenigsberg has posted comments on this change.

Change subject: automation: Create coverage report sooner
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.ovirt.org/#/c/65414/1//COMMIT_MSG
Commit Message:

Line 8: 
Line 9: For some reason a build artifacts step was inserted before creating the
Line 10: coverage report. There is no reason not to create the report 
immediately
Line 11: after running the tests, and jenkins seems to fail in an unclear way
Line 12: just after creating the report.
I don't understand the English here. Nor the importance of creating the report 
before creating RPMs.
Line 13: 
Line 14: Change-Id: I8688383eb5e99fdd0ee48815cc71afd6483819b4


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8688383eb5e99fdd0ee48815cc71afd6483819b4
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Edward Haas <edwa...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to