DataDog / dd-trace-dotnet

.NET Client Library for Datadog APM
https://docs.datadoghq.com/tracing/
Apache License 2.0
437 stars 137 forks source link

Revert "Add workflow monitor to all workflows (#5733)" #5741

Closed andrewlock closed 3 months ago

andrewlock commented 3 months ago

Summary of changes

Reverts the workflow monitor

Reason for change

It appears to absolutely cripple the GitHub action network speed. Actions have been failing on master ever since we merged this PR, but here are some examples:

Checkout stage

Install .NET stage

Download Azure Build assets

Implementation details

Revert the monitor, we'll have to YOLO the permissions changes with what we have

Test coverage

I've tested this is the cause by disabling it using the variable. But if it's disabled, there's kind of no point in including it in the workflows, so may as well tear it out

Other details

grmbl grmbl, gonna have to just YOLO our permission changes in that case