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

Am/chore/fix wasm bench timeout #1517

Merged
merged 2 commits into from
Sep 6, 2024
Merged

Conversation

IceTDrinker
Copy link
Member

No description provided.

@IceTDrinker IceTDrinker requested a review from soonum September 6, 2024 12:00
@cla-bot cla-bot bot added the cla-signed label Sep 6, 2024
@IceTDrinker
Copy link
Member Author

I have to adapt the timing slightly the kill from GitHub removes the log we are interested in 🙄

- avoids a stuck runner for 6 hours
- actions timeouts are slightly larger than the test runner timeout to
have a chance to get a log out
@IceTDrinker IceTDrinker force-pushed the am/chore/fix-wasm-bench-timeout branch from 830fb63 to ab33646 Compare September 6, 2024 14:11
@zama-bot zama-bot removed the approved label Sep 6, 2024
@IceTDrinker
Copy link
Member Author

new manual run https://github.com/zama-ai/tfhe-rs/actions/runs/10739912424 (we'll merge and treat potential issues showing in logs later)

@IceTDrinker IceTDrinker requested a review from soonum September 6, 2024 14:12
@IceTDrinker
Copy link
Member Author

no outputs even leaving some time for the log to appear, will merge this and try to find something else that may be failing

@IceTDrinker IceTDrinker merged commit 019548d into main Sep 6, 2024
85 of 87 checks passed
@IceTDrinker IceTDrinker deleted the am/chore/fix-wasm-bench-timeout branch September 6, 2024 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants