Skip to content

Update timeout on yaml rest tests because they often exceed 1h#142715

Merged
elasticsearchmachine merged 2 commits intoelastic:mainfrom
craigtaverner:increase_yaml_rest_timeout
Feb 19, 2026
Merged

Update timeout on yaml rest tests because they often exceed 1h#142715
elasticsearchmachine merged 2 commits intoelastic:mainfrom
craigtaverner:increase_yaml_rest_timeout

Conversation

@craigtaverner
Copy link
Contributor

What is happening now is that whichever test happens to be running when the timeout threshold is crossed will get muted by the CI automation. Once sufficient tests are muted, the timeout is no longer crossed. This means arbitrary tests are muted, not the ones that actually take the longest.

Extending the timeout is an interim measure and should be followed up by one or both of these options:

  • Investigate the slowest tests and decide if they are needed or could be simplified, and try to reduce the total time to within reason
  • Split the tests into groups that are run on separate test runs, probably in parallel. This costs more in infrastructure, but with less risk of reduce coverage

Fixes #141113
Fixes #141856

What is happening now is that whichever test happens to be running when the timeout threshold is crossed will get muted by the CI automation. Once sufficient tests are muted, the timeout is no longer crossed. This means arbitrary tests are muted, not the ones that actually take the longest.

Extending the timeout is an interim measure and should be followed up by one or both of these options:
* Investigate the slowest tests and decide if they are needed or could be simplified, and try to reduce the total time to within reason
* Split the tests into groups that are run on separate test runs, probably in parallel. This costs more in infrastructure, but with less risk of reduce coverage
@craigtaverner craigtaverner added >test Issues or PRs that are addressing/adding tests Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL labels Feb 19, 2026
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@craigtaverner craigtaverner added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Feb 19, 2026
@elasticsearchmachine elasticsearchmachine merged commit 97e4396 into elastic:main Feb 19, 2026
35 checks passed
@craigtaverner craigtaverner deleted the increase_yaml_rest_timeout branch February 19, 2026 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.4.0

Projects

None yet

3 participants

Comments