Bug 1384609 - RFE: Download/generate the SRPM only once per build request
Summary: RFE: Download/generate the SRPM only once per build request
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Copr
Classification: Community
Component: backend
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Miroslav Suchý
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1420540
TreeView+ depends on / blocked
 
Reported: 2016-10-13 16:22 UTC by John Ellson
Modified: 2017-06-16 12:04 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-06-16 12:04:04 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1310868 0 unspecified CLOSED "Mock SCM" builds print "This field is required." 2021-02-22 00:41:40 UTC
Red Hat Bugzilla 1412658 0 unspecified CLOSED Exception raised during srpm import during build using Mock SCM 2021-02-22 00:41:40 UTC

Internal Links: 1310868 1412658

Description John Ellson 2016-10-13 16:22:45 UTC
Description of problem:
Graphviz has a distroagnostic src.rpm for el[67],fc2[3456]
but COPR downloads the same src.rpm 6 times

Version-Release number of selected component (if applicable):


How reproducible:
100%

Steps to Reproduce:
1. copr-cli build graphviz http://john.ellson.com/copr/graphviz-2.39.20161012.1437-1.src.rpm
2.
3.

Actual results:
src.rpm downloaded 6 stimes

Expected results:
src.rpm downloaded 1 time

Additional info:

Comment 1 Pavel Raiskup 2017-01-17 14:53:32 UTC
Stealing the bug report to make it a bit more general.

This becomes extremely important WRT mock-scm/tito builds;  where not only
download happens multiple times, but also docker container (doing mock
--srpmbuildm thus installing minimal buildroot) happens N-times, I guess
in all cases this happens on "dist-git" machine (not on builders) so this
causes huge load.

Comment 2 Pavel Raiskup 2017-01-23 09:41:58 UTC
Looks like to OOM bug 1412658 is related.

Comment 3 Pavel Raiskup 2017-06-16 12:04:04 UTC
Fixed by https://pagure.io/copr/copr/pull-request/70 and deployed to
production.


Note You need to log in before you can comment on or make changes to this bug.