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.

Sign up for GitHub

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

Jump to bottom

Improve locking on hosts in high host count accounts #2776

Closed
chiiph opened this issue Nov 3, 2021 · 2 comments
Closed

Improve locking on hosts in high host count accounts #2776

chiiph opened this issue Nov 3, 2021 · 2 comments
Assignees
@chiiph
Labels
bug Something isn't working as documented
Projects
🚀 Core engineering (triweekly) (curre...

Comments

@chiiph
Copy link
Contributor

chiiph commented Nov 3, 2021

We learned that the amount of locking time in the hosts can be problematic in high host count deployments with software inventory enabled.

To address this, we'll reduce the amount of concurrent writes to the host table as much as possible per fleet instance. The bulk updating of host seen time is also problematic as it can lock the whole table, so it should be separated into another table.
@chiiph chiiph added bug Something isn't working as documented 8 labels Nov 3, 2021
@chiiph chiiph self-assigned this Nov 3, 2021
@chiiph chiiph added this to 🔜 Ready to resume in 🚀 Core engineering (triweekly) (currently: Fleet v4.6.0) via automation Nov 3, 2021
@chiiph
Copy link
Contributor Author

chiiph commented Nov 3, 2021

The PR for this is #2753

@chiiph chiiph moved this from 🔜 Ready to resume to 🔜 In progress right now in 🚀 Core engineering (triweekly) (currently: Fleet v4.6.0) Nov 3, 2021
@chiiph chiiph moved this from 🔜 In progress right now to 🔜 Self-checked, review plz in 🚀 Core engineering (triweekly) (currently: Fleet v4.6.0) Nov 5, 2021
@chiiph chiiph moved this from 🔜 Self-checked, review plz to 🔜 QA'd, ready for release in 🚀 Core engineering (triweekly) (currently: Fleet v4.6.0) Nov 12, 2021
@chiiph
Copy link
Contributor Author

chiiph commented Nov 12, 2021

Skipping "please QA" state on this one as it was tested in the loadtest env.

@chiiph chiiph closed this as completed Apr 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chiiph chiiph

Labels
bug Something isn't working as documented
Projects
No open projects
🚀 Core engineering (triweekly) (curre...
  
🔜 QA'd, ready for release
Milestone
No milestone
Development

No branches or pull requests
1 participant
@chiiph