追加: 起動前にengine_manifest.jsonをチェックする

[nanae772]

内容

check_release_build.py でエンジン起動前にengine_manifest.jsonの

1. 存在チェック
2. jsonとして読み込めるかチェック
3. 適当なキー(manifest_versionとした)の存在チェック

を行うようにしました。

fork先でbuildのCIが通ることを確認しました。
https://github.com/nanae772/voicevox_engine/actions/runs/13263787224

関連 Issue

close #1300

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

その他

[Hiroshiba]

プルリクエストありがとうございます！

skip_run_processはその名の通り「runの実行をスキップする」という引数なので、その中でマニフェストのチェックをするのは役割が混じってるかもです！

ということで別で引数を用意してあげるのはどうでしょうか。
例えばskip_check_manifestを用意し、falseのときだけチェックするとか！

マニフェストの存在を確認するチェックコードは現状で問題ないと思います！

[nanae772]

レビューいただきありがとうございます、skip_check_manifestフラグを持たせる方針で修正いたしました。

[Hiroshiba]

すみません１点だけ！

たぶんdockerで動かすtest-engine-containerテストの方ではマニフェストファイルがないと思うので、--skip_check_manifestを指定しないとエラーになるかもです！
たぶんここ？

voicevox_engine/.github/workflows/test-engine-container.yml

Line 89 in b13c61b

run: python tools/check_release_build.py --skip_run_process --dist_dir dist/

間違ったことを言ってそうだったらご指摘いただけると 🙇

[nanae772]

ご指摘ありがとうございます、全く把握しておりませんでした。
fork先で試してみようと思ったのですが、自前のDockerHubアカウントを使って動かす必要があるみたいですね。
そのあたりも分かってなかったので、もう一度確認しておきます。

[Hiroshiba]

検証の提案ありがとうございます！！
Dockerアカウントの作成が難しそうとか、チェックが難しそうであれば言ってください！
マージした後正しく動かなければ修正する、とかもできると思うので･･･！

[nanae772]

ご親切にありがとうございます。
DockerHubの利用まではいけたのですが、build-dockerのCICDでDocker ImageのBuildに失敗してしまうようでちょっと検証するのが難しい感じです。
もしかしたら、#1525 と関連しているかもしれないので、ログなどはそちらにアップロードさせていただきます。

ただ仰るようにdockerコンテナを利用するためmanifestファイルが無いというのは、その通りだと思ったので
そちら、5b70f21 にて修正いたしました。

[Hiroshiba]

LGTM！！

多分問題ないと思うのでマージさせていただきます！