-
Notifications
You must be signed in to change notification settings - Fork 286
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
perf(tools): fabric 1.x AIO image pre-fetching #649
Fixes #649 In addition to that fix it also fixes a pre-fetching bug in the 2.x image which was stopping it from actually using the pre-fetched images on the file-system => the directory paths had to be specified one by one for each image that we pre-fetched, it did not work if you specified a common parent directory where the images were stored, so this commit provides a fix for both 1.4.x and 2.x in the sense that they both should no longer hit DockerHub for image downloads at container startup time, only build-time as long as the FABRIC_VERSION env vars were not altered by the person launching the containers (if they were then those different versions will still have to be fetched at runtime). The biggest net positive effect(s) of this commit: 1. Image boot will be much faster, especially on machines where the network is slow. 2. The risk of getting slapped with DockerHub image pull rate limiting goes down significantly. Also fixes a health check bug where the FABRIC_VERSION build arg was not being exposed as an ENV var in the container and the parsing of the version was failing due to this. Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
- Loading branch information
Showing
3 changed files
with
68 additions
and
10 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters