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

オープン
#47 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
35/100
issue の種類
機能追加
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
aws, docker, docker-compose, python

調査の方向性

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.

索引モデルが issue の本文から書いたものです。

説明

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

主要言語
Python
スター
246
フォーク
27
PR マージ指標
30日以内にマージされた PR はありません

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

graphistry/graph-app-kit のほかの issue

graphistry/graph-app-kit の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。