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

[Index Management] Fix mapping_tab error due to _data_stream_timestamp #183187

Merged
merged 3 commits into from May 13, 2024

Conversation

saikatsarkar056
Copy link
Contributor

@saikatsarkar056 saikatsarkar056 commented May 10, 2024

An error or warning message is displayed on the index mappings tab when viewing mappings for a backing index. This looks like _data_stream_timestamp is missing in the mappingsConfigurationSchema.

Fixes #183055

Screen-recording after bug-fix

Screen.Recording.2024-05-10.at.11.47.03.AM.mov

@saikatsarkar056 saikatsarkar056 requested a review from a team as a code owner May 10, 2024 17:38
@saikatsarkar056 saikatsarkar056 self-assigned this May 10, 2024
@saikatsarkar056 saikatsarkar056 added Team:EnterpriseSearch release_note:skip Skip the PR/issue when compiling release notes v8.15.0 v8.14.0 bug Fixes for quality problems that affect the customer experience labels May 10, 2024
@saikatsarkar056 saikatsarkar056 changed the title Fix mapping_tab error due to _data_stream_timestamp [Index Management] Fix mapping_tab error due to _data_stream_timestamp May 10, 2024
@alisonelizabeth alisonelizabeth added the ci:cloud-deploy Create or update a Cloud deployment label May 10, 2024
@alisonelizabeth
Copy link
Contributor

@elasticmachine merge upstream

@alisonelizabeth
Copy link
Contributor

/ci

@alisonelizabeth
Copy link
Contributor

Hi @saikatsarkar056 this change looks good. However, I would like to validate it on a cloud deployment before merging as I'm still a little confused why I saw 2 different types of errors. I want to make sure both are addressed.

@alisonelizabeth
Copy link
Contributor

@saikatsarkar056 I verified your change on cloud and everything looks good 👍. I'm going to ask an engineer on my team do the code review.

Screenshot 2024-05-11 at 8 05 18 AM

Copy link
Contributor

@yuliacech yuliacech left a comment

Choose a reason for hiding this comment

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

Thanks a lot for fixing this, @saikatsarkar056!
Code changes LGTM 👍
I think we are missing an e2e test for the index details page so I'll open an issue to track that.

@alisonelizabeth
Copy link
Contributor

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

kibana-ci commented May 13, 2024

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
indexManagement 662.9KB 662.9KB +56.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @saikatsarkar056

@alisonelizabeth alisonelizabeth added the backport:prev-minor Backport to the previous minor version (i.e. one version back from main) label May 13, 2024
@alisonelizabeth alisonelizabeth merged commit 3d79d43 into elastic:main May 13, 2024
20 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request May 13, 2024
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.14

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

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request May 13, 2024
…imestamp (#183187) (#183306)

# Backport

This will backport the following commits from `main` to `8.14`:
- [[Index Management] Fix mapping_tab error due to
_data_stream_timestamp
(#183187)](#183187)

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

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

<!--BACKPORT [{"author":{"name":"Saikat
Sarkar","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-05-13T15:15:19Z","message":"[Index
Management] Fix mapping_tab error due to _data_stream_timestamp
(#183187)","sha":"3d79d4349443608c92ad42b981d5f010921af97f","branchLabelMapping":{"^v8.15.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:skip","Team:EnterpriseSearch","backport:prev-minor","ci:cloud-deploy","v8.14.0","v8.15.0"],"title":"[Index
Management] Fix mapping_tab error due to
_data_stream_timestamp","number":183187,"url":"#183187
Management] Fix mapping_tab error due to _data_stream_timestamp
(#183187)","sha":"3d79d4349443608c92ad42b981d5f010921af97f"}},"sourceBranch":"main","suggestedTargetBranches":["8.14"],"targetPullRequestStates":[{"branch":"8.14","label":"v8.14.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.15.0","branchLabelMappingKey":"^v8.15.0$","isSourceBranch":true,"state":"MERGED","url":"#183187
Management] Fix mapping_tab error due to _data_stream_timestamp
(#183187)","sha":"3d79d4349443608c92ad42b981d5f010921af97f"}}]}]
BACKPORT-->

Co-authored-by: Saikat Sarkar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to the previous minor version (i.e. one version back from main) bug Fixes for quality problems that affect the customer experience ci:cloud-deploy Create or update a Cloud deployment release_note:skip Skip the PR/issue when compiling release notes Team:EnterpriseSearch v8.14.0 v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Index Management] Mappings tab shows error for backing indices
5 participants