Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make sandbox deployments easier #1383

Merged
merged 2 commits into from
Nov 13, 2024
Merged

Make sandbox deployments easier #1383

merged 2 commits into from
Nov 13, 2024

Conversation

ccostino
Copy link
Contributor

@ccostino ccostino commented Oct 30, 2024

A note to PR reviewers: it may be helpful to review our code review documentation to know what to keep in mind while reviewing pull requests.

Description

This changeset makes a few adjustments to our sandbox environment config to make the sandbox deployments of the API much easier. It does this with the following:

  • Adds a few environment variable values to the sandbox.yml file to cover the ones that were not there previously
  • Adds a new script that can be run in place of memorizing any commands
  • Adds documentation on how to configure and run the sandbox environment

Security Considerations

  • We still need to be mindful of environment variables and not sharing sensitive credentials/secrets.
  • This helps make the management of the environment variables much easier for sandbox deployment.

This changeset makes a few adjustments to our sandbox environment config to make the sandbox deployments of the API much easier.  It does this with the following:

* Adds a few environment variable values to the sandbox.yml file to cover the ones that were not there previously
* Adds a new script that can be run in place of memorizing any commands
* Adds documentation on how to configure and run the sandbox environment

Signed-off-by: Carlo Costino <carlo.costino@gsa.gov>
Signed-off-by: Carlo Costino <carlo.costino@gsa.gov>
@ccostino ccostino merged commit c242d2d into main Nov 13, 2024
7 checks passed
@ccostino ccostino deleted the update-sandbox-steps branch November 13, 2024 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants