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

Add ".reindexed-v8-" prefix to the valid prefixes list #204819

Merged
merged 9 commits into from
Jan 7, 2025

Conversation

ymao1
Copy link
Contributor

@ymao1 ymao1 commented Dec 18, 2024

As part of v9.0 readiness, we reindex the indices in v.8 but still has data from v.7x.
As a result of the process, the reindexed indices get .reindexed-v8- prefix.
This PR add that prefix to the valid prefixes list.

To verify:

Run Kibana and ES in 7.17 (use -E path.data=your-data-path to keep the data in your local)
Create some rules that generate alerts (Observability rules to have AAD)
Let them run for a while.
Stop Kibana and ES, switch to 8.x branch and run ES and Kibana again
Open the Upgrade Assistant.
It should show the .internal.alerts-* indices
Click on them and start reindexing on the opened flyout.
Check that .reindexed-v8-internal.alerts-* index has been created
Let you rules run for a while again.
Your alerts should be updated and there shouldn't be any error on your terminal.

'.alerts-',
'.internal.preview.alerts-',
'.internal.alerts-',
'.reindexed-v8-internal.alerts',
Copy link
Contributor

@ersin-erdal ersin-erdal Dec 18, 2024

Choose a reason for hiding this comment

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

should this have a "-" suffix?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure..feel free to update!

@ersin-erdal ersin-erdal self-assigned this Jan 2, 2025
@ersin-erdal ersin-erdal added release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Jan 3, 2025
@ersin-erdal ersin-erdal added the ci:project-deploy-observability Create an Observability project label Jan 3, 2025
Copy link
Contributor

github-actions bot commented Jan 3, 2025

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@ersin-erdal ersin-erdal marked this pull request as ready for review January 6, 2025 11:54
@ersin-erdal ersin-erdal requested a review from a team as a code owner January 6, 2025 11:54
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@ersin-erdal ersin-erdal changed the title wip Add ".reindexed-v8-" prefix to the valid prefixes list Jan 6, 2025
@elasticmachine
Copy link
Contributor

⏳ Build in-progress

  • Buildkite Build
  • Commit: 54d7781
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-204819-54d77817c521

History

cc @ersin-erdal

Copy link
Contributor

@mikecote mikecote left a comment

Choose a reason for hiding this comment

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

Changes LGTM! Saw the .reindexed index get created and rules running successfully afterwards.

@ersin-erdal ersin-erdal merged commit 8a9202e into elastic:main Jan 7, 2025
8 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 7, 2025
As part of v9.0 readiness, we reindex the indices in v.8 but still has
data from v.7x.
As a result of the process, the reindexed indices get `.reindexed-v8-`
prefix.
This PR add that prefix to the valid prefixes list.

# To verify:

Run Kibana and ES in 7.17 (use `-E path.data=your-data-path` to keep the
data in your local)
Create some rules that generate alerts (Observability rules to have AAD)
Let them run for a while.
Stop Kibana and ES, switch to 8.x branch and run ES and Kibana again
Open the Upgrade Assistant.
It should show the `.internal.alerts-*` indices
Click on them and start reindexing on the opened flyout.
Check that `.reindexed-v8-internal.alerts-*` index has been created
Let you rules run for a while again.
Your alerts should be updated and there shouldn't be any error on your
terminal.

---------

Co-authored-by: Ersin Erdal <[email protected]>
Co-authored-by: Ersin Erdal <[email protected]>
(cherry picked from commit 8a9202e)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

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

Questions ?

Please refer to the Backport tool documentation

kowalczyk-krzysztof pushed a commit to kowalczyk-krzysztof/kibana that referenced this pull request Jan 7, 2025
As part of v9.0 readiness, we reindex the indices in v.8 but still has
data from v.7x.
As a result of the process, the reindexed indices get `.reindexed-v8-`
prefix.
This PR add that prefix to the valid prefixes list.


# To verify:

Run Kibana and ES in 7.17 (use `-E path.data=your-data-path` to keep the
data in your local)
Create some rules that generate alerts (Observability rules to have AAD)
Let them run for a while.
Stop Kibana and ES, switch to 8.x branch and run ES and Kibana again
Open the Upgrade Assistant.
It should show the `.internal.alerts-*` indices 
Click on them and start reindexing on the opened flyout.
Check that `.reindexed-v8-internal.alerts-*` index has been created
Let you rules run for a while again.
Your alerts should be updated and there shouldn't be any error on your
terminal.

---------

