Skip to content

Conversation

@craigtaverner
Copy link
Contributor

The random license selection in #125584 was incorrectly assuming ENTERPRISE was needed, when it should have been PLATINUM. This resulted in occasional test failures (expected to fail on PLATINUM, and succeeded).

Fixes #125734

@craigtaverner craigtaverner added >test Issues or PRs that are addressing/adding tests :Analytics/Geo Indexing, search aggregations of geo points and shapes Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) auto-backport Automatically create backport pull requests when merged v9.0.1 v9.1.0 labels Mar 27, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@craigtaverner craigtaverner removed auto-backport Automatically create backport pull requests when merged v9.0.1 labels Mar 27, 2025
@elasticsearchmachine elasticsearchmachine merged commit f64efbd into elastic:main Mar 27, 2025
17 checks passed
@craigtaverner craigtaverner deleted the use_platinum_license_spatial_extent_test branch March 27, 2025 19:18
omricohenn pushed a commit to omricohenn/elasticsearch that referenced this pull request Mar 28, 2025
The random license selection in
elastic#125584 was incorrectly
assuming ENTERPRISE was needed, when it should have been PLATINUM. This
resulted in occasional test failures (expected to fail on PLATINUM, and
succeeded).

Fixes elastic#125734
elasticsearchmachine pushed a commit that referenced this pull request Apr 3, 2025
This was fixed last week in
#125802. It seems this
report was from the day before the fix, but the PR was based on main
without the mute, so a timing issue. I'll make a PR that simply unmutes
the test.

Fixes #125735
andreidan pushed a commit to andreidan/elasticsearch that referenced this pull request Apr 9, 2025
This was fixed last week in
elastic#125802. It seems this
report was from the day before the fix, but the PR was based on main
without the mute, so a timing issue. I'll make a PR that simply unmutes
the test.

Fixes elastic#125735
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL :Analytics/Geo Indexing, search aggregations of geo points and shapes 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.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] SpatialExtentAggregationNoLicenseIT testStExtentAggregationWithShapes failing

2 participants