Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

renew_attestations uses a lot of CPU #7411

Closed
richvdh opened this issue May 5, 2020 · 6 comments
Closed

renew_attestations uses a lot of CPU #7411

richvdh opened this issue May 5, 2020 · 6 comments
Assignees
@anoadragon453
Labels
A-Performance Performance, both client-facing and admin-facing

Comments

@richvdh
Copy link
Member

richvdh commented May 5, 2020

I had to slow it down on matrix.org: 14bb4cf. We should probably make that permanent
@richvdh
Copy link
Member Author

richvdh commented May 5, 2020

release blocker because it's important for matrix.org

@babolivier babolivier added the A-Performance Performance, both client-facing and admin-facing label May 5, 2020
@clokep
Copy link
Member

clokep commented May 5, 2020

(This seems to partially revert #7344.)

@richvdh
Copy link
Member Author

richvdh commented May 5, 2020

(This seems to partially revert #7344.)

yes, it removes the magic needed for tracking logcontexts across parallel jobs, because we no longer run the jobs in parallel.

@anoadragon453 anoadragon453 self-assigned this May 6, 2020
@anoadragon453 anoadragon453 mentioned this issue May 6, 2020
Merged
@anoadragon453
Copy link
Member

With #7411 merged, I believe this can be closed.

@richvdh
Copy link
Member Author

richvdh commented May 22, 2020

With #7411 merged, I believe this can be closed.

itym #7442

@richvdh
Copy link
Member Author

richvdh commented May 22, 2020

this was basically a dup of #5936

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@anoadragon453 anoadragon453

Labels
A-Performance Performance, both client-facing and admin-facing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@clokep @anoadragon453 @richvdh @babolivier