Co-authored-by: Ersin Erdal <[email protected]>
Co-authored-by: Ersin Erdal <[email protected]>
kibanamachine added a commit that referenced this pull request Jan 7, 2025
#204819) (#205809)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Add &quot;.reindexed-v8-&quot; prefix to the valid prefixes list
(#204819)](#204819)

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

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

<!--BACKPORT [{"author":{"name":"Ying
Mao","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-07T20:50:52Z","message":"Add
\".reindexed-v8-\" prefix to the valid prefixes list (#204819)\n\nAs
part of v9.0 readiness, we reindex the indices in v.8 but still
has\r\ndata from v.7x.\r\nAs a result of the process, the reindexed
indices get `.reindexed-v8-`\r\nprefix.\r\nThis PR add that prefix to
the valid prefixes list.\r\n\r\n\r\n# To verify:\r\n\r\nRun Kibana and
ES in 7.17 (use `-E path.data=your-data-path` to keep the\r\ndata in
your local)\r\nCreate some rules that generate alerts (Observability
rules to have AAD)\r\nLet them run for a while.\r\nStop Kibana and ES,
switch to 8.x branch and run ES and Kibana again\r\nOpen the Upgrade
Assistant.\r\nIt should show the `.internal.alerts-*` indices \r\nClick
on them and start reindexing on the opened flyout.\r\nCheck that
`.reindexed-v8-internal.alerts-*` index has been created\r\nLet you
rules run for a while again.\r\nYour alerts should be updated and there
shouldn't be any error on
your\r\nterminal.\r\n\r\n---------\r\n\r\nCo-authored-by: Ersin Erdal
<[email protected]>\r\nCo-authored-by: Ersin Erdal
<[email protected]>","sha":"8a9202ed8ec8a7cc8347ad228d15f3c602d75c99","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:ResponseOps","v9.0.0","backport:prev-minor","ci:project-deploy-observability"],"title":"Add
\".reindexed-v8-\" prefix to the valid prefixes
list","number":204819,"url":"https://github.com/elastic/kibana/pull/204819","mergeCommit":{"message":"Add
\".reindexed-v8-\" prefix to the valid prefixes list (#204819)\n\nAs
part of v9.0 readiness, we reindex the indices in v.8 but still
has\r\ndata from v.7x.\r\nAs a result of the process, the reindexed
indices get `.reindexed-v8-`\r\nprefix.\r\nThis PR add that prefix to
the valid prefixes list.\r\n\r\n\r\n# To verify:\r\n\r\nRun Kibana and
ES in 7.17 (use `-E path.data=your-data-path` to keep the\r\ndata in
your local)\r\nCreate some rules that generate alerts (Observability
rules to have AAD)\r\nLet them run for a while.\r\nStop Kibana and ES,
switch to 8.x branch and run ES and Kibana again\r\nOpen the Upgrade
Assistant.\r\nIt should show the `.internal.alerts-*` indices \r\nClick
on them and start reindexing on the opened flyout.\r\nCheck that
`.reindexed-v8-internal.alerts-*` index has been created\r\nLet you
rules run for a while again.\r\nYour alerts should be updated and there
shouldn't be any error on
your\r\nterminal.\r\n\r\n---------\r\n\r\nCo-authored-by: Ersin Erdal
<[email protected]>\r\nCo-authored-by: Ersin Erdal
<[email protected]>","sha":"8a9202ed8ec8a7cc8347ad228d15f3c602d75c99"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/204819","number":204819,"mergeCommit":{"message":"Add
\".reindexed-v8-\" prefix to the valid prefixes list (#204819)\n\nAs
part of v9.0 readiness, we reindex the indices in v.8 but still
has\r\ndata from v.7x.\r\nAs a result of the process, the reindexed
indices get `.reindexed-v8-`\r\nprefix.\r\nThis PR add that prefix to
the valid prefixes list.\r\n\r\n\r\n# To verify:\r\n\r\nRun Kibana and
ES in 7.17 (use `-E path.data=your-data-path` to keep the\r\ndata in
your local)\r\nCreate some rules that generate alerts (Observability
rules to have AAD)\r\nLet them run for a while.\r\nStop Kibana and ES,
switch to 8.x branch and run ES and Kibana again\r\nOpen the Upgrade
Assistant.\r\nIt should show the `.internal.alerts-*` indices \r\nClick
on them and start reindexing on the opened flyout.\r\nCheck that
`.reindexed-v8-internal.alerts-*` index has been created\r\nLet you
rules run for a while again.\r\nYour alerts should be updated and there
shouldn't be any error on
your\r\nterminal.\r\n\r\n---------\r\n\r\nCo-authored-by: Ersin Erdal
<[email protected]>\r\nCo-authored-by: Ersin Erdal
<[email protected]>","sha":"8a9202ed8ec8a7cc8347ad228d15f3c602d75c99"}}]}]
BACKPORT-->

Co-authored-by: Ying Mao <[email protected]>
crespocarlos pushed a commit to crespocarlos/kibana that referenced this pull request Jan 8, 2025
As part of v9.0 readiness, we reindex the indices in v.8 but still has
data from v.7x.
As a result of the process, the reindexed indices get `.reindexed-v8-`
prefix.
This PR add that prefix to the valid prefixes list.


# To verify:

Run Kibana and ES in 7.17 (use `-E path.data=your-data-path` to keep the
data in your local)
Create some rules that generate alerts (Observability rules to have AAD)
Let them run for a while.
Stop Kibana and ES, switch to 8.x branch and run ES and Kibana again
Open the Upgrade Assistant.
It should show the `.internal.alerts-*` indices 
Click on them and start reindexing on the opened flyout.
Check that `.reindexed-v8-internal.alerts-*` index has been created
Let you rules run for a while again.
Your alerts should be updated and there shouldn't be any error on your
terminal.

---------

Co-authored-by: Ersin Erdal <[email protected]>
Co-authored-by: Ersin Erdal <[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 (8.x) the previous minor version (i.e. one version back from main) ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants