search

Azure / azure-sdk-tools

Tools repository leveraged by the Azure SDK team.
MIT License
111 stars 176 forks source link

[stress] Diagnostic output from script should be refined #6043

Open richardpark-msft opened 1 year ago

richardpark-msft commented 1 year ago

We recently ran into this with @HarshaNalluru.

There was a compilation failure in his stress test, which was easily diagnosable but there was so much diagnostic output that it wasn't easy to see where the failure was. Once we found the error line from the docker build (when it compiled his code) it was obvious, but getting there wasn't as easy as we'd like it to be.

We should be able to improve this a bit, possibly by just re-echoing the most relevant command failure's output at the end, rather than just printing out the failed command line, which is what we do today.

lmolkova commented 1 year ago

we can look into forwarding docker container and build logs with severity higher than warning to appinsights and add alert on build errors. Would it help?

richardpark-msft commented 1 year ago

Probably not for this - these are errors that are happening on your local client so the output is right there already.

We did talk about extracting some helpful telemetry at some point just to track failures, but that'd be disconnected from this effort.