You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@yunikorn.apache.org by "Peter Bacsko (Jira)" <ji...@apache.org> on 2022/12/01 16:46:00 UTC

[jira] [Created] (YUNIKORN-1441) Data race in TestTimeoutPlaceholderAllocReleased

Peter Bacsko created YUNIKORN-1441:
--------------------------------------

             Summary: Data race in TestTimeoutPlaceholderAllocReleased
                 Key: YUNIKORN-1441
                 URL: https://issues.apache.org/jira/browse/YUNIKORN-1441
             Project: Apache YuniKorn
          Issue Type: Bug
          Components: test - unit
            Reporter: Peter Bacsko
            Assignee: Peter Bacsko


Found the following data race when running unit tests locally for core:

{noformat}
==================
WARNING: DATA RACE
Read at 0x00c0004ceda0 by goroutine 18:
  github.com/apache/yunikorn-core/pkg/scheduler/objects.(*Application).timeoutPlaceholderProcessing()
      /home/bacskop/repos/incubator-yunikorn-core/pkg/scheduler/objects/application.go:415 +0xcf8
  github.com/apache/yunikorn-core/pkg/scheduler/objects.(*Application).timeoutPlaceholderProcessing-fm()
      <autogenerated>:1 +0x39

Previous write at 0x00c0004ceda0 by goroutine 17:
  github.com/apache/yunikorn-core/pkg/scheduler/objects.(*Application).addPlaceholderData()
      /home/bacskop/repos/incubator-yunikorn-core/pkg/scheduler/objects/application.go:1752 +0xb1
  github.com/apache/yunikorn-core/pkg/scheduler/objects.TestTimeoutPlaceholderAllocReleased()
      /home/bacskop/repos/incubator-yunikorn-core/pkg/scheduler/objects/application_test.go:1313 +0x384
  testing.tRunner()
      /snap/go/9991/src/testing/testing.go:1439 +0x213
  testing.(*T).Run.func1()
      /snap/go/9991/src/testing/testing.go:1486 +0x47

Goroutine 18 (running) created at:
  time.goFunc()
      /snap/go/9991/src/time/sleep.go:176 +0x47

Goroutine 17 (running) created at:
  testing.(*T).Run()
      /snap/go/9991/src/testing/testing.go:1486 +0x724
  testing.runTests.func1()
      /snap/go/9991/src/testing/testing.go:1839 +0x99
  testing.tRunner()
      /snap/go/9991/src/testing/testing.go:1439 +0x213
  testing.runTests()
      /snap/go/9991/src/testing/testing.go:1837 +0x7e4
  testing.(*M).Run()
      /snap/go/9991/src/testing/testing.go:1719 +0xa71
  main.main()
      _testmain.go:415 +0x3a9
==================
{noformat}

We need to use a locked version of {{addPlaceholderData()}}.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@yunikorn.apache.org
For additional commands, e-mail: issues-help@yunikorn.apache.org