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

Consolidate deploy.sh scripts #4381

Closed
wants to merge 1 commit into from

Conversation

marrobi
Copy link
Member

@marrobi marrobi commented Feb 19, 2025

Consolidate deploy.sh scripts into a single script for terraform deployments.

  • Add devops/scripts/terraform_deploy.sh to handle terraform deployments with parameterized inputs and environment variables.
  • Modify Makefile to use the new devops/scripts/terraform_deploy.sh script and update references to the renamed devops/scripts/terraform_upgrade_provider.sh.
  • Delete deploy.sh scripts from various template folders:
    • templates/shared_services/admin-vm/terraform/deploy.sh
    • templates/shared_services/airlock_notifier/terraform/deploy.sh
    • templates/shared_services/cyclecloud/terraform/deploy.sh
    • templates/shared_services/sonatype-nexus-vm/terraform/deploy.sh
    • templates/workspace_services/databricks/terraform/deploy.sh
    • templates/workspace_services/gitea/terraform/deploy.sh
    • templates/workspace_services/guacamole/terraform/deploy.sh
    • templates/workspaces/base/terraform/deploy.sh
  • Delete upgrade.sh script from templates/workspaces/base/terraform/upgrade.sh.

Consolidate deploy.sh scripts into a single script for terraform deployments.

* Add `devops/scripts/terraform_deploy.sh` to handle terraform deployments with parameterized inputs and environment variables.
* Modify `Makefile` to use the new `devops/scripts/terraform_deploy.sh` script and update references to the renamed `devops/scripts/terraform_upgrade_provider.sh`.
* Delete `deploy.sh` scripts from various template folders:
  - `templates/shared_services/admin-vm/terraform/deploy.sh`
  - `templates/shared_services/airlock_notifier/terraform/deploy.sh`
  - `templates/shared_services/cyclecloud/terraform/deploy.sh`
  - `templates/shared_services/sonatype-nexus-vm/terraform/deploy.sh`
  - `templates/workspace_services/databricks/terraform/deploy.sh`
  - `templates/workspace_services/gitea/terraform/deploy.sh`
  - `templates/workspace_services/guacamole/terraform/deploy.sh`
  - `templates/workspaces/base/terraform/deploy.sh`
* Delete `upgrade.sh` script from `templates/workspaces/base/terraform/upgrade.sh`.
@marrobi marrobi marked this pull request as draft February 19, 2025 14:24
Copy link

Unit Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 0d56965.

@marrobi marrobi closed this Feb 19, 2025
@marrobi marrobi deleted the marrobi/consolidate-deploy-scripts branch February 19, 2025 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant