Merge pull request kubernetes#4724 from mikedanese/readiness-states-c…

…leanup Don't lock readinessStates on concurrent reads
duda · Feb 23, 2015 · b261d8c · b261d8c
2 parents cbcd355 + d6fb8ce
commit b261d8c
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/pkg/kubelet/probe.go b/pkg/kubelet/probe.go
@@ -162,7 +162,8 @@ func newReadinessStates() *readinessStates {
 }
 
 type readinessStates struct {
-	sync.Mutex
+	// guards states
+	sync.RWMutex
 	states map[string]bool
 }
 
@@ -174,8 +175,8 @@ func (r *readinessStates) IsReady(c api.ContainerStatus) bool {
 }
 
 func (r *readinessStates) get(key string) bool {
-	r.Lock()
-	defer r.Unlock()
+	r.RLock()
+	defer r.RUnlock()
 	state, found := r.states[key]
 	return state && found
 }