Fixes occasional data race flake with TestSDKServerAddListValue

Puts gsListUpdates call after the mutex lock
googleforgames · Nov 14, 2023 · b026150 · b026150
1 parent 28eebb9
commit b026150
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/pkg/sdkserver/sdkserver.go b/pkg/sdkserver/sdkserver.go
@@ -1175,10 +1175,11 @@ func (s *SDKServer) updateList(ctx context.Context) error {
 	}
 	gsCopy := gs.DeepCopy()
 
-	s.logger.WithField("batchListUpdates", s.gsListUpdates).Debug("Batch updating List(s)")
 	s.gsUpdateMutex.Lock()
 	defer s.gsUpdateMutex.Unlock()
 
+	s.logger.WithField("batchListUpdates", s.gsListUpdates).Debug("Batch updating List(s)")
+
 	names := []string{}
 
 	for name, listReq := range s.gsListUpdates {