Dockerfile (Linux)・GitHub Actionsによる自動pushの追加 #84 #88
Conversation
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
aoirint
changed the title
aoirint
changed the title
aoirint
changed the title
Hiroshiba
reviewed
Sep 12, 2021
There was a problem hiding this comment.
非常に素晴らしいです。とても勉強になります。
run.pyをnuitkaでビルドするか、しないか
いったんnuitka無しで良いと思います。
（nuitkaはキャッシュが効かないので毎回ビルドに数十分ほどかかり、actionsが耐えられないと思われます。）
voicevox_coreリポジトリのexample/python/setup.pyを利用しています
いったん良いと思います。のちのちengine用のsetup.pyを設けたいですね。
暫定的に自分のアカウント以下のものになっているため、用意・変更をお願いしたいです
承知しました。
まだvoicevoxをorg化できていないので、やり方把握も兼ねていったん自分のトークンを使う予定です。
Hiroshiba
approved these changes
Sep 12, 2021
|
こちらから書き換えられないこともあるんですね・・・ |
こちらには 私のリポジトリの方に設定していた、Docker Hubに書き込めるSecretsが読めてしまうように見えたので、あまりよくないと思ってOFFにしていました（これは勘違いかもしれません）。お手数おかけします。 |
|
へーーー editとaccess to secretが分かれてないんですね。 |
aoirint
added a commit
to aoirint/voicevox_engine
that referenced
this pull request
Sep 13, 2021
* linux docker * add workflow to build linux docker * update dockerhub description automatically * fix ld.so.conf.d directory name * use python3 instead python * call ldconfig * remove hub desc update because token cannot be used * install core package * install libtorch * add doc: github secrets * force update ldconfig cache * docker cpu execute command in Makefile * lock python version 3.7.12 (self compile) * remove dev requirements install * general user execution with gosu * add .PHONY to Makefile * versioned image deploy * fix workflow: image tag * fix workflow: image tag * fix cache step * on.push.branches * fix cache * fix docker tag version order * revert image tag version order * simple tag formatting * fix head hyphen tag name * fix head hyphen tag name * fix head hyphen tag name * cache per base image * one step multi tags (cache overflow) * registry cache * run build-docker on push master * fix doc: -latest * fix doc: whitespace * update ldconfig on container start * cat voicevox core README on container start * cat README to stderr
This was referenced Sep 15, 2021
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ref: #84
実装
確認
aoirint/voicevox_engineの変更（変更案：voicevox/voicevox_engineまたはhiroshiba/voicevox_engine）Dockerイメージのデプロイ先の候補
現在、Docker HubのみWorkflowのStepを用意しています。
Docker Hubは、Freeアカウント（Docker Personal）ではアクセストークンを1アカウントにつき1つしか発行できないため、
既存の個人アカウントを使うとスコープの分離ができない懸念があります。
Docker Hubのアクセストークンにはイメージの説明（Overviewに表示される文章）を更新する権限がないため、
README.mdと同期させたい場合は、手動で更新が必要です。
メンテナ側で必要な設定
GitHub Secretsの設定
https://hub.docker.com/settings/security
上記URLで生成したアクセストークンを
DOCKERHUB_TOKENとして、GitHub Secretsに登録する必要があります。また、Docker Hubのアカウント名を
DOCKERHUB_USERNAMEとして、GitHub Secretsに登録する必要があります。