graph-app-kit icon indicating copy to clipboard operation
graph-app-kit copied to clipboard

[ENH] Faster bootstrap: Skip apt-get update on aws and correct base image

Open lmeyerov opened this issue 4 years ago • 1 comments

Is your feature request related to a problem? Please describe.

Bootstraps are slow to start on aws, with a lot of initial time on:

########################################
+ echo '##'
##
+ echo '## Start: Installing CloudFormation Bootstrap'
## Start: Installing CloudFormation Bootstrap
+ echo '##'
##
+ echo '########################################'
########################################
+ echo

+ echo

+ sudo apt-get update -y

The update is unnecessary because AMIs are updated every 2-4w, and if users are more patch-sensitive, they should be regularly doing their own schedule. ~All other other dependencies should be within the container scripts so not need system ones.

  1. Then, it's slow on:
+ cd ../../docker
+ docker-compose build
The BASE_PATH variable is not set. Defaulting to a blank string.
autoheal uses an image, skipping
Building streamlit
Step 1/20 : ARG DOCKER_TAG=latest
Step 2/20 : ARG GRAPHISTRY_FORGE_BASE_VERSION=latest
Step 3/20 : FROM graphistry/graphistry-forge-base:${GRAPHISTRY_FORGE_BASE_VERSION}
v2.35.9-11.0: Pulling from graphistry/graphistry-forge-base

The pull should be unnecessary. A temporary workaround is basing on forge-etl-python. Even better is if Graphistry explicitly tagsforge-etl-python (which is public container).

Describe the solution you'd like

This should just work :)

Describe alternatives you've considered

Graphistry AMIs can also just start shipping w/ graph-app-kit so most of this goes away

lmeyerov avatar Mar 07 '21 06:03 lmeyerov

Graphistry 2.36+ now explicitly tags the graphistry-forge-base layer, so that will help future versions

lmeyerov avatar Mar 07 '21 06:03 lmeyerov