Skip to content

Test failure: Microsoft.Extensions.Caching.Memory.CapacityTests.DoNotAddIfSizeOverflows #39241

Closed

Description

@BruceForstall

Windows, x86, COMPlus_JitStressRegs=8

https://dev.azure.com/dnceng/public/_build/results?buildId=726304&view=ms.vss-test-web.build-test-results-tab&runId=22490824&resultId=102068&paneView=dotnet-dnceng.dnceng-build-release-tasks.helix-test-information-tab

I don't think this is related to JitStressRegs; I can't repro it locally like that. I think the test simply fails sometimes on loaded systems due to this code:

            // Wait for compaction to complete
            Assert.True(await sem.WaitAsync(TimeSpan.FromSeconds(10)));

Kusto shows 6 such failures in the last 30 days (all Windows x86, JitStress).

Could this timeout number be made much, much higher?

Activity

  1. danmoseley commented on Jul 14, 2020

    @danmoseley
    Contributor

    Increasing the timeout would not affect tests that pass today, right? I think it would be fine if you want to increase those numbers.

  2. BruceForstall commented on Jul 14, 2020

    @BruceForstall
    ContributorAuthor

    Increasing the timeout would not affect tests that pass today, right?

    No idea. That would be up to the test owner to determine.

    I think it would be fine if you want to increase those numbers.

    I would prefer the test/area owner make that change, to contribute to a better, more stable CI.

  3. added this to the Future milestone on Jul 16, 2020
  4. BruceForstall commented on Jul 26, 2020

    @BruceForstall
    ContributorAuthor

    Dup'ing to #33993

  5. ghost locked as resolved and limited conversation to collaborators on Dec 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions