Bug 1465779 - aspnetcore.all meta package restore slows down s2i build and consumes bandwidth [NEEDINFO]
aspnetcore.all meta package restore slows down s2i build and consumes bandwidth
Status: NEW
Product: dotNET
Classification: Red Hat
Component: rh-dotnet20-docker (Show other bugs)
2.0
Unspecified Unspecified
unspecified Severity unspecified
: ga
: 2.0
Assigned To: Andrew Slice
jiri vanek
Les Williams
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2017-06-28 03:50 EDT by Tom Deseyn
Modified: 2017-07-06 09:57 EDT (History)
8 users (show)

See Also:
Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed:
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
sgehwolf: needinfo? (andrew.slice)
tdeseyn: needinfo? (omajid)


Attachments (Terms of Use)

  None (edit)
Description Tom Deseyn 2017-06-28 03:50:11 EDT
ASP.NET Core 2.0 introduces a meta package to reference the major packages that make up the web framework.

The Microsoft dotnet docker sdk image caches these packages from nuget.org in the image.
This has the advantage of both speeding up the build and not wasting the bandwidth of downloading for each build.

With my internet connection/machine this takes about 1-2min and downloads 130MB of packages.

```
$ rm -rf ~/.nuget ~/.local/share/NuGet/; time dotnet new mvc
real	1m47.271s
$ du -ch $(find ~/.nuget/packages -name '*.nupkg') | tail -1
129M	total
```

Since we are not caching these packages, this restore currently happens for every s2i build.
Comment 1 Severin Gehwolf 2017-06-28 06:52:10 EDT
Andrew, how about we do this nuget cache warmup in the 2.0 s2i builder image? This should solve the issue, right?
Comment 2 Tom Deseyn 2017-06-28 07:08:39 EDT
Severin, I had implemented that but this is being removed as part of https://github.com/redhat-developer/s2i-dotnetcore/pull/88/files#diff-6f2669af799dae0df04fa82a92c1d7f9L60. The motivation is we are not building these packages.

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