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

[Entity Store] Add enablement errors to entity store management page #198668

Merged

Conversation

tiansivive
Copy link
Contributor

@tiansivive tiansivive commented Nov 1, 2024

Summary

This PR adds callouts for any errors that occur during entity store enablement and deletion from the management page. These are the same callouts as currently seen in the Entity Analytics dashboard.

Screenshot 2024-11-01 at 13 31 47

@tiansivive tiansivive added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:Entity Analytics Security Entity Analytics Team v8.16.0 backport:version Backport to applied version labels v8.17.0 labels Nov 1, 2024
@tiansivive tiansivive requested a review from a team as a code owner November 1, 2024 12:35
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-entity-analytics (Team:Entity Analytics)

iconType="alert"
>
<p>
{(deleteEntityEngineMutation.error as { body: { message: string } }).body.message}
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a way we can add this to the mutation typing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

probably, there's a refactor issue already here

I'll add a comment there

Copy link
Member

@machadoum machadoum Nov 4, 2024

Choose a reason for hiding this comment

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

useMutation<DataType, ErrorType> receive accepts types, the first is the data and the second the error.
So, the change would be on useInitEntityEngineMutation function.

@tiansivive tiansivive enabled auto-merge (squash) November 4, 2024 12:11
@tiansivive tiansivive merged commit 4a0a736 into elastic:main Nov 4, 2024
44 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.16, 8.x

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

@elasticmachine
Copy link
Contributor

💚 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
securitySolution 21.0MB 21.0MB +994.0B

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 4, 2024
…lastic#198668)

## Summary

This PR adds callouts for any errors that occur during entity store
enablement and deletion from the management page. These are the same
callouts as currently seen in the Entity Analytics dashboard.

![Screenshot 2024-11-01 at 13 31
47](https://github.com/user-attachments/assets/162074c2-e977-4e12-a9a1-50efbd7f74b0)

(cherry picked from commit 4a0a736)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 4, 2024
…lastic#198668)

## Summary

This PR adds callouts for any errors that occur during entity store
enablement and deletion from the management page. These are the same
callouts as currently seen in the Entity Analytics dashboard.

![Screenshot 2024-11-01 at 13 31
47](https://github.com/user-attachments/assets/162074c2-e977-4e12-a9a1-50efbd7f74b0)

(cherry picked from commit 4a0a736)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.16
8.x

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 Nov 4, 2024
…t page (#198668) (#198784)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[Entity Store] Add enablement errors to entity store management page
(#198668)](#198668)

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

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

<!--BACKPORT [{"author":{"name":"Tiago Vila
Verde","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-04T13:57:32Z","message":"[Entity
Store] Add enablement errors to entity store management page
(#198668)\n\n## Summary\r\n\r\nThis PR adds callouts for any errors that
occur during entity store\r\nenablement and deletion from the management
page. These are the same\r\ncallouts as currently seen in the Entity
Analytics dashboard.\r\n\r\n![Screenshot 2024-11-01 at 13
31\r\n47](https://github.com/user-attachments/assets/162074c2-e977-4e12-a9a1-50efbd7f74b0)","sha":"4a0a73674e5f51d4296e121537dd59dd63eef589","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:Entity
Analytics","v8.16.0","backport:version","v8.17.0"],"title":"[Entity
Store] Add enablement errors to entity store management
page","number":198668,"url":"https://github.com/elastic/kibana/pull/198668","mergeCommit":{"message":"[Entity
Store] Add enablement errors to entity store management page
(#198668)\n\n## Summary\r\n\r\nThis PR adds callouts for any errors that
occur during entity store\r\nenablement and deletion from the management
page. These are the same\r\ncallouts as currently seen in the Entity
Analytics dashboard.\r\n\r\n![Screenshot 2024-11-01 at 13
31\r\n47](https://github.com/user-attachments/assets/162074c2-e977-4e12-a9a1-50efbd7f74b0)","sha":"4a0a73674e5f51d4296e121537dd59dd63eef589"}},"sourceBranch":"main","suggestedTargetBranches":["8.16","8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/198668","number":198668,"mergeCommit":{"message":"[Entity
Store] Add enablement errors to entity store management page
(#198668)\n\n## Summary\r\n\r\nThis PR adds callouts for any errors that
occur during entity store\r\nenablement and deletion from the management
page. These are the same\r\ncallouts as currently seen in the Entity
Analytics dashboard.\r\n\r\n![Screenshot 2024-11-01 at 13
31\r\n47](https://github.com/user-attachments/assets/162074c2-e977-4e12-a9a1-50efbd7f74b0)","sha":"4a0a73674e5f51d4296e121537dd59dd63eef589"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Tiago Vila Verde <[email protected]>
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Nov 6, 2024
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

1 similar comment
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

kibanamachine added a commit that referenced this pull request Nov 8, 2024
… page (#198668) (#198785)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Entity Store] Add enablement errors to entity store management page
(#198668)](#198668)

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

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

<!--BACKPORT [{"author":{"name":"Tiago Vila
Verde","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-04T13:57:32Z","message":"[Entity
Store] Add enablement errors to entity store management page
(#198668)\n\n## Summary\r\n\r\nThis PR adds callouts for any errors that
occur during entity store\r\nenablement and deletion from the management
page. These are the same\r\ncallouts as currently seen in the Entity
Analytics dashboard.\r\n\r\n![Screenshot 2024-11-01 at 13
31\r\n47](https://github.com/user-attachments/assets/162074c2-e977-4e12-a9a1-50efbd7f74b0)","sha":"4a0a73674e5f51d4296e121537dd59dd63eef589","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:Entity
Analytics","v8.16.0","backport:version","v8.17.0"],"title":"[Entity
Store] Add enablement errors to entity store management
page","number":198668,"url":"https://github.com/elastic/kibana/pull/198668","mergeCommit":{"message":"[Entity
Store] Add enablement errors to entity store management page
(#198668)\n\n## Summary\r\n\r\nThis PR adds callouts for any errors that
occur during entity store\r\nenablement and deletion from the management
page. These are the same\r\ncallouts as currently seen in the Entity
Analytics dashboard.\r\n\r\n![Screenshot 2024-11-01 at 13
31\r\n47](https://github.com/user-attachments/assets/162074c2-e977-4e12-a9a1-50efbd7f74b0)","sha":"4a0a73674e5f51d4296e121537dd59dd63eef589"}},"sourceBranch":"main","suggestedTargetBranches":["8.16","8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/198668","number":198668,"mergeCommit":{"message":"[Entity
Store] Add enablement errors to entity store management page
(#198668)\n\n## Summary\r\n\r\nThis PR adds callouts for any errors that
occur during entity store\r\nenablement and deletion from the management
page. These are the same\r\ncallouts as currently seen in the Entity
Analytics dashboard.\r\n\r\n![Screenshot 2024-11-01 at 13
31\r\n47](https://github.com/user-attachments/assets/162074c2-e977-4e12-a9a1-50efbd7f74b0)","sha":"4a0a73674e5f51d4296e121537dd59dd63eef589"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Tiago Vila Verde <[email protected]>
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Entity Analytics Security Entity Analytics Team v8.16.0 v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants