Ankit Jain
Ankit Jain
One of the workload tests failed with: ``` Failed to download package 'OpenTelemetry.Instrumentation.Http.1.9.0' from 'https://pkgs.dev.azure.com/dnceng/9ee6d478-d288-47f7-aacc-f6e6d082ae6d/_packaging/45bacae2-5efb-47c8-91e5-8ec20c22b4f8/nuget/v3/flat2/opentelemetry.instrumentation.http/1.9.0/opentelemetry.instrumentation.http.1.9.0.nupkg'.
 The HTTP request to 'GET https://pkgs.dev.azure.com/dnceng/9ee6d478-d288-47f7-aacc-f6e6d082ae6d/_packaging/45bacae2-5efb-47c8-91e5-8ec20c22b4f8/nuget/v3/flat2/opentelemetry.instrumentation.http/1.9.0/opentelemetry.instrumentation.http.1.9.0.nupkg' has timed out after 100000ms.
 ``` We can re-try.
This is covered by @RussKie's on-going work. Re-assigning.
Some of my comments might be non-sensical, or could be addressed in a follow-up PR. I'll let others decide that, since I'm very new to this code:)
> This test isn't flaky Closing this in favor of fixing the PR.
> > This test isn't flaky > > Agreed, any failure in this test is real Oops, missing hitting the `close` button!
Can you add a test for the playground app in `tests/Aspire.Playground.Tests` like [`d715ace` (#5294)](https://github.com/dotnet/aspire/pull/5294/commits/d715ace34988527024b2f9f59f18c316d23ef374) ?
Could you also please add an explanation in the PR description of what is being done here, and why?
@adamint would you be interested in taking this up, since you recently worked on dashboard playwright tests?
Do we have a similar bad pattern being followed in other integrations?
cc @mitchdenny