Craig Condit created YUNIKORN-1216:
--------------------------------------

             Summary: Data race in PlaceholderManager
                 Key: YUNIKORN-1216
                 URL: https://issues.apache.org/jira/browse/YUNIKORN-1216
             Project: Apache YuniKorn
          Issue Type: Bug
          Components: shim - kubernetes
            Reporter: Craig Condit
            Assignee: Craig Condit


During a recent test run, a data race was detected in PlaceholderManager:


{noformat}
==================
WARNING: DATA RACE
Write at 0x00c0015a6618 by goroutine 55:
  
github.com/apache/yunikorn-k8shim/pkg/cache.(*PlaceholderManager).setCleanupTime()
      
/home/runner/work/yunikorn-k8shim/yunikorn-k8shim/pkg/cache/placeholder_manager.go:198
 +0xa9
  runtime.call16()
      /opt/hostedtoolcache/go/1.16.15/x64/src/runtime/asm_amd64.s:550 +0x3d
  testing.(*T).FailNow()
      <autogenerated>:1 +0x44
  gotest.tools/assert.TestingT.FailNow-fm()
      
/home/runner/go/pkg/mod/[email protected]+incompatible/assert/assert.go:83 
+0x4f
  gotest.tools/assert.assert()
      
/home/runner/go/pkg/mod/[email protected]+incompatible/assert/assert.go:137 
+0x1fa
  gotest.tools/assert.Equal()
      
/home/runner/go/pkg/mod/[email protected]+incompatible/assert/assert.go:260 
+0x144
  github.com/apache/yunikorn-k8shim/pkg/cache.TestPlaceholderManagerCleanup()
      
/home/runner/work/yunikorn-k8shim/yunikorn-k8shim/pkg/cache/placeholder_manager_test.go:370
 +0x6b9
  testing.tRunner()
      /opt/hostedtoolcache/go/1.16.15/x64/src/testing/testing.go:1203 +0x202

Previous read at 0x00c0015a6618 by goroutine 52:
  
github.com/apache/yunikorn-k8shim/pkg/cache.(*PlaceholderManager).Start.func1()
      
/home/runner/work/yunikorn-k8shim/yunikorn-k8shim/pkg/cache/placeholder_manager.go:165
 +0xd9

Goroutine 55 (running) created at:
  testing.(*T).Run()
      /opt/hostedtoolcache/go/1.16.15/x64/src/testing/testing.go:1248 +0x5d7
  testing.runTests.func1()
      /opt/hostedtoolcache/go/1.16.15/x64/src/testing/testing.go:1521 +0xa6
  testing.tRunner()
      /opt/hostedtoolcache/go/1.16.15/x64/src/testing/testing.go:1203 +0x202
  testing.runTests()
      /opt/hostedtoolcache/go/1.16.15/x64/src/testing/testing.go:1519 +0x612
  testing.(*M).Run()
      /opt/hostedtoolcache/go/1.16.15/x64/src/testing/testing.go:1427 +0x3b3
  main.main()
      _testmain.go:341 +0x356

Goroutine 52 (running) created at:
  github.com/apache/yunikorn-k8shim/pkg/cache.(*PlaceholderManager).Start()
      
/home/runner/work/yunikorn-k8shim/yunikorn-k8shim/pkg/cache/placeholder_manager.go:157
 +0x1a6
  github.com/apache/yunikorn-k8shim/pkg/cache.TestPlaceholderManagerCleanup()
      
/home/runner/work/yunikorn-k8shim/yunikorn-k8shim/pkg/cache/placeholder_manager_test.go:363
 +0x3c4
  testing.tRunner()
      /opt/hostedtoolcache/go/1.16.15/x64/src/testing/testing.go:1203 +0x202
==================
{noformat}




--
This message was sent by Atlassian Jira
(v8.20.7#820007)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to