dotnet / performance

This repo contains benchmarks used for testing the performance of all .NET Runtimes
MIT License
684 stars 263 forks source link

Don't include AzureCLI for PDN if running a public or PR build #4295

Closed LoopedBard3 closed 1 month ago

LoopedBard3 commented 1 month ago

Fix public runs not running due to attempting to instantiate a service connection that does not exist on our public runs.

DrewScoggins commented 1 month ago

I am confused. The title talks about secret synchronization, but the change seems to be about limiting when we download PDN.

LoopedBard3 commented 1 month ago

I am confused. The title talks about secret synchronization, but the change seems to be about limiting when we download PDN.

Updated. The initial draft PR I made included the assumption that the cause of the error was related to secret-synchronization, this was not the case.