From eb5d2ca9d8efd6dcbbea582920c700ae86db4132 Mon Sep 17 00:00:00 2001 From: Abhishek Kr Srivastav Date: Wed, 20 Mar 2024 14:23:33 +0530 Subject: [PATCH] Added locks when accessing dummyStorage to prevent data race Removed redundant locks Removed locks and used existing func --- .../k8s.io/apiserver/pkg/storage/cacher/cacher_whitebox_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/staging/src/k8s.io/apiserver/pkg/storage/cacher/cacher_whitebox_test.go b/staging/src/k8s.io/apiserver/pkg/storage/cacher/cacher_whitebox_test.go index 8caa3551a77..72d42367b7f 100644 --- a/staging/src/k8s.io/apiserver/pkg/storage/cacher/cacher_whitebox_test.go +++ b/staging/src/k8s.io/apiserver/pkg/storage/cacher/cacher_whitebox_test.go @@ -1665,7 +1665,7 @@ func TestWaitUntilWatchCacheFreshAndForceAllEvents(t *testing.T) { ResourceVersion: "105", }, verifyBackingStore: func(t *testing.T, s *dummyStorage) { - require.NotEqual(t, 0, s.requestWatchProgressCounter, "expected store.RequestWatchProgressCounter to be > 0. It looks like watch progress wasn't requested!") + require.NotEqual(t, 0, s.getRequestWatchProgressCounter(), "expected store.RequestWatchProgressCounter to be > 0. It looks like watch progress wasn't requested!") }, },