-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Closed
Labels
Description
What is the problem you're trying to solve
As commented by @samuelkarp here
#8596 (comment)
copying it here as well
As is, I think the code has a few races in it which can lead to inconsistency in reading the generated config file (no mutex around the logic in UpdateRuntimeConfig combined with not writing the file atomically). If we don't remove the logic, I'd at least like to get those issues fixed.
Describe the solution you'd like
Update the code to avoid race conditions
Additional context
No response