Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Crash repro as task #830

Closed
wants to merge 96 commits into from
Closed

Conversation

bmc-msft
Copy link
Contributor

@bmc-msft bmc-msft commented Apr 23, 2021

No description provided.

@bmc-msft bmc-msft marked this pull request as ready for review July 1, 2021 19:10
@bmc-msft bmc-msft requested review from ranweiler and chkeita July 1, 2021 21:14
// while we monitor the runtime of the debugger, we don't fail the task if
// the debugger exits non-zero. This frequently happens during normal use of
// debuggers.
monitor_process(output, "crash-repro".to_string(), true, None)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't we be storing the logs of the analysis ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. monitor_process logs stdout and stderr to application insights.

@mgreisen
Copy link
Contributor

This is work that we will probably want to take in a later timeframe, but we need a few other things in place first. Closing for now.
cc : @ranweiler @chkeita @stishkin

@mgreisen mgreisen closed this Oct 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants