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

Add a link to documentation on OOM task failure - #3602

Open
Kanan B (kananb) wants to merge 5 commits into
mainfrom
kanan/oom-logging
Open

Add a link to documentation on OOM task failure#3602
Kanan B (kananb) wants to merge 5 commits into
mainfrom
kanan/oom-logging

Conversation

@kananb

Copy link
Copy Markdown
Contributor

Summary of the Pull Request

Add more logging to OOM task failures, including a link to our docs on such events.

PR Checklist

  • Applies to work item: #xxx
  • CLA signed. If not, go over here and sign the CLI.
  • Tests added/passed
  • Requires documentation to be updated
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

@codecov-commenter

Codecov Comments Bot (codecov-commenter) commented Oct 25, 2023

Copy link
Copy Markdown

Codecov Report

Merging #3602 (71d870d) into main (d50fd48) will increase coverage by 1.06%.
The diff coverage is 0.00%.

@@ Coverage Diff @@## main #3602 +/- ##
==========================================
+ Coverage 39.95% 41.02% +1.06% 
==========================================
Files 303 303 Lines 37724 37736 +12 Branches 1738 1738 ==========================================
+ Hits 15073 15481 +408 + Misses 22330 21934 -396 
Partials 321 321 
FilesCoverage Δ
src/agent/onefuzz-task/src/managed/cmd.rs0.00% <0.00%> (ø)

... and 11 files with indirect coverage changes

@kananb
Kanan B (kananb) marked this pull request as ready for review October 31, 2023 01:39
@kananbKanan B (kananb) changed the title Add room for a link to documentation on OOM task failureAdd a link to documentation on OOM task failureOct 31, 2023
Sign up for freeto 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.

2 participants

@kananb@codecov-commenter