Skip to content

Ensure regression tasks don't start new tasks when not making progress#4705

Merged
jonathanmetzman merged 1 commit intogoogle:masterfrom
mi-ac:progress1
Feb 28, 2025
Merged

Ensure regression tasks don't start new tasks when not making progress#4705
jonathanmetzman merged 1 commit intogoogle:masterfrom
mi-ac:progress1

Conversation

@mi-ac
Copy link
Collaborator

@mi-ac mi-ac commented Feb 25, 2025

This returns a bad-build error type after a regression task, when a known min/max input pair didn't shrink. Previously, a timeout error type was returned, after which post-process respawns the task with the same min/max pair, leading to a task loop.

This doesn't change behavior when either min or max are not known (e.g. when tasks are initially started) or when some progress is made (e.g. at least one improvement to min or max during several hours this task can run) or when other error conditions happen.

Chrome bug: https://crbug.com/396344382

@jonathanmetzman
Copy link
Collaborator

/gcbrun

Copy link
Collaborator

@jonathanmetzman jonathanmetzman left a comment

Choose a reason for hiding this comment

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

lgtm

@jonathanmetzman jonathanmetzman merged commit 656f49d into google:master Feb 28, 2025
7 checks passed
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.

2 participants