Skip to content

[8.19] [Data Streams] Fix forward-compatibility api test failures (#224795)#225641

Merged
SoniaSanzV merged 4 commits intoelastic:8.19from
SoniaSanzV:backport/8.19/pr-224795
Jul 2, 2025
Merged

[8.19] [Data Streams] Fix forward-compatibility api test failures (#224795)#225641
SoniaSanzV merged 4 commits intoelastic:8.19from
SoniaSanzV:backport/8.19/pr-224795

Conversation

@SoniaSanzV
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

…24795)

Fixes elastic#223990

## Summary

This PR fixes the forward-compatibility test for index mode, which
failed because the index mode was added to Get Data Streams API in 8.19
and 9.1 (see elastic/elasticsearch#122486), so
if Kibana 8.19 is run with Es 9.0, the index mode is always displayed as
"Standard" because Es doesn't return an index mode field.

In this PR, we separate all index mode-related tests into a separate
file, and only run it for Es versions 8.19 or 9.1+.

This was also fixed for functional tests in
elastic#223129

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 493ca9f)

# Conflicts:
#	x-pack/platform/test/api_integration/apis/management/index_management/data_streams.ts
@SoniaSanzV SoniaSanzV added the backport This PR is a backport of another PR label Jun 27, 2025
@SoniaSanzV SoniaSanzV enabled auto-merge (squash) June 27, 2025 13:59
@SoniaSanzV SoniaSanzV added the backport This PR is a backport of another PR label Jun 27, 2025
@SoniaSanzV SoniaSanzV requested a review from a team June 27, 2025 14:00
@SoniaSanzV SoniaSanzV self-assigned this Jun 27, 2025
Copy link
Copy Markdown
Contributor

@ElenaStoeva ElenaStoeva left a comment

Choose a reason for hiding this comment

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

Lgtm!

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Jul 2, 2025

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #13 / getNextRunAt should use the rrule with a fixed time when it is given to calculate the next runAt (next day)
  • [job] [logs] Jest Tests #13 / getNextRunAt should use the rrule with a fixed time when it is given to calculate the next runAt (next day)
  • [job] [logs] Jest Tests #13 / getNextRunAt should use the rrule with a fixed time when it is given to calculate the next runAt (next day)
  • [job] [logs] Jest Tests #13 / getNextRunAt should use the rrule with a fixed time when it is given to calculate the next runAt (same day)
  • [job] [logs] Jest Tests #13 / getNextRunAt should use the rrule with a fixed time when it is given to calculate the next runAt (same day)
  • [job] [logs] Jest Tests #13 / getNextRunAt should use the rrule with a fixed time when it is given to calculate the next runAt (same day)

Metrics [docs]

✅ unchanged

History

cc @SoniaSanzV

@SoniaSanzV SoniaSanzV merged commit 3715ca7 into elastic:8.19 Jul 2, 2025
8 checks passed
@SoniaSanzV SoniaSanzV deleted the backport/8.19/pr-224795 branch July 14, 2025 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants