Skip to content

Conversation

@kkrik-es
Copy link
Contributor

Yet another test 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 #100913

@kkrik-es kkrik-es added >test-mute Use for PR that only mute tests Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :StorageEngine/TSDB You know, for Metrics labels Oct 16, 2023
@kkrik-es kkrik-es self-assigned this Oct 16, 2023
@kkrik-es kkrik-es requested a review from martijnvg October 16, 2023 14:44
@elasticsearchmachine
Copy link
Collaborator

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

@kkrik-es kkrik-es added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) v8.11.1 v8.10.5 and removed v8.12.0 labels Oct 16, 2023
@kkrik-es kkrik-es changed the title Skip cat tsdb test for versions 8.7-8-10 [TEST] Skip cat tsdb test for versions 8.7-8-10 Oct 16, 2023
@elasticsearchmachine elasticsearchmachine merged commit 30ac8fe into elastic:main Oct 16, 2023
@kkrik-es kkrik-es deleted the fix/100913 branch October 16, 2023 15:00
@kkrik-es kkrik-es restored the fix/100913 branch October 16, 2023 15:08
@kkrik-es kkrik-es deleted the fix/100913 branch October 16, 2023 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :StorageEngine/TSDB You know, for Metrics Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test-mute Use for PR that only mute 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=cat.segments/10_basic/tsdb} failing

2 participants