Skip to content

Conversation

@kkrik-es
Copy link
Contributor

@kkrik-es kkrik-es commented Oct 16, 2023

Verify that there are no failures to create the expected indexes in field_caps/time_series tests.

This is also affected by the fix for showing the synthetic source, #98808. This can trigger an assert in older versions as the mapping they produce (without synthetic source) doesn't match the one they may get from the master, if the latter is in version 8.10+.

Fixes #100882

@kkrik-es kkrik-es self-assigned this Oct 16, 2023
@kkrik-es kkrik-es added :StorageEngine/TSDB You know, for Metrics >test Issues or PRs that are addressing/adding tests labels Oct 16, 2023
@kkrik-es kkrik-es changed the title Assert that both time-series indexes are created [TEST] Assert that both time-series indexes are created Oct 16, 2023
@kkrik-es kkrik-es added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Oct 16, 2023
@kkrik-es kkrik-es added the test-full-bwc Trigger full BWC version matrix tests label Oct 16, 2023
# Conflicts:
#	rest-api-spec/src/yamlRestTest/resources/rest-api-spec/test/field_caps/40_time_series.yml
@kkrik-es kkrik-es removed the test-full-bwc Trigger full BWC version matrix tests label Oct 16, 2023
@kkrik-es kkrik-es requested a review from martijnvg October 16, 2023 08:35
@kkrik-es kkrik-es marked this pull request as ready for review October 16, 2023 08:35
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

@kkrik-es kkrik-es added v8.11.1 auto-backport Automatically create backport pull requests when merged v8.10.5 labels Oct 16, 2023
Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

lgtm

@kkrik-es kkrik-es merged commit 3247acc into elastic:main Oct 16, 2023
kkrik-es added a commit to kkrik-es/elasticsearch that referenced this pull request Oct 16, 2023
* Assert that both time-series indexes are created

* Exclude from 8.7-8.10 mixedClusterTests

* Restore asserts

* Fix assert
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.11
8.10

kkrik-es added a commit to kkrik-es/elasticsearch that referenced this pull request Oct 16, 2023
* Assert that both time-series indexes are created

* Exclude from 8.7-8.10 mixedClusterTests

* Restore asserts

* Fix assert
elasticsearchmachine pushed a commit that referenced this pull request Oct 16, 2023
…00897)

* Assert that both time-series indexes are created

* Exclude from 8.7-8.10 mixedClusterTests

* Restore asserts

* Fix assert
elasticsearchmachine pushed a commit that referenced this pull request Oct 16, 2023
…00896)

* Assert that both time-series indexes are created

* Exclude from 8.7-8.10 mixedClusterTests

* Restore asserts

* Fix assert
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged :StorageEngine/TSDB You know, for Metrics Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.10.5 v8.11.1 v8.12.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] MixedClusterClientYamlTestSuiteIT test {p0=field_caps/40_time_series/Get time series field caps with conflicts} failing

3 participants