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

Added deploy.py flags onefuzz_app_id and auto_create_cli_app to allow custom app registrations. #2305

Merged

Conversation

nharper285
Copy link
Contributor

@nharper285 nharper285 commented Aug 26, 2022

Summary of the Pull Request

What is this about?
Previously, the OneFuzz CLI app registration id was hardcoded into the deployment script. Now, users are allowed to specify an app registration and specify whether they want one created automatically.

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?

  • Manually deploy with custom id
  • Manually upgrade
  • Manually deploy without id or flag
  • Manually deploy without id and flag

src/deployment/deploy.py Outdated Show resolved Hide resolved
@nharper285 nharper285 merged commit 0c88116 into microsoft:main Aug 31, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Sep 30, 2022
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.

3 participants