Fix concurrent read write error for entity store #1484
Merged
+9
−2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the issue
Fixes #1479. Entity store's operations to read and write the logFile and logGroups maps can occur from diff go routines corresponding to files monitored by the agent and can lead to a panic if accessed concurrently.
Description of changes
Use a mutex lock to prevent concurrent access to the log maps.
License
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Tests
Wrote up a script to generate a high load of 1000 logs being constantly written to and monitored by the agent. Prior to fix, I see the agent crash every few mins:
Post this fix, I no longer see the agent crash.
Requirements
Before commit the code, please do the following steps.
make fmt
andmake fmt-sh
make lint