Skip to content

Conversation

@gmarouli
Copy link
Contributor

Backports the following commits to 8.x:

In this PR we add a test and we fix the issues we encountered when we
enabled the failure store for TSDS and logsdb.

**Logsdb** Logsdb worked out of the box, so we just added the test that
indexes with a bulk request a couple of documents and tests how they are
ingested.

**TSDS** Here it was a bit trickier. We encountered the following
issues:

- TSDS requires a timestamp to determine the write index of the data stream meaning the failure happens earlier than we have anticipated so far. We added a special exception to detect this case and we treat it accordingly.
- The template of a TSDS data stream sets certain settings that we do not want to have in the failure store index. We added an allowlist that gets applied before we add the necessary index settings. 

Furthermore, we added a test case to capture this.
@gmarouli gmarouli added :Data Management/Data streams Data streams and their lifecycles >non-issue 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 Oct 22, 2024
@elasticsearchmachine elasticsearchmachine merged commit 21312da into elastic:8.x Oct 22, 2024
15 checks passed
@gmarouli gmarouli deleted the backport/8.x/pr-114307 branch October 22, 2024 08:45
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 >non-issue Team:Data Management Meta label for data/management team v8.17.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants