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

Aperta
  • Qualcosa non va?
#47 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
35/100
Tipo di issue
Funzionalità
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
aws, docker, docker-compose, python
Ambito
devops, infrastructure

Direzione di ricerca

Start by reading the CloudFormation bootstrap script that runs sudo apt-get update -y and the Docker Compose files under docker, where the streamlit image is built from graphistry/graphistry-forge-base. Identify how the bootstrap and image base are selected, then check whether the proposed changes can be made without breaking dependency installation. Done means the bootstrap avoids the unnecessary update and the image build avoids the reported pull.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

enhancement performance

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

Lingua principale
Python
Stelle
246
Fork
27
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di graphistry/graph-app-kit

Tutte le issue di graphistry/graph-app-kit

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.