Skip to content

Conversation

@jansel
Copy link
Contributor

@jansel jansel commented Sep 26, 2025

I am seeing jobs unstable because they hit the timeout

stack-info: PR: #697, branch: jansel/stack/153
Copy link
Contributor

@oulgen oulgen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like we should rewrite/disable tests that take a minute?

Running all the tests locally already takes over 5 minutes now

@jansel
Copy link
Contributor Author

jansel commented Sep 26, 2025

The autotuner tests sometimes fails with no configs found if you decrease the count.

@jansel jansel changed the base branch from main to jansel/stack/150 September 26, 2025 18:48
jansel added a commit that referenced this pull request Sep 26, 2025
I am seeing jobs unstable because they hit the timeout

stack-info: PR: #697, branch: jansel/stack/153
@jansel jansel force-pushed the jansel/stack/153 branch 2 times, most recently from d8a8fb7 to 158780e Compare September 26, 2025 20:02
@jansel jansel changed the base branch from jansel/stack/150 to main September 26, 2025 20:02
@jansel
Copy link
Contributor Author

jansel commented Sep 27, 2025

I support speeding up benchmarks, but flaky tests is not a good state to be in so landing this in short term.

@jansel jansel merged commit e92d95c into main Sep 27, 2025
13 checks passed
yf225 pushed a commit that referenced this pull request Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants