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 Analytics] [Entity Store] Run init requests sequentially to prevent resource exists error #198268

Merged
merged 9 commits into from
Oct 30, 2024

Conversation

tiansivive
Copy link
Contributor

@tiansivive tiansivive commented Oct 30, 2024

Summary

This PR fixes an issue where running init for both user and host entity engines in parallel would cause a race condition while enabling the risk engine, resulting in a Resource already exists error.

@tiansivive tiansivive added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Theme: entity_analytics Team:Entity Analytics Security Entity Analytics Team v8.16.0 backport:version Backport to applied version labels labels Oct 30, 2024
@tiansivive tiansivive requested a review from a team as a code owner October 30, 2024 09:24
@elasticmachine
Copy link
Contributor

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

@tiansivive tiansivive enabled auto-merge (squash) October 30, 2024 10:33
@tiansivive tiansivive merged commit 6a50066 into elastic:main Oct 30, 2024
44 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.16

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

@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 +69.0B

History

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.16 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 198268

Questions ?

Please refer to the Backport tool documentation

tiansivive added a commit to tiansivive/kibana that referenced this pull request Oct 31, 2024
…revent resource exists error (elastic#198268)

## Summary

This PR fixes an issue where running init for both `user` and `host`
entity engines in parallel would cause a race condition while enabling
the risk engine, resulting in a `Resource already exists` error.

---------

Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 6a50066)
tiansivive added a commit to tiansivive/kibana that referenced this pull request Oct 31, 2024
…revent resource exists error (elastic#198268)

## Summary

This PR fixes an issue where running init for both `user` and `host`
entity engines in parallel would cause a race condition while enabling
the risk engine, resulting in a `Resource already exists` error.

---------

Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 6a50066)
tiansivive added a commit that referenced this pull request Oct 31, 2024
…y to prevent resource exists error (#198268) (#198476)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Entity Analytics] [Entity Store] Run init requests sequentially to
prevent resource exists error (#198268) (6a50066)

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

### 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-10-30T16:03:07Z","message":"[Entity
Analytics] [Entity Store] Run init requests sequentially to prevent
resource exists error (#198268)\n\n## Summary\r\n\r\nThis PR fixes an
issue where running init for both `user` and `host`\r\nentity engines in
parallel would cause a race condition while enabling\r\nthe risk engine,
resulting in a `Resource already exists`
error.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<[email protected]>","sha":"6a50066e00ae38a64c5365fd66b4dc32857ba1fc"},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[]}]
BACKPORT-->
tiansivive added a commit that referenced this pull request Oct 31, 2024
…ly to prevent resource exists error (#198268) (#198477)

# Backport

This will backport the following commits from `main` to `8.16`:
- [Entity Analytics] [Entity Store] Run init requests sequentially to
prevent resource exists error (#198268) (6a50066)

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

### 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-10-30T16:03:07Z","message":"[Entity
Analytics] [Entity Store] Run init requests sequentially to prevent
resource exists error (#198268)\n\n## Summary\r\n\r\nThis PR fixes an
issue where running init for both `user` and `host`\r\nentity engines in
parallel would cause a race condition while enabling\r\nthe risk engine,
resulting in a `Resource already exists`
error.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<[email protected]>","sha":"6a50066e00ae38a64c5365fd66b4dc32857ba1fc"},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[]}]
BACKPORT-->
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 Theme: entity_analytics v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants