forked from hyperledger-cacti/cacti
-
Notifications
You must be signed in to change notification settings - Fork 2
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 hyperledger-cacti#649
Fixes hyperledger-cacti#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. Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
- Loading branch information
Showing
3 changed files
with
63 additions
and
3 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