Skip to content
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

[Dataset quality] 🐞 Rely solely on _index instead of data_stream properties #210329

Merged
merged 2 commits into from
Feb 11, 2025

Conversation

yngrdyn
Copy link
Contributor

@yngrdyn yngrdyn commented Feb 10, 2025

Closes https://github.com/elastic/logs-dev/issues/192.

Background

This have been an long running issue within dataset quality page which became more noticeable when introducing failure store. Before this change Dataset quality details page was already solely relying on _index instead of filtering documents using data_stream properties while the main page was filtering out the documents.

Before

prev.mov

After

Screen.Recording.2025-02-10.at.12.03.58.mov

@yngrdyn yngrdyn requested a review from a team as a code owner February 10, 2025 10:59
@yngrdyn yngrdyn added release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) backport:prev-major Backport to (8.x, 8.18, 8.17, 8.16) the previous major branch and other branches in development labels Feb 10, 2025
@yngrdyn yngrdyn changed the title [Dataset quality] Rely solely on _index instead of data_stream properties [Dataset quality] 🐞 Rely solely on _index instead of data_stream properties Feb 10, 2025
Copy link
Contributor

@achyutjhunjhunwala achyutjhunjhunwala left a comment

Choose a reason for hiding this comment

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

LGTM

@yngrdyn yngrdyn merged commit d26f9ff into elastic:main Feb 11, 2025
15 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.16, 8.17, 8.18, 8.x, 9.0

