Skip to content

Fix failing release tests related to views and other recent changes#144470

Merged
jfreden merged 5 commits intoelastic:mainfrom
jfreden:fix_release_tests
Mar 19, 2026
Merged

Fix failing release tests related to views and other recent changes#144470
jfreden merged 5 commits intoelastic:mainfrom
jfreden:fix_release_tests

Conversation

@jfreden
Copy link
Copy Markdown
Contributor

@jfreden jfreden commented Mar 18, 2026

Includes fixes to CsvIT from #144465, as well as release test failures for a few other cases.

@jfreden jfreden added the test-release Trigger CI checks against release build label Mar 18, 2026
@jfreden jfreden force-pushed the fix_release_tests branch from 74b6ced to 9607086 Compare March 18, 2026 10:28
@jfreden jfreden added >test Issues or PRs that are addressing/adding tests :Analytics/ES|QL AKA ESQL labels Mar 19, 2026
@jfreden jfreden marked this pull request as ready for review March 19, 2026 07:54
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Mar 19, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Contributor

@craigtaverner craigtaverner left a comment

Choose a reason for hiding this comment

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

LGTM

@craigtaverner craigtaverner changed the title Fix failing views tests Fix failing release tests related to views and other recent changes Mar 19, 2026
}
}
}
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think I saw this in another PR. Was this change already merged?

Copy link
Copy Markdown
Contributor

@idegtiarenko idegtiarenko left a comment

Choose a reason for hiding this comment

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

Thanks for fixing it!
Overall 👍 , however I have a question about a change I assume was already merged before.

@jfreden jfreden merged commit 8dbd5e4 into elastic:main Mar 19, 2026
35 of 39 checks passed
michalborek pushed a commit to michalborek/elasticsearch that referenced this pull request Mar 23, 2026
…lastic#144470)

* Fix failing views tests

* [CI] Auto commit changes from spotless

* Fix CsvIT to check test capabilities

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
Co-authored-by: Craig Taverner <craig@amanzi.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests test-release Trigger CI checks against release build v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants