Fix concurrent writes for UpdateContainerStats

danlenar · danlenar · commit a48dbefc151f · 2023-02-17T15:13:18.000-06:00
Signed-off-by: Daniel Lenar &lt;dlenar@vailsys.com&gt;
diff --git a/pkg/cri/store/sandbox/sandbox.go b/pkg/cri/store/sandbox/sandbox.go
@@ -129,8 +129,8 @@ func (s *Store) List() []Sandbox {
 // stats present in 'newContainerStats'. Returns errdefs.ErrNotFound
 // if the sandbox does not exist in the store.
 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 {
