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

deploy Terraform modules without wrapping scripts #415

Merged
merged 10 commits into from
Sep 17, 2021

Conversation

glennmusa
Copy link
Contributor

@glennmusa glennmusa commented Sep 16, 2021

Description

This change removes the BASH scripts from src/scripts and updates the Terraform modules to deploy from the command-line with native Terraform CLI commands without any modifications to source.

There are extensive docs updates to this changeset, so please do tear my grammar and instructions apart where necessary!

To demo this, follow the instructions at src/terraform/README.md:

  1. Deploy the root module at src/terraform/mlz/main.tf with just terraform apply
  2. Make note of the outputs so that you can...
  3. Deploy the new workload module at src/terraform/tier3/main.tf with just terraform apply
  4. Clean up the new workload module at src/terraform/tier3/main.tf with just terraform destroy
  5. Clean up the root module at src/terraform/mlz/main.tf with just terraform destroy

Issue reference

The issue this PR closes #409

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles or validates correctly
  • All tests pass (manual and automated)
  • The documentation is updated to cover any new or changed features
  • Markdown files have been linted using the recommended linter. (See .vscode/extensions.json.)
  • Relevant issues are linked to this PR

@glennmusa glennmusa merged commit c308deb into bicep Sep 17, 2021
@glennmusa glennmusa deleted the glenn/terraformWithoutScripts branch September 17, 2021 17:57
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.

Deploy Terraform without MLZ-specific scripts
2 participants