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

CI: Set timeout to ccpp job ASan-22 #1365

Merged
merged 1 commit into from
Mar 16, 2024
Merged

Conversation

ma8ma
Copy link
Collaborator

@ma8ma ma8ma commented Mar 16, 2024

AddressSanitizer を使うCIジョブ ASan-22 のテストでハングアップすることがありました。
ハングアップするとCIの制限時間が過ぎるまで実行が続くためテストのタイムアウトを設定して一定時間で実行を打ち切るようにします。
テストがタイムアウトしたときはジョブをスキップします。

ハングアップを確認したPull request: #1363
googletestのissue: google/googletest 4491

@ma8ma ma8ma added bug バグの追跡 test テストの変更 labels Mar 16, 2024
AddressSanitizerを使うCIジョブ ASan-22 のテストでハングアップすることが
ありました。 ハングアップするとCIの制限時間が過ぎるまで実行が
続くためテストのタイムアウトを設定して一定時間で実行を打ち切る
ようにします。テストがタイムアウトしたときは残りのジョブは実行しません。

ハングアップを確認したPull request: #1363
googletestのissue: google/googletest 4491
@ma8ma ma8ma force-pushed the set-timeout-to-ci-job-asan-22 branch from aa7af45 to 411b739 Compare March 16, 2024 14:40
@ma8ma ma8ma merged commit a65876f into master Mar 16, 2024
20 checks passed
@ma8ma ma8ma deleted the set-timeout-to-ci-job-asan-22 branch March 16, 2024 14:48
ma8ma added a commit that referenced this pull request Mar 24, 2024
This reverts commit a65876f.

AddressSanitizerを使ったとき`AddressSanitizer:DEADLYSIGNAL`が
出力され続けてハングアップする問題はGitHub Actionsのrunner-imagesに
報告されています。この不具合を回避する修正が入ったためCI job ASan-22の
応急処置を差し戻します。

actions/runner-images#9491 (comment)
ma8ma added a commit that referenced this pull request Mar 24, 2024
This reverts commit a65876f.

AddressSanitizerを使ったとき`AddressSanitizer:DEADLYSIGNAL`が
出力され続けてハングアップする問題はGitHub Actionsのrunner-imagesに
報告されています。この不具合を回避する修正が入ったためCI job ASan-22の
応急処置を差し戻します。

actions/runner-images#9491 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug バグの追跡 test テストの変更
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

1 participant