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

Downloading files to locally repro crashes #3160

Merged
merged 9 commits into from Jun 7, 2023
Merged

Conversation

kananb
Copy link
Contributor

@kananb kananb commented Jun 5, 2023

Summary of the Pull Request

Add a command to the CLI for downloading the files to locally repro a crash
Command:
onefuzz repro get_files <report_container> <report_name> [--output_dir <path>] [--include-setup]

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

Validation Steps Performed

Ran the command with correct and invalid arguments.

@kananb kananb requested a review from chkeita June 5, 2023 22:49
@kananb kananb changed the title User/kanan/local repro Downloading files to locally repro crashes Jun 5, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jun 5, 2023

Codecov Report

Merging #3160 (13f920c) into main (b44cff5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3160   +/-   ##
=======================================
  Coverage   29.97%   29.97%           
=======================================
  Files         327      327           
  Lines       38816    38816           
=======================================
  Hits        11634    11634           
  Misses      27182    27182           

src/cli/onefuzz/api.py Outdated Show resolved Hide resolved
src/cli/onefuzz/api.py Outdated Show resolved Hide resolved
@kananb kananb merged commit 8e2e11f into main Jun 7, 2023
24 checks passed
@kananb kananb deleted the user/kanan/local-repro branch June 7, 2023 15:17
@AdamL-Microsoft AdamL-Microsoft mentioned this pull request Jun 20, 2023
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.

None yet

4 participants