https://github.com/elastic/kibana/actions/runs/13261182414

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Feb 11, 2025
…erties (elastic#210329)

Closes elastic/logs-dev#192.

## Background

This have been an long running issue within dataset quality page which
became more noticeable when introducing failure store. Before this
change `Dataset quality details` page was already solely relying on
`_index` instead of filtering documents using `data_stream` properties
while the main page was filtering out the documents.

### Before

https://github.com/user-attachments/assets/02d14cb9-81a6-4f61-a199-5d1e55443a20

### After

https://github.com/user-attachments/assets/09a4e523-b927-4147-99d1-6ceff40f1027
(cherry picked from commit d26f9ff)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Feb 11, 2025
…erties (elastic#210329)

Closes elastic/logs-dev#192.

## Background

This have been an long running issue within dataset quality page which
became more noticeable when introducing failure store. Before this
change `Dataset quality details` page was already solely relying on
`_index` instead of filtering documents using `data_stream` properties
while the main page was filtering out the documents.

### Before

https://github.com/user-attachments/assets/02d14cb9-81a6-4f61-a199-5d1e55443a20

### After

https://github.com/user-attachments/assets/09a4e523-b927-4147-99d1-6ceff40f1027
(cherry picked from commit d26f9ff)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Feb 11, 2025
…erties (elastic#210329)

Closes elastic/logs-dev#192.

## Background

This have been an long running issue within dataset quality page which
became more noticeable when introducing failure store. Before this
change `Dataset quality details` page was already solely relying on
`_index` instead of filtering documents using `data_stream` properties
while the main page was filtering out the documents.

### Before

https://github.com/user-attachments/assets/02d14cb9-81a6-4f61-a199-5d1e55443a20

### After

https://github.com/user-attachments/assets/09a4e523-b927-4147-99d1-6ceff40f1027
(cherry picked from commit d26f9ff)
@kibanamachine
Copy link
Contributor

kibanamachine commented Feb 11, 2025

💔 Some backports could not be created

Status Branch Result
8.17
8.18
8.x
9.0

Note: Successful backport PRs will be merged automatically after passing CI.

Manual backport

To create the backport manually run:

node scripts/backport --pr 210329

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Feb 11, 2025
…am properties (#210329) (#210532)

# Backport

This will backport the following commits from `main` to `8.18`:
- [[Dataset quality] 🐞 Rely solely on _index instead of data_stream
properties (#210329)](#210329)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Yngrid
Coello","email":"yngrid.coello@elastic.co"},"sourceCommit":{"committedDate":"2025-02-11T10:48:45Z","message":"[Dataset
quality] 🐞 Rely solely on _index instead of data_stream properties
(#210329)\n\nCloses
https://github.com/elastic/logs-dev/issues/192.\r\n\r\n##
Background\r\n\r\nThis have been an long running issue within dataset
quality page which\r\nbecame more noticeable when introducing failure
store. Before this\r\nchange `Dataset quality details` page was already
solely relying on\r\n`_index` instead of filtering documents using
`data_stream` properties\r\nwhile the main page was filtering out the
documents.\r\n\r\n### Before
\r\n\r\n\r\nhttps://github.com/user-attachments/assets/02d14cb9-81a6-4f61-a199-5d1e55443a20\r\n\r\n###
After\r\n\r\n\r\nhttps://github.com/user-attachments/assets/09a4e523-b927-4147-99d1-6ceff40f1027","sha":"d26f9ffbb60a7ec6327920ddb520320ed83241d3","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:prev-minor","backport:prev-major","v9.1.0"],"title":"[Dataset
quality] 🐞 Rely solely on _index instead of data_stream
properties","number":210329,"url":"https://github.com/elastic/kibana/pull/210329","mergeCommit":{"message":"[Dataset
quality] 🐞 Rely solely on _index instead of data_stream properties
(#210329)\n\nCloses
https://github.com/elastic/logs-dev/issues/192.\r\n\r\n##
Background\r\n\r\nThis have been an long running issue within dataset
quality page which\r\nbecame more noticeable when introducing failure
store. Before this\r\nchange `Dataset quality details` page was already
solely relying on\r\n`_index` instead of filtering documents using
`data_stream` properties\r\nwhile the main page was filtering out the
documents.\r\n\r\n### Before
\r\n\r\n\r\nhttps://github.com/user-attachments/assets/02d14cb9-81a6-4f61-a199-5d1e55443a20\r\n\r\n###
After\r\n\r\n\r\nhttps://github.com/user-attachments/assets/09a4e523-b927-4147-99d1-6ceff40f1027","sha":"d26f9ffbb60a7ec6327920ddb520320ed83241d3"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210329","number":210329,"mergeCommit":{"message":"[Dataset
quality] 🐞 Rely solely on _index instead of data_stream properties
(#210329)\n\nCloses
https://github.com/elastic/logs-dev/issues/192.\r\n\r\n##
Background\r\n\r\nThis have been an long running issue within dataset
quality page which\r\nbecame more noticeable when introducing failure
store. Before this\r\nchange `Dataset quality details` page was already
solely relying on\r\n`_index` instead of filtering documents using
`data_stream` properties\r\nwhile the main page was filtering out the
documents.\r\n\r\n### Before
\r\n\r\n\r\nhttps://github.com/user-attachments/assets/02d14cb9-81a6-4f61-a199-5d1e55443a20\r\n\r\n###
After\r\n\r\n\r\nhttps://github.com/user-attachments/assets/09a4e523-b927-4147-99d1-6ceff40f1027","sha":"d26f9ffbb60a7ec6327920ddb520320ed83241d3"}}]}]
BACKPORT-->

Co-authored-by: Yngrid Coello <yngrid.coello@elastic.co>
kibanamachine added a commit that referenced this pull request Feb 11, 2025
…m properties (#210329) (#210534)

# Backport

This will backport the following commits from `main` to `9.0`:
- [[Dataset quality] 🐞 Rely solely on _index instead of data_stream
properties (#210329)](#210329)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Yngrid
Coello","email":"yngrid.coello@elastic.co"},"sourceCommit":{"committedDate":"2025-02-11T10:48:45Z","message":"[Dataset
quality] 🐞 Rely solely on _index instead of data_stream properties
(#210329)\n\nCloses
https://github.com/elastic/logs-dev/issues/192.\r\n\r\n##
Background\r\n\r\nThis have been an long running issue within dataset
quality page which\r\nbecame more noticeable when introducing failure
store. Before this\r\nchange `Dataset quality details` page was already
solely relying on\r\n`_index` instead of filtering documents using
`data_stream` properties\r\nwhile the main page was filtering out the
documents.\r\n\r\n### Before
\r\n\r\n\r\nhttps://github.com/user-attachments/assets/02d14cb9-81a6-4f61-a199-5d1e55443a20\r\n\r\n###
After\r\n\r\n\r\nhttps://github.com/user-attachments/assets/09a4e523-b927-4147-99d1-6ceff40f1027","sha":"d26f9ffbb60a7ec6327920ddb520320ed83241d3","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:prev-minor","backport:prev-major","v9.1.0"],"title":"[Dataset
quality] 🐞 Rely solely on _index instead of data_stream
properties","number":210329,"url":"https://github.com/elastic/kibana/pull/210329","mergeCommit":{"message":"[Dataset
quality] 🐞 Rely solely on _index instead of data_stream properties
(#210329)\n\nCloses
https://github.com/elastic/logs-dev/issues/192.\r\n\r\n##
Background\r\n\r\nThis have been an long running issue within dataset
quality page which\r\nbecame more noticeable when introducing failure
store. Before this\r\nchange `Dataset quality details` page was already
solely relying on\r\n`_index` instead of filtering documents using
`data_stream` properties\r\nwhile the main page was filtering out the
documents.\r\n\r\n### Before
\r\n\r\n\r\nhttps://github.com/user-attachments/assets/02d14cb9-81a6-4f61-a199-5d1e55443a20\r\n\r\n###
After\r\n\r\n\r\nhttps://github.com/user-attachments/assets/09a4e523-b927-4147-99d1-6ceff40f1027","sha":"d26f9ffbb60a7ec6327920ddb520320ed83241d3"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210329","number":210329,"mergeCommit":{"message":"[Dataset
quality] 🐞 Rely solely on _index instead of data_stream properties
(#210329)\n\nCloses
https://github.com/elastic/logs-dev/issues/192.\r\n\r\n##
Background\r\n\r\nThis have been an long running issue within dataset
quality page which\r\nbecame more noticeable when introducing failure
store. Before this\r\nchange `Dataset quality details` page was already
solely relying on\r\n`_index` instead of filtering documents using
`data_stream` properties\r\nwhile the main page was filtering out the
documents.\r\n\r\n### Before
\r\n\r\n\r\nhttps://github.com/user-attachments/assets/02d14cb9-81a6-4f61-a199-5d1e55443a20\r\n\r\n###
After\r\n\r\n\r\nhttps://github.com/user-attachments/assets/09a4e523-b927-4147-99d1-6ceff40f1027","sha":"d26f9ffbb60a7ec6327920ddb520320ed83241d3"}}]}]
BACKPORT-->

Co-authored-by: Yngrid Coello <yngrid.coello@elastic.co>
kibanamachine added a commit that referenced this pull request Feb 11, 2025
…m properties (#210329) (#210533)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Dataset quality] 🐞 Rely solely on _index instead of data_stream
properties (#210329)](#210329)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Yngrid
Coello","email":"yngrid.coello@elastic.co"},"sourceCommit":{"committedDate":"2025-02-11T10:48:45Z","message":"[Dataset
quality] 🐞 Rely solely on _index instead of data_stream properties
(#210329)\n\nCloses
https://github.com/elastic/logs-dev/issues/192.\r\n\r\n##
Background\r\n\r\nThis have been an long running issue within dataset
quality page which\r\nbecame more noticeable when introducing failure
store. Before this\r\nchange `Dataset quality details` page was already
solely relying on\r\n`_index` instead of filtering documents using
`data_stream` properties\r\nwhile the main page was filtering out the
documents.\r\n\r\n### Before
\r\n\r\n\r\nhttps://github.com/user-attachments/assets/02d14cb9-81a6-4f61-a199-5d1e55443a20\r\n\r\n###
After\r\n\r\n\r\nhttps://github.com/user-attachments/assets/09a4e523-b927-4147-99d1-6ceff40f1027","sha":"d26f9ffbb60a7ec6327920ddb520320ed83241d3","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:prev-minor","backport:prev-major","v9.1.0"],"title":"[Dataset
quality] 🐞 Rely solely on _index instead of data_stream
properties","number":210329,"url":"https://github.com/elastic/kibana/pull/210329","mergeCommit":{"message":"[Dataset
quality] 🐞 Rely solely on _index instead of data_stream properties
(#210329)\n\nCloses
https://github.com/elastic/logs-dev/issues/192.\r\n\r\n##
Background\r\n\r\nThis have been an long running issue within dataset
quality page which\r\nbecame more noticeable when introducing failure
store. Before this\r\nchange `Dataset quality details` page was already
solely relying on\r\n`_index` instead of filtering documents using
`data_stream` properties\r\nwhile the main page was filtering out the
documents.\r\n\r\n### Before
\r\n\r\n\r\nhttps://github.com/user-attachments/assets/02d14cb9-81a6-4f61-a199-5d1e55443a20\r\n\r\n###
After\r\n\r\n\r\nhttps://github.com/user-attachments/assets/09a4e523-b927-4147-99d1-6ceff40f1027","sha":"d26f9ffbb60a7ec6327920ddb520320ed83241d3"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210329","number":210329,"mergeCommit":{"message":"[Dataset
quality] 🐞 Rely solely on _index instead of data_stream properties
(#210329)\n\nCloses
https://github.com/elastic/logs-dev/issues/192.\r\n\r\n##
Background\r\n\r\nThis have been an long running issue within dataset
quality page which\r\nbecame more noticeable when introducing failure
store. Before this\r\nchange `Dataset quality details` page was already
solely relying on\r\n`_index` instead of filtering documents using
`data_stream` properties\r\nwhile the main page was filtering out the
documents.\r\n\r\n### Before
\r\n\r\n\r\nhttps://github.com/user-attachments/assets/02d14cb9-81a6-4f61-a199-5d1e55443a20\r\n\r\n###
After\r\n\r\n\r\nhttps://github.com/user-attachments/assets/09a4e523-b927-4147-99d1-6ceff40f1027","sha":"d26f9ffbb60a7ec6327920ddb520320ed83241d3"}}]}]
BACKPORT-->

Co-authored-by: Yngrid Coello <yngrid.coello@elastic.co>
yngrdyn added a commit that referenced this pull request Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-major Backport to (8.x, 8.18, 8.17, 8.16) the previous major branch and other branches in development backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes v8.18.0 v8.19.0 v9.0.0 v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants