Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

新しいGithub Actions環境でmacOSのテストが一部落ちるのを解決したい #2007

Open
Hiroshiba opened this issue Apr 25, 2024 · 5 comments
Labels

Comments

@Hiroshiba
Copy link
Member

不具合の内容

2024/04/22辺りにmacOSのlatestが更新され、macos-14が利用されるようになりました。

以前
image

以降
image

その影響でテストが2つ落ちるようになったのでメモです。
応急処置としてskipすることにしていますが、可能なら解決したいです。

現象・ログ

このissueを作った時点でのmainブランチの結果
https://github.com/VOICEVOX/voicevox/actions/runs/8732000344/attempts/2

同じコミットIDでの以前の結果
https://github.com/VOICEVOX/voicevox/actions/runs/8732000344/attempts/1

その他

原因調査に取り組まれた場合はぜひコメントをいただけると助かります。

skipは @weweweok さんが実装してくださいました、ありがとうございます!

@weweweok
Copy link
Contributor

weweweok commented Apr 25, 2024

ローカル環境で何回かe2eテストを実行してみましたが、かなりの頻度で時間切れになりますね。。。(毎回同じ箇所で落ちる)
ファイル数が少ないと通ります。

image

@Hiroshiba
Copy link
Member Author

@weweweok なるほどです!!
fullyParallelをfalseにし、workersを1にするとどうでしょう?
このあたりです↓

// ファイルシステムが関連してくるので、Electronテストでは並列化しない
fullyParallel: !isElectron,
workers: isElectron ? 1 : undefined,

@weweweok
Copy link
Contributor

weweweok commented Apr 25, 2024

全体として遅くなりますが、ほとんど通りました。
image

@sevenc-nanashi
Copy link
Member

ファイル数が少ないと通ります。

もしかして:エンジンにキャラクターが多いせいで時間内にデータ準備が終わっていない?
VRTでやってるようなモックを全体に適用してもいいかも。全部差し替えなくてもGET /speakersを5人に絞るとかでも。

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Apr 25, 2024

ほとんどが時間切れになるのは同時実行していてPCスペックが間に合わないからだと思います。
Nemoじゃなく通常VOICEVOXエンジン使ってるならなおさら。
なので並列実行数を下げたらほぼ通ったんだと思います。

並列実行数を下げても通らなかったテストは問題だと思います。なんでだろう・・・。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants