github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-27 21:47:07 +00:00
Code Issues Projects Releases Wiki Activity

Use RWMutex in watchBasedManager

Browse Source
This commit is contained in:
caiweidong 2019-08-13 13:09:21 +08:00
parent 0610bf0c7e
commit 92851d213a
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
pkg/kubelet/util/manager/watch_based_manager.go
View File

@ -61,7 +61,7 @@ type objectCache struct {
newObject newObjectFunc newObject newObjectFunc
groupResource schema.GroupResource groupResource schema.GroupResource
lock sync.Mutex lock sync.RWMutex
items map[objectKey]*objectCacheItem items map[objectKey]*objectCacheItem
} }
@ -117,7 +117,7 @@ func (c *objectCache) AddReference(namespace, name string) {
key := objectKey{namespace: namespace, name: name} key := objectKey{namespace: namespace, name: name}
// AddReference is called from RegisterPod thus it needs to be efficient. // AddReference is called from RegisterPod thus it needs to be efficient.
// Thus, it is only increaisng refCount and in case of first registration // Thus, it is only increasing refCount and in case of first registration
// of a given object it starts corresponding reflector. // of a given object it starts corresponding reflector.
// It's responsibility of the first Get operation to wait until the // It's responsibility of the first Get operation to wait until the
// reflector propagated the store. // reflector propagated the store.
@ -158,9 +158,9 @@ func (c *objectCache) key(namespace, name string) string {
func (c *objectCache) Get(namespace, name string) (runtime.Object, error) { func (c *objectCache) Get(namespace, name string) (runtime.Object, error) {
key := objectKey{namespace: namespace, name: name} key := objectKey{namespace: namespace, name: name}
c.lock.Lock() c.lock.RLock()
item, exists := c.items[key] item, exists := c.items[key]
c.lock.Unlock() c.lock.RUnlock()
if !exists { if !exists {
return nil, fmt.Errorf("object %q/%q not registered", namespace, name) return nil, fmt.Errorf("object %q/%q not registered", namespace, name)