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

Puppeteer E2E test: Set timeout to 15 minutes #25501

Merged
merged 1 commit into from
Feb 14, 2023
Merged

Conversation

LeviPesin
Copy link
Contributor

@LeviPesin LeviPesin commented Feb 14, 2023

Related issue: #25500 (comment)

Description

The default is 6 hours.

@gkjohnson
Copy link
Collaborator

I've cancelled the runs in #25500. We'll see if the others get stuck, too.

@LeviPesin
Copy link
Contributor Author

Seems like https://github.com/mrdoob/three.js/actions/runs/4171044015 also got stuck.

@LeviPesin
Copy link
Contributor Author

LeviPesin commented Feb 14, 2023

Maybe we should just cancel all stale runs in https://github.com/mrdoob/three.js/actions/workflows/ci.yml and https://github.com/mrdoob/three.js/actions/workflows/codeql-code-scanning.yml and then re-trigger them one by one... @mrdoob @gkjohnson

@mrdoob mrdoob added this to the r150 milestone Feb 14, 2023
@mrdoob mrdoob merged commit 6ba91f1 into mrdoob:dev Feb 14, 2023
@LeviPesin LeviPesin deleted the patch-4 branch February 14, 2023 08:57
@LeviPesin
Copy link
Contributor Author

LeviPesin commented Feb 14, 2023

I'm afraid something broke in the browser download loop (e.g. OmahaProxy and/or Chromium snaphosts storage is inaccessible, or it returns impossible data)... I've started a test run in my fork to debug it: https://github.com/LeviPesin/three.js-1/actions/runs/4172070897
Yes, it seems like the number there is going into negatives...

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

Successfully merging this pull request may close these issues.

3 participants