Skip to content

[8.19] Remove test usages of getDefaultBackingIndexName in DS and LogsDB tests (#127384) #127502

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

nielsbauman
Copy link
Contributor

Backports the following commits to 8.19:

@nielsbauman nielsbauman added :Data Management/Data streams Data streams and their lifecycles >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Data Management Meta label for data/management team labels Apr 29, 2025
…ests (elastic#127384)

We replace usages of time sensitive
`DataStream#getDefaultBackingIndexName` with the retrieval of the name
via an API call. The problem with using the time sensitive method is
that we can have test failures around midnight.

Relates elastic#123376
@nielsbauman nielsbauman force-pushed the backport/8.19/pr-127384 branch from 84b8c3d to 58a0679 Compare April 30, 2025 08:13
@elasticsearchmachine elasticsearchmachine merged commit 4b13e27 into elastic:8.19 Apr 30, 2025
15 checks passed
@nielsbauman nielsbauman deleted the backport/8.19/pr-127384 branch April 30, 2025 09:21
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!) backport :Data Management/Data streams Data streams and their lifecycles Team:Data Management Meta label for data/management team >test Issues or PRs that are addressing/adding tests v8.19.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants