Fix issue/1312 - Use caching in docker build #1313
Closed
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
Per comment in issue #1312
the current state of the dockerfile in the aspnet/home repo does not use docker caching on DNU RESTORE (soon to be dotnet restore)
A small fix to the sample dockerfile can make the compile time of "second deploy" in docker 23x faster
old-speed for every deploy & first deploy under new method: ~3m30s
Start: 06:03:13.313
Finish: 06:06:40.640
new-speed - second & subsequent deploys if dependencies are unchanged: ~9s
Start: 06:00:36:036
Finish: 06:00:45:045
I also posted the FULL LOG of the difference over here: aspnet/aspnet-docker#123