github/containerd
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

Merge pull request #8130 from danlenar/main

Browse Source 
Fix concurrent writes for UpdateContainerStats
This commit is contained in:
Fu Wei 2023-02-19 17:04:35 +08:00 committed by GitHub
commit 887395aa12
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
pkg/cri/store/sandbox/sandbox.go
View File

@ -129,8 +129,8 @@ func (s *Store) List() []Sandbox {
// stats present in 'newContainerStats'. Returns errdefs.ErrNotFound // stats present in 'newContainerStats'. Returns errdefs.ErrNotFound
// if the sandbox does not exist in the store. // if the sandbox does not exist in the store.
func (s *Store) UpdateContainerStats(id string, newContainerStats *stats.ContainerStats) error { func (s *Store) UpdateContainerStats(id string, newContainerStats *stats.ContainerStats) error {
s.lock.RLock() s.lock.Lock()
defer s.lock.RUnlock() defer s.lock.Unlock()
id, err := s.idIndex.Get(id) id, err := s.idIndex.Get(id)
if err != nil { if err != nil {
if err == truncindex.ErrNotExist { if err == truncindex.ErrNotExist {