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

Bringing more granularity to reset command option #161

Merged
merged 18 commits into from
Oct 20, 2020

Conversation

anshuman-goel
Copy link
Contributor

@anshuman-goel anshuman-goel commented Oct 15, 2020

Summary of the Pull Request

Often times I want to specifically delete certain aspects like only deleting all the jobs. But I have to delete it one by one.
Also, after new deployment I want to delete all the stuff except for containers and there is no option to do.

What is this about?
This will bring in more granularity in reset option.
Let me know if anything should not be allowed which can lead to system in weird state.

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?
Tried out various all the combinations in reset for it.

@anshuman-goel anshuman-goel marked this pull request as draft October 15, 2020 18:39
@anshuman-goel anshuman-goel marked this pull request as ready for review October 15, 2020 18:51
src/cli/onefuzz/api.py Outdated Show resolved Hide resolved
src/cli/onefuzz/api.py Show resolved Hide resolved
src/cli/onefuzz/api.py Outdated Show resolved Hide resolved
src/cli/onefuzz/api.py Show resolved Hide resolved
src/cli/onefuzz/api.py Show resolved Hide resolved
@bmc-msft bmc-msft merged commit ecd322b into microsoft:main Oct 20, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Apr 17, 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.

2 participants