Fix concurrent writes for UpdateContainerStats

Signed-off-by: Daniel Lenar <dlenar@vailsys.com> (cherry picked from commit a48dbef) Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
containerd · Jul 14, 2023 · 9f65014 · 9f65014
1 parent 3787ef4
commit 9f65014
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/cri/store/sandbox/sandbox.go b/pkg/cri/store/sandbox/sandbox.go
@@ -125,8 +125,8 @@ func (s *Store) List() []Sandbox {
 }
 
 func (s *Store) UpdateContainerStats(id string, newContainerStats *stats.ContainerStats) error {
-	s.lock.RLock()
-	defer s.lock.RUnlock()
+	s.lock.Lock()
+	defer s.lock.Unlock()
 	id, err := s.idIndex.Get(id)
 	if err != nil {
 		if err == truncindex.ErrNotExist {