Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BuildManager instances acquire its own BuildTelemetry instance (#8444) #8552

Merged
merged 4 commits into from
Mar 11, 2023

Commits on Mar 10, 2023

  1. BuildManager instances acquire its own BuildTelemetry instance (dotne…

    …t#8444)
    
    Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1708215
    
    Context
    In VS there are multiple instances of BuildManager called asynchronously. For DTB and normal build and maybe other which I have not identified yet.
    
    Changes Made
    BuildManager instances acquire its own BuildTelemetry instance as oppose to sharing single BuildTelemetry instance in non thread safe manner.
    
    Testing
    Locally
    rokonec committed Mar 10, 2023
    Configuration menu
    Copy the full SHA
    1f83c7d View commit details
    Browse the repository at this point in the history
  2. Turn off static graph restore. (dotnet#8498)

    Our CI builds fails because of bug NuGet/Home#12373. 
    It is fixed in NuGet/NuGet.Client#5010. 
    We are waiting for it to flow to CI machines. Meanwhile this PR applies a workaround.
    
    Note: This PR needs to be reverted once it happens.
    AR-May authored and rokonec committed Mar 10, 2023
    Configuration menu
    Copy the full SHA
    18c8529 View commit details
    Browse the repository at this point in the history
  3. Bumping version

    rokonec committed Mar 10, 2023
    Configuration menu
    Copy the full SHA
    7aabaea View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    d0b20c9 View commit details
    Browse the repository at this point in the history