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

[Security Solution][Data Quality Dashboard] fix pattern state reset on ilm phase filter change #198549

Merged
merged 1 commit into from
Nov 4, 2024

Conversation

kapral18
Copy link
Contributor

@kapral18 kapral18 commented Oct 31, 2024

addresses #196523

  • Fixes ilm phase change propagation on patterns.
  • Adds missing tests for useResultsRollup functionality

UI changes

Before

Screen.Recording.2024-08-15.at.15.48.05.mov

After

Screen.Recording.Nov.4.2024.mp4

@kapral18 kapral18 self-assigned this Oct 31, 2024
@kapral18 kapral18 requested a review from a team as a code owner October 31, 2024 14:40
@kapral18 kapral18 added release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting Security Solution Threat Hunting Team Team:Threat Hunting:Explore backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Oct 31, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting (Team:Threat Hunting)

@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting-explore (Team:Threat Hunting:Explore)

@kapral18 kapral18 added ci:cloud-deploy Create or update a Cloud deployment ci:project-deploy-security Create a Security Serverless Project labels Oct 31, 2024
@kapral18 kapral18 force-pushed the fix/DQD/191053-fix-ilm-phase-reset branch 3 times, most recently from 5169e4d to 6b5dfd9 Compare November 4, 2024 00:35
@kapral18 kapral18 force-pushed the fix/DQD/191053-fix-ilm-phase-reset branch 2 times, most recently from bd7133d to 29afab6 Compare November 4, 2024 11:28
@kapral18 kapral18 enabled auto-merge (squash) November 4, 2024 11:29
@kapral18 kapral18 force-pushed the fix/DQD/191053-fix-ilm-phase-reset branch from 29afab6 to 7f26ceb Compare November 4, 2024 13:03
@kapral18 kapral18 force-pushed the fix/DQD/191053-fix-ilm-phase-reset branch from 7f26ceb to 972210d Compare November 4, 2024 13:22
@elasticmachine
Copy link
Contributor

elasticmachine commented Nov 4, 2024

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 6146 6147 +1

Async chunks

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

id before after diff
securitySolution 21.0MB 21.0MB -136.0B

History

cc @kapral18

@kapral18 kapral18 merged commit ddf55ea into elastic:main Nov 4, 2024
40 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

@kapral18 kapral18 deleted the fix/DQD/191053-fix-ilm-phase-reset branch November 4, 2024 15:11
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 4, 2024
…n ilm phase filter change (elastic#198549)

addresses elastic#196523

- Fixes ilm phase change propagation on patterns.
- Adds missing tests for useResultsRollup functionality

## UI changes

### Before

https://github.com/user-attachments/assets/78a1d809-6a9a-4bfc-88a9-079f829a2017

### After

https://github.com/user-attachments/assets/f689fcc9-e1c6-4ccf-a7ca-8f13e9507ba4
(cherry picked from commit ddf55ea)
@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

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Nov 5, 2024
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

1 similar comment
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

kibanamachine added a commit that referenced this pull request Nov 7, 2024
…eset on ilm phase filter change (#198549) (#198806)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Security Solution][Data Quality Dashboard] fix pattern state reset
on ilm phase filter change
(#198549)](#198549)

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

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

<!--BACKPORT [{"author":{"name":"Karen
Grigoryan","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-04T15:09:12Z","message":"[Security
Solution][Data Quality Dashboard] fix pattern state reset on ilm phase
filter change (#198549)\n\naddresses #196523\r\n\r\n- Fixes ilm phase
change propagation on patterns.\r\n- Adds missing tests for
useResultsRollup functionality\r\n\r\n## UI changes\r\n\r\n###
Before\r\n\r\nhttps://github.com/user-attachments/assets/78a1d809-6a9a-4bfc-88a9-079f829a2017\r\n\r\n###
After\r\n\r\nhttps://github.com/user-attachments/assets/f689fcc9-e1c6-4ccf-a7ca-8f13e9507ba4","sha":"ddf55ea3a79ad7439dc02cb1f93291b1bc95c3b9","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
Hunting","Team:Threat
Hunting:Explore","backport:prev-minor","ci:cloud-deploy","ci:project-deploy-security"],"title":"[Security
Solution][Data Quality Dashboard] fix pattern state reset on ilm phase
filter
change","number":198549,"url":"https://github.com/elastic/kibana/pull/198549","mergeCommit":{"message":"[Security
Solution][Data Quality Dashboard] fix pattern state reset on ilm phase
filter change (#198549)\n\naddresses #196523\r\n\r\n- Fixes ilm phase
change propagation on patterns.\r\n- Adds missing tests for
useResultsRollup functionality\r\n\r\n## UI changes\r\n\r\n###
Before\r\n\r\nhttps://github.com/user-attachments/assets/78a1d809-6a9a-4bfc-88a9-079f829a2017\r\n\r\n###
After\r\n\r\nhttps://github.com/user-attachments/assets/f689fcc9-e1c6-4ccf-a7ca-8f13e9507ba4","sha":"ddf55ea3a79ad7439dc02cb1f93291b1bc95c3b9"}},"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/198549","number":198549,"mergeCommit":{"message":"[Security
Solution][Data Quality Dashboard] fix pattern state reset on ilm phase
filter change (#198549)\n\naddresses #196523\r\n\r\n- Fixes ilm phase
change propagation on patterns.\r\n- Adds missing tests for
useResultsRollup functionality\r\n\r\n## UI changes\r\n\r\n###
Before\r\n\r\nhttps://github.com/user-attachments/assets/78a1d809-6a9a-4bfc-88a9-079f829a2017\r\n\r\n###
After\r\n\r\nhttps://github.com/user-attachments/assets/f689fcc9-e1c6-4ccf-a7ca-8f13e9507ba4","sha":"ddf55ea3a79ad7439dc02cb1f93291b1bc95c3b9"}}]}]
BACKPORT-->

Co-authored-by: Karen Grigoryan <[email protected]>
@kibanamachine kibanamachine added v8.17.0 and removed backport missing Added to PRs automatically when the are determined to be missing a backport. labels Nov 7, 2024
@kapral18
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.16

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

Questions ?

Please refer to the Backport tool documentation

kapral18 added a commit to kapral18/kibana that referenced this pull request Nov 18, 2024
…n ilm phase filter change (elastic#198549)

addresses elastic#196523

- Fixes ilm phase change propagation on patterns.
- Adds missing tests for useResultsRollup functionality

## UI changes

### Before

https://github.com/user-attachments/assets/78a1d809-6a9a-4bfc-88a9-079f829a2017

### After

https://github.com/user-attachments/assets/f689fcc9-e1c6-4ccf-a7ca-8f13e9507ba4
(cherry picked from commit ddf55ea)
kapral18 added a commit that referenced this pull request Nov 18, 2024
…reset on ilm phase filter change (#198549) (#200618)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[Security Solution][Data Quality Dashboard] fix pattern state reset
on ilm phase filter change
(#198549)](#198549)

<!--- Backport version: 8.9.8 -->

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

<!--BACKPORT [{"author":{"name":"Karen
Grigoryan","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-04T15:09:12Z","message":"[Security
Solution][Data Quality Dashboard] fix pattern state reset on ilm phase
filter change (#198549)\n\naddresses #196523\r\n\r\n- Fixes ilm phase
change propagation on patterns.\r\n- Adds missing tests for
useResultsRollup functionality\r\n\r\n## UI changes\r\n\r\n###
Before\r\n\r\nhttps://github.com/user-attachments/assets/78a1d809-6a9a-4bfc-88a9-079f829a2017\r\n\r\n###
After\r\n\r\nhttps://github.com/user-attachments/assets/f689fcc9-e1c6-4ccf-a7ca-8f13e9507ba4","sha":"ddf55ea3a79ad7439dc02cb1f93291b1bc95c3b9","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
Hunting","Team:Threat
Hunting:Explore","backport:prev-minor","ci:cloud-deploy","ci:project-deploy-security","v8.17.0"],"number":198549,"url":"https://github.com/elastic/kibana/pull/198549","mergeCommit":{"message":"[Security
Solution][Data Quality Dashboard] fix pattern state reset on ilm phase
filter change (#198549)\n\naddresses #196523\r\n\r\n- Fixes ilm phase
change propagation on patterns.\r\n- Adds missing tests for
useResultsRollup functionality\r\n\r\n## UI changes\r\n\r\n###
Before\r\n\r\nhttps://github.com/user-attachments/assets/78a1d809-6a9a-4bfc-88a9-079f829a2017\r\n\r\n###
After\r\n\r\nhttps://github.com/user-attachments/assets/f689fcc9-e1c6-4ccf-a7ca-8f13e9507ba4","sha":"ddf55ea3a79ad7439dc02cb1f93291b1bc95c3b9"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/198549","number":198549,"mergeCommit":{"message":"[Security
Solution][Data Quality Dashboard] fix pattern state reset on ilm phase
filter change (#198549)\n\naddresses #196523\r\n\r\n- Fixes ilm phase
change propagation on patterns.\r\n- Adds missing tests for
useResultsRollup functionality\r\n\r\n## UI changes\r\n\r\n###
Before\r\n\r\nhttps://github.com/user-attachments/assets/78a1d809-6a9a-4bfc-88a9-079f829a2017\r\n\r\n###
After\r\n\r\nhttps://github.com/user-attachments/assets/f689fcc9-e1c6-4ccf-a7ca-8f13e9507ba4","sha":"ddf55ea3a79ad7439dc02cb1f93291b1bc95c3b9"}},{"branch":"8.x","label":"v8.17.0","labelRegex":"^v8.17.0$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/198806","number":198806,"state":"MERGED","mergeCommit":{"sha":"fa0bddaac166c1bd17989a54b183677203f1812b","message":"[8.x]
[Security Solution][Data Quality Dashboard] fix pattern state reset on
ilm phase filter change (#198549) (#198806)\n\n# Backport\n\nThis will
backport the following commits from `main` to `8.x`:\n- [[Security
Solution][Data Quality Dashboard] fix pattern state reset\non ilm phase
filter
change\n(#198549)](https://github.com/elastic/kibana/pull/198549)\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\":\"Karen\nGrigoryan\",\"email\":\"[email protected]\"},\"sourceCommit\":{\"committedDate\":\"2024-11-04T15:09:12Z\",\"message\":\"[Security\nSolution][Data
Quality Dashboard] fix pattern state reset on ilm phase\nfilter change
(#198549)\\n\\naddresses #196523\\r\\n\\r\\n- Fixes ilm phase\nchange
propagation on patterns.\\r\\n- Adds missing tests for\nuseResultsRollup
functionality\\r\\n\\r\\n## UI
changes\\r\\n\\r\\n###\nBefore\\r\\n\\r\\nhttps://github.com/user-attachments/assets/78a1d809-6a9a-4bfc-88a9-079f829a2017\\r\\n\\r\\n###\nAfter\\r\\n\\r\\nhttps://github.com/user-attachments/assets/f689fcc9-e1c6-4ccf-a7ca-8f13e9507ba4\",\"sha\":\"ddf55ea3a79ad7439dc02cb1f93291b1bc95c3b9\",\"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\",\"Team:Threat\nHunting:Explore\",\"backport:prev-minor\",\"ci:cloud-deploy\",\"ci:project-deploy-security\"],\"title\":\"[Security\nSolution][Data
Quality Dashboard] fix pattern state reset on ilm
phase\nfilter\nchange\",\"number\":198549,\"url\":\"https://github.com/elastic/kibana/pull/198549\",\"mergeCommit\":{\"message\":\"[Security\nSolution][Data
Quality Dashboard] fix pattern state reset on ilm phase\nfilter change
(#198549)\\n\\naddresses #196523\\r\\n\\r\\n- Fixes ilm phase\nchange
propagation on patterns.\\r\\n- Adds missing tests for\nuseResultsRollup
functionality\\r\\n\\r\\n## UI
changes\\r\\n\\r\\n###\nBefore\\r\\n\\r\\nhttps://github.com/user-attachments/assets/78a1d809-6a9a-4bfc-88a9-079f829a2017\\r\\n\\r\\n###\nAfter\\r\\n\\r\\nhttps://github.com/user-attachments/assets/f689fcc9-e1c6-4ccf-a7ca-8f13e9507ba4\",\"sha\":\"ddf55ea3a79ad7439dc02cb1f93291b1bc95c3b9\"}},\"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/198549\",\"number\":198549,\"mergeCommit\":{\"message\":\"[Security\nSolution][Data
Quality Dashboard] fix pattern state reset on ilm phase\nfilter change
(#198549)\\n\\naddresses #196523\\r\\n\\r\\n- Fixes ilm phase\nchange
propagation on patterns.\\r\\n- Adds missing tests for\nuseResultsRollup
functionality\\r\\n\\r\\n## UI
changes\\r\\n\\r\\n###\nBefore\\r\\n\\r\\nhttps://github.com/user-attachments/assets/78a1d809-6a9a-4bfc-88a9-079f829a2017\\r\\n\\r\\n###\nAfter\\r\\n\\r\\nhttps://github.com/user-attachments/assets/f689fcc9-e1c6-4ccf-a7ca-8f13e9507ba4\",\"sha\":\"ddf55ea3a79ad7439dc02cb1f93291b1bc95c3b9\"}}]}]\nBACKPORT-->\n\nCo-authored-by:
Karen Grigoryan <[email protected]>"}}]}] BACKPORT-->
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:cloud-deploy Create or update a Cloud deployment ci:project-deploy-security Create a Security Serverless Project release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Explore Team:Threat Hunting Security Solution Threat Hunting Team v8.16.1 v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants