[8.x] [Security Solution] Migration of Alert Page controls for non-default Spaces. (#200058) #200093
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…Spaces. (elastic#200058) ## Summary Recently, we created a PR to migrate the alert page filters controls to `8.16`. Unfortunately, it does not do migration for non-default spaces so any users upgrading to `8.16` will face the issue where Alert page errors out as shown in below screenshot.  ## Desk Testing 1. Checkout to `v8.15` branch by running `git checkout 8.15`. 2. Create a new space and go to that space. 3. Go to the alert page and do some modifications to the page controls. This store `v8.15` page controls in local storage. - You can, for example, delete one page control. - Change selected value for one page control. - Additionally, you can also add a custom control. 4. Checkout `main` now and repeat the above steps. 5. Your changes should be retained on the alert page and there should not be any error. (cherry picked from commit b7ca722)
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
cc @logeekal |
logeekal
added a commit
that referenced
this pull request
Nov 14, 2024
…efault Spaces. (#200058) (#200104) # Backport This will backport the following commits from `main` to `8.16`: - [[Security Solution] Migration of Alert Page controls for non-default Spaces. (#200058)](#200058) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Jatin Kathuria","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-13T20:52:00Z","message":"[Security Solution] Migration of Alert Page controls for non-default Spaces. (#200058)\n\n## Summary\r\n\r\nRecently, we created a PR to migrate the alert page filters controls to\r\n`8.16`. Unfortunately, it does not do migration for non-default spaces\r\nso any users upgrading to `8.16` will face the issue where Alert page\r\nerrors out as shown in below screenshot.\r\n\r\n\r\n\r\n\r\n\r\n## Desk Testing\r\n\r\n1. Checkout to `v8.15` branch by running `git checkout 8.15`. \r\n2. Create a new space and go to that space.\r\n3. Go to the alert page and do some modifications to the page controls.\r\nThis store `v8.15` page controls in local storage.\r\n - You can, for example, delete one page control.\r\n - Change selected value for one page control.\r\n - Additionally, you can also add a custom control.\r\n4. Checkout `main` now and repeat the above steps.\r\n5. Your changes should be retained on the alert page and there should\r\nnot be any error.","sha":"b7ca7228315393c6672f638982dbac5196c9ad90","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["backport","release_note:skip","v9.0.0","Team:Threat Hunting:Investigations","backport:prev-minor","v8.17.0","v8.16.1"],"number":200058,"url":"https://github.com/elastic/kibana/pull/200058","mergeCommit":{"message":"[Security Solution] Migration of Alert Page controls for non-default Spaces. (#200058)\n\n## Summary\r\n\r\nRecently, we created a PR to migrate the alert page filters controls to\r\n`8.16`. Unfortunately, it does not do migration for non-default spaces\r\nso any users upgrading to `8.16` will face the issue where Alert page\r\nerrors out as shown in below screenshot.\r\n\r\n\r\n\r\n\r\n\r\n## Desk Testing\r\n\r\n1. Checkout to `v8.15` branch by running `git checkout 8.15`. \r\n2. Create a new space and go to that space.\r\n3. Go to the alert page and do some modifications to the page controls.\r\nThis store `v8.15` page controls in local storage.\r\n - You can, for example, delete one page control.\r\n - Change selected value for one page control.\r\n - Additionally, you can also add a custom control.\r\n4. Checkout `main` now and repeat the above steps.\r\n5. Your changes should be retained on the alert page and there should\r\nnot be any error.","sha":"b7ca7228315393c6672f638982dbac5196c9ad90"}},"sourceBranch":"main","suggestedTargetBranches":["8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/200058","number":200058,"mergeCommit":{"message":"[Security Solution] Migration of Alert Page controls for non-default Spaces. (#200058)\n\n## Summary\r\n\r\nRecently, we created a PR to migrate the alert page filters controls to\r\n`8.16`. Unfortunately, it does not do migration for non-default spaces\r\nso any users upgrading to `8.16` will face the issue where Alert page\r\nerrors out as shown in below screenshot.\r\n\r\n\r\n\r\n\r\n\r\n## Desk Testing\r\n\r\n1. Checkout to `v8.15` branch by running `git checkout 8.15`. \r\n2. Create a new space and go to that space.\r\n3. Go to the alert page and do some modifications to the page controls.\r\nThis store `v8.15` page controls in local storage.\r\n - You can, for example, delete one page control.\r\n - Change selected value for one page control.\r\n - Additionally, you can also add a custom control.\r\n4. Checkout `main` now and repeat the above steps.\r\n5. Your changes should be retained on the alert page and there should\r\nnot be any error.","sha":"b7ca7228315393c6672f638982dbac5196c9ad90"}},{"branch":"8.x","label":"v8.17.0","labelRegex":"^v8.17.0$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/200093","number":200093,"state":"MERGED","mergeCommit":{"sha":"255086dc66b19be0f879f5797660f3ddf1339d93","message":"[8.x] [Security Solution] Migration of Alert Page controls for non-default Spaces. (#200058) (#200093)\n\n# Backport\n\nThis will backport the following commits from `main` to `8.x`:\n- [[Security Solution] Migration of Alert Page controls for non-default\nSpaces. (#200058)](https://github.com/elastic/kibana/pull/200058)\n\n<!--- Backport version: 9.4.3 -->\n\n### Questions ?\nPlease refer to the [Backport tool\ndocumentation](https://github.com/sqren/backport)\n\n<!--BACKPORT [{\"author\":{\"name\":\"Jatin\nKathuria\",\"email\":\"[email protected]\"},\"sourceCommit\":{\"committedDate\":\"2024-11-13T20:52:00Z\",\"message\":\"[Security\nSolution] Migration of Alert Page controls for non-default Spaces.\n(#200058)\\n\\n## Summary\\r\\n\\r\\nRecently, we created a PR to migrate the\nalert page filters controls to\\r\\n`8.16`. Unfortunately, it does not do\nmigration for non-default spaces\\r\\nso any users upgrading to `8.16`\nwill face the issue where Alert page\\r\\nerrors out as shown in below\nscreenshot.\\r\\n\\r\\n\\r\\n\\r\\n\\r\\n\\r\\n##\nDesk Testing\\r\\n\\r\\n1. Checkout to `v8.15` branch by running `git\ncheckout 8.15`. \\r\\n2. Create a new space and go to that space.\\r\\n3. Go\nto the alert page and do some modifications to the page\ncontrols.\\r\\nThis store `v8.15` page controls in local storage.\\r\\n -\nYou can, for example, delete one page control.\\r\\n - Change selected\nvalue for one page control.\\r\\n - Additionally, you can also add a\ncustom control.\\r\\n4. Checkout `main` now and repeat the above\nsteps.\\r\\n5. Your changes should be retained on the alert page and there\nshould\\r\\nnot be any\nerror.\",\"sha\":\"b7ca7228315393c6672f638982dbac5196c9ad90\",\"branchLabelMapping\":{\"^v9.0.0$\":\"main\",\"^v8.17.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"release_note:skip\",\"v9.0.0\",\"Team:Threat\nHunting:Investigations\",\"backport:prev-minor\"],\"title\":\"[Security\nSolution] Migration of Alert Page controls for non-default\nSpaces.\",\"number\":200058,\"url\":\"https://github.com/elastic/kibana/pull/200058\",\"mergeCommit\":{\"message\":\"[Security\nSolution] Migration of Alert Page controls for non-default Spaces.\n(#200058)\\n\\n## Summary\\r\\n\\r\\nRecently, we created a PR to migrate the\nalert page filters controls to\\r\\n`8.16`. Unfortunately, it does not do\nmigration for non-default spaces\\r\\nso any users upgrading to `8.16`\nwill face the issue where Alert page\\r\\nerrors out as shown in below\nscreenshot.\\r\\n\\r\\n\\r\\n\\r\\n\\r\\n\\r\\n##\nDesk Testing\\r\\n\\r\\n1. Checkout to `v8.15` branch by running `git\ncheckout 8.15`. \\r\\n2. Create a new space and go to that space.\\r\\n3. Go\nto the alert page and do some modifications to the page\ncontrols.\\r\\nThis store `v8.15` page controls in local storage.\\r\\n -\nYou can, for example, delete one page control.\\r\\n - Change selected\nvalue for one page control.\\r\\n - Additionally, you can also add a\ncustom control.\\r\\n4. Checkout `main` now and repeat the above\nsteps.\\r\\n5. Your changes should be retained on the alert page and there\nshould\\r\\nnot be any\nerror.\",\"sha\":\"b7ca7228315393c6672f638982dbac5196c9ad90\"}},\"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/200058\",\"number\":200058,\"mergeCommit\":{\"message\":\"[Security\nSolution] Migration of Alert Page controls for non-default Spaces.\n(#200058)\\n\\n## Summary\\r\\n\\r\\nRecently, we created a PR to migrate the\nalert page filters controls to\\r\\n`8.16`. Unfortunately, it does not do\nmigration for non-default spaces\\r\\nso any users upgrading to `8.16`\nwill face the issue where Alert page\\r\\nerrors out as shown in below\nscreenshot.\\r\\n\\r\\n\\r\\n\\r\\n\\r\\n\\r\\n##\nDesk Testing\\r\\n\\r\\n1. Checkout to `v8.15` branch by running `git\ncheckout 8.15`. \\r\\n2. Create a new space and go to that space.\\r\\n3. Go\nto the alert page and do some modifications to the page\ncontrols.\\r\\nThis store `v8.15` page controls in local storage.\\r\\n -\nYou can, for example, delete one page control.\\r\\n - Change selected\nvalue for one page control.\\r\\n - Additionally, you can also add a\ncustom control.\\r\\n4. Checkout `main` now and repeat the above\nsteps.\\r\\n5. Your changes should be retained on the alert page and there\nshould\\r\\nnot be any\nerror.\",\"sha\":\"b7ca7228315393c6672f638982dbac5196c9ad90\"}}]}]\nBACKPORT-->\n\nCo-authored-by: Jatin Kathuria <[email protected]>"}},{"branch":"8.16","label":"v8.16.1","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT-->
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport
This will backport the following commits from
mainto8.x:Questions ?
Please refer to the Backport tool documentation