Skip to content

Conversation

@gladjohn
Copy link
Contributor

Update benchmark-action.yml to use net 9

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Description

This pull request includes updates to the .NET version used in the benchmarking process and changes to the build properties for the benchmark project. The most important changes are as follows:

.NET Version Update:

  • Updated the .NET version from 8.x to 9.x in the GitHub Actions workflow file .github/workflows/benchmark-action.yml to ensure the benchmarks run with the latest version.

Build Properties Update:

  • Changed the target framework from net8.0 to net9.0 in benchmark/Directory.Build.props to align with the updated .NET version.
  • Modified the signing properties in benchmark/Directory.Build.props to disable assembly signing by setting SignAssembly and DelaySign to False.

Fixes #3270

@gladjohn gladjohn requested a review from a team as a code owner February 27, 2025 22:23
@github-actions
Copy link

Summary

Summary
Generated on: 2/27/2025 - 10:37:31 PM
Coverage date: 2/27/2025 - 10:36:33 PM - 2/27/2025 - 10:37:26 PM
Parser: MultiReport (3x Cobertura)
Assemblies: 0
Classes: 0
Files: 0
Line coverage:
Covered lines: 0
Uncovered lines: 0
Coverable lines: 0
Total lines: 0
Covered branches: 0
Total branches: 0
Method coverage: Feature is only available for sponsors

Coverage

No assemblies have been covered.

@gladjohn gladjohn marked this pull request as draft March 1, 2025 22:07
@gladjohn
Copy link
Contributor Author

gladjohn commented Mar 1, 2025

We now have a different error, and based on the discussion with Jenny, no further action needed on this now.

{"error":"invalid_client","error_description":"AADSTS7000215: Invalid client secret provided. Ensure the secret being sent in the request is the client secret value, not the client secret ID, for a secret added to app 'f6b698c0-140c-448f-8155-4aa9bf77ceba'. Trace ID: 19585a17-8a9d-4575-b36b-95c2d3bc4600 Correlation ID: 42eb56e7-81f8-4282-879b-36d379d22753 Timestamp: 2025-02-27 22:20:50Z","error_codes":[7000215],"timestamp":"2025-02-27 22:20:50Z","trace_id":"19585a17-8a9d-4575-b36b-95c2d3bc4600","correlation_id":"42eb56e7-81f8-4282-879b-36d379d22753","error_uri":"https://login.microsoftonline.com/error?code=7000215"} 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GitHub Actions Fails Due to Missing .NET SDK 9.0

2 participants