Bug 1314920

Summary:	No logs when Git or Mock SCM import fails
Product:	[Community] Copr	Reporter:	Daniel Vrátil <me>
Component:	backend	Assignee:	clime
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	unspecified	CC:	clime, jkadlcik
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-03-29 12:18:01 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Daniel Vrátil 2016-03-04 21:25:16 UTC

Description of problem:
When import fails, the build details page says

   "Import failed. See logs for more details. "

However there are no logs for the build available.

The obvious simple fix is to remove the "See logs...." sentence, however it would be much much much better if there were actual logs available, since it would make figuring out what went wrong much easier.

I had a problem with my Mock SCM repo having different name than the actual package and I spent almost an hour figuring what's wrong, until I eventually found the relevant code in Copr source and realized what's wrong - having the mock log available would've saved me bunch of time....

How reproducible:
always (when there's an import error)


Steps to Reproduce:
1. Have an error in Mock SCM data (wrong URL, invalid .spec path ...)
2. start build

Actual results:
Import fails, no logs available


Expected results:
Import fails, logs available :)

Comment 1 clime 2016-03-08 12:14:46 UTC

Thank you for the suggestion. The added logs might prove quite useful.

Fixed by 4d6656bcf8d & 7623ba57f7.

Comment 2 Jakub Kadlčík 2016-03-15 08:51:48 UTC

Code with this fix is currently deployed on testing server.
You can test it before it will be deployed to production.
Please see: http://copr-fe-dev.cloud.fedoraproject.org

Comment 3 Jakub Kadlčík 2016-03-29 12:18:01 UTC

Copr production machines were upgraded to version containing a fix to this issue.
If you are not satisfied with provided solution, please feel free to reopen this.