sakamomo554101 / YouyakuAI

日本語用の要約を行うデモサービス
0 stars 0 forks source link

クラウドで稼働させる全Dockerファイルで、production用のイメージ作成のコマンドを用意する #53

Closed sakamomo554101 closed 2 years ago

sakamomo554101 commented 2 years ago

概要

クラウドで稼働させる全Dockerファイルで、production用のイメージ作成のコマンドを用意する。

対応としては、servingコンテナでやっているような対応を行う(ローカルとproductionでステージを分ける感じ)

sakamomo554101 commented 2 years ago

まず、terraform検討の前に、これを対応する。

sakamomo554101 commented 2 years ago

Dockerfileであけるポート情報をENVファイルから読み込みたい。 どうやるんだっけな。

sakamomo554101 commented 2 years ago

https://zenn.dev/dogscos/articles/3277fced50f6f4 https://docs.docker.jp/engine/reference/commandline/build.html

docker buildだと、build-argで変数を渡せるから、シェルスクリプトとかを作って、イメージをビルドするのが良いかなぁ。

sakamomo554101 commented 2 years ago

credentialsはDocker Imageには含めていけないので、外す。 ※クラウド側で各コンポーネントの接続を適切に定義すれば良い

sakamomo554101 commented 2 years ago

うーむ、思ったよりDockerfileの書き換え面倒だな。

sakamomo554101 commented 2 years ago

build_and_deploy_image.pyを有効活用したい

sakamomo554101 commented 2 years ago

うーむ、summazrizerとsummazrizer_processorでそれぞれちゃんとテスト作るべきだな・・。 今結合されてる感じあるし。

sakamomo554101 commented 2 years ago

gcp関連のテストコードは、credentialsの環境変数が設定されていない場合はスキップするようにするか。

sakamomo554101 commented 2 years ago

ローカルの動作検証が大丈夫そうなら、一旦マージかな。

sakamomo554101 commented 2 years ago

あー、見事にmake run-buildでこけるな。

これはlocal, vertexaiのビルドパターンで、ちゃんとイメージを分けないとダメかも。

sakamomo554101 commented 2 years ago

以下の二つの方針がある。

sakamomo554101 commented 2 years ago

あ、違うわ。docker-compose.ymlでtarget: localを指定してないコンポーネントがあった。

sakamomo554101 commented 2 years ago

https://docs.github.com/ja/actions/creating-actions/creating-a-docker-container-action

github actionsについて

sakamomo554101 commented 2 years ago

https://docs.github.com/ja/actions/creating-actions/metadata-syntax-for-github-actions#runs-for-docker-container-actions dockerでgithub actionsをやる場合

sakamomo554101 commented 2 years ago

github actionsを利用したCI周りは別で検討やな。