Docker イメージをリリース時にテストするように追加 & ubuntu18 をベースとしたイメージのサポートを終了 #650

K-shir0 · 2023-04-02T09:55:44Z

内容

Docker ビルドしたあとに動くかテストを追加
- 確認に利用した DockerHub Repo
  - https://hub.docker.com/r/likemikan/voicevox_engine
- 検証で利用した PR
  - Feature/add release test docker K-shir0/voicevox_engine#3
check_release_build.py にて --skip_run_process を指定している場合に process の起動をスキップできるようにしまた、この変更は Docker 内でプロセスを立ち上げているのにプロセスを立ち上げないようにするためです。

スクリーンショット・動画など

テストの様子

その他

Docker の部分は一応実行確認済み（模擬的）
- build -> check_release_build.py のパターンだけテストできていない
テストの実行時間は大体2分ほど。
build-docker のビルド部分のコードを変更したのは、run-release-test-workflow の if 節で env.〇〇 が使用できなかったため、config の部分で宣言するようにし利用できるようにした。
arm 版はテストできていません。（arm が動かせる run-on がないため）

github-actions · 2023-04-02T10:00:17Z

Coverage Result

Resultを開く

Name	Stmts	Miss
voicevox_engine/init.py	1	0
voicevox_engine/acoustic_feature_extractor.py	75	0
voicevox_engine/dev/synthesis_engine/init.py	2	0
voicevox_engine/dev/synthesis_engine/mock.py	36	2
voicevox_engine/full_context_label.py	162	3
voicevox_engine/kana_parser.py	86	1
voicevox_engine/metas/Metas.py	33	0
voicevox_engine/metas/MetasStore.py	29	14
voicevox_engine/metas/init.py	2	0
voicevox_engine/model.py	150	9
voicevox_engine/mora_list.py	4	0
voicevox_engine/part_of_speech_data.py	5	0
voicevox_engine/preset/Preset.py	12	0
voicevox_engine/preset/PresetError.py	2	0
voicevox_engine/preset/PresetManager.py	81	2
voicevox_engine/preset/init.py	4	0
voicevox_engine/setting/Setting.py	11	0
voicevox_engine/setting/SettingLoader.py	19	0
voicevox_engine/setting/init.py	3	0
voicevox_engine/synthesis_engine/init.py	5	0
voicevox_engine/synthesis_engine/core_wrapper.py	201	159
voicevox_engine/synthesis_engine/make_synthesis_engines.py	57	49
voicevox_engine/synthesis_engine/synthesis_engine.py	130	11
voicevox_engine/synthesis_engine/synthesis_engine_base.py	67	9
voicevox_engine/user_dict.py	144	11
voicevox_engine/utility/init.py	4	0
voicevox_engine/utility/connect_base64_waves.py	37	0
voicevox_engine/utility/mutex_utility.py	10	0
voicevox_engine/utility/path_utility.py	26	8
TOTAL	1398	278

.github/workflows/release-test-docker.yml

y-chan

LGTM！

1点だけコメントしましたが、将来的に解決するであろう問題に対するコメントなので、ちょっとどうするか検討したいかもです...!

cc: @Hiroshiba

Hiroshiba · 2023-04-03T10:31:49Z

まだPR見れてないのですが、127.0.0.1に統一していくのが良いかなと思いました🙏

…se-test-docker # Conflicts: # .github/workflows/build-docker.yml

.github/workflows/build-docker.yml

Hiroshiba

いくつかコメントしてみました！

実際にrelease-test-dockerを動かすにはreleasesを作る必要があるかもです。
フォーク先のActionsでブランチ指定してbuild-dockerを回せば実行される･･･はず？

.github/workflows/release-test-docker.yml

K-shir0 · 2023-04-06T13:05:14Z

コード修正し、 release-test-docker を網羅的にテストしてみました。

この PR とは若干別案件なのですがどうやら、 ubuntu18.04 がベースとなっている cpu の image が動かないっぽいです。（他の ubuntu 18.04 ベースのイメージも要確認）自身の環境でも試しましたが、起動後即落ちてるようでログが流れずデバッグが時間かかりそうな感じであまりこちらは見られてないです。

ただ、反面テスト書かないと ubuntu 18.04 ベースのイメージを使う人が少なそうで発見できなかったなと感じています。

Hiroshiba · 2023-04-08T00:01:05Z

ubuntu18.04 がベースとなっている cpu の image が動かないっぽい

おっと！！　テストの効果が出ててすごくいいですね･･･！！！
個人的にはもうubuntu18のサポートは終了で良いかなと思いました！（@y-chan ping）

ついでにubuntu22追加しちゃっても良いかもですね！
意外とパッとubuntu22では動かなかったりするので別PRのが良いかもですが。

Hiroshiba

ubuntu18のとこ以外はLGTMです！！

K-shir0 · 2023-04-10T15:22:55Z

ubuntu22 の検証&対応は別の PR で対応したいと考えてます -> #660

Hiroshiba

LGTM！！

修正ありがとうございます！！
ubuntu22対応も嬉しいです！！

Hiroshiba · 2023-04-10T16:20:28Z

こちらで実際にテストreleaseを作ってみました！
https://github.com/VOICEVOX/voicevox_engine/releases/tag/0.15.0-preview.2

actionsはたぶんこちらです。
https://github.com/VOICEVOX/voicevox_engine/actions/runs/4659151360

K-shir0 added 6 commits April 1, 2023 03:04

add: release-test-docker.yml

2cd4614

add: docker のテストを追加

3949831

add: build-docker に追加

294b7eb

add: build-docker の後にテストが走るように変更

158d8f0

fix: check_release_build.py にて一部コードが意図せずに改変されていたので戻した

81f2d41

fix: skip_process -> skip_run_process

22510b5

K-shir0 requested a review from a team as a code owner April 2, 2023 09:55

K-shir0 requested review from y-chan and removed request for a team April 2, 2023 09:55

K-shir0 changed the title ~~Docker をリリース時にテストするように追加~~ Docker イメージをリリース時にテストするように追加 Apr 3, 2023

y-chan reviewed Apr 3, 2023

View reviewed changes

.github/workflows/release-test-docker.yml Outdated Show resolved Hide resolved

y-chan approved these changes Apr 3, 2023

View reviewed changes

K-shir0 added 2 commits April 4, 2023 00:08

fix: localhost -> 127.0.0.1

df67d88

Merge remote-tracking branch 'upstream/master' into feature/add-relea…

212a731

…se-test-docker # Conflicts: # .github/workflows/build-docker.yml

Hiroshiba reviewed Apr 3, 2023

View reviewed changes

.github/workflows/build-docker.yml Outdated Show resolved Hide resolved

Hiroshiba requested changes Apr 3, 2023

View reviewed changes

.github/workflows/release-test-docker.yml Outdated Show resolved Hide resolved

.github/workflows/release-test-docker.yml Outdated Show resolved Hide resolved

.github/workflows/release-test-docker.yml Outdated Show resolved Hide resolved

fix: 不要なコードを削除

2960c18

K-shir0 force-pushed the feature/add-release-test-docker branch from 5a96bec to 2960c18 Compare April 6, 2023 12:55

K-shir0 requested a review from Hiroshiba April 6, 2023 12:57

Hiroshiba reviewed Apr 8, 2023

View reviewed changes

delete: ubuntu18.04 のサポートは終了

2e947e4

K-shir0 mentioned this pull request Apr 10, 2023

Docker イメージ ubuntu22 サポート追加 #660

Closed

3 tasks

K-shir0 changed the title ~~Docker イメージをリリース時にテストするように追加~~ Docker イメージをリリース時にテストするように追加 & ubuntu18 をベースとしたイメージのサポートを終了 Apr 10, 2023

K-shir0 requested a review from Hiroshiba April 10, 2023 15:24

Hiroshiba approved these changes Apr 10, 2023

View reviewed changes

Hiroshiba merged commit fa8ecee into VOICEVOX:master Apr 10, 2023

This was referenced Apr 15, 2023

SecretsのDOCKERHUB_USERNAMEをVariablesに移行する #663

Closed

Dockerリリース時テストのバージョンパラメータに空文字列が渡されて失敗するのを修正 #666

Merged

This was referenced Jun 17, 2023

CUDA 11.8.0, cuDNN 8.9.2に更新 #695

Merged

Github Actionsでshell: bashを毎回書かなくても良いようにする #697

Merged

aoirint mentioned this pull request May 3, 2024

修正：macos-11を使わないようにする #1205

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Docker イメージをリリース時にテストするように追加 & ubuntu18 をベースとしたイメージのサポートを終了 #650

Docker イメージをリリース時にテストするように追加 & ubuntu18 をベースとしたイメージのサポートを終了 #650

K-shir0 commented Apr 2, 2023 •

edited

Loading

github-actions bot commented Apr 2, 2023 •

edited

Loading

y-chan left a comment

Hiroshiba commented Apr 3, 2023

Hiroshiba left a comment •

edited

Loading

K-shir0 commented Apr 6, 2023

Hiroshiba commented Apr 8, 2023

Hiroshiba left a comment

K-shir0 commented Apr 10, 2023

Hiroshiba left a comment

Hiroshiba commented Apr 10, 2023

Docker イメージをリリース時にテストするように追加 & ubuntu18 をベースとしたイメージのサポートを終了 #650

Docker イメージをリリース時にテストするように追加 & ubuntu18 をベースとしたイメージのサポートを終了 #650

Conversation

K-shir0 commented Apr 2, 2023 • edited Loading

内容

関連 Issue

スクリーンショット・動画など

その他

github-actions bot commented Apr 2, 2023 • edited Loading

Coverage Result

y-chan left a comment

Choose a reason for hiding this comment

Hiroshiba commented Apr 3, 2023

Hiroshiba left a comment • edited Loading

Choose a reason for hiding this comment

K-shir0 commented Apr 6, 2023

Hiroshiba commented Apr 8, 2023

Hiroshiba left a comment

Choose a reason for hiding this comment

K-shir0 commented Apr 10, 2023

Hiroshiba left a comment

Choose a reason for hiding this comment

Hiroshiba commented Apr 10, 2023

K-shir0 commented Apr 2, 2023 •

edited

Loading

github-actions bot commented Apr 2, 2023 •

edited

Loading

Hiroshiba left a comment •

edited

Loading