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

fix(terraform): Update module names #1062

Closed
wants to merge 2 commits into from
Closed

fix(terraform): Update module names #1062

wants to merge 2 commits into from

Conversation

Ayelet41
Copy link
Contributor

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Your change follows guidelines in the Contributor guide
  • Communicate test infrastructure changes, i.e. API enablement, secrets
  • Appropriate docs were updated (if necessary)

🛠️ Fixes #<issue_number_goes_here>

minherz and others added 2 commits January 10, 2024 15:56
use GTK for Terraform to replace native GKE and ASM provisioning and configuration.
temporary disable output variables.
@minherz
Copy link
Contributor

minherz commented Apr 29, 2024

Thank you for catching up the problem. Indeed, the provisioning of Online Boutique should start after ASM is installed.
I am reviewing the fact that the "main" and "release/0.9" branches are out of sync. The result of this work will be a patch 0.9.4 that should address the current problem with installing 0.9.3 and synchronize the branches.
If the use of gke and asm modules will be proved impossible, the changes to the relative terraform modules will be rolled back.

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