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

Vrms client terraform migration #29

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Tyson-miller
Copy link
Member

@Tyson-miller Tyson-miller commented Nov 13, 2023

Migrating the configuration for vrms-client to the new terraform-incubator directory.

This PR is strictly additive it does not remove any of the old code in the incubator directory.

@ExperimentsInHonesty
Copy link
Member

@erniep278 or @chelseybeck can you review this pr?

Copy link

github-actions bot commented Feb 29, 2024

Terraform plan in terraform-incubator/people-depot/dev

Error: No value for required variable
Error: No value for required variable

  on main.tf line 15:
  15: variable "root_db_password" {

The root module input variable "root_db_password" is not set, and has no
default value. Use a -var or -var-file command line argument to provide a
value for this variable.

Error: No value for required variable

  on main.tf line 20:
  20: variable "app_db_password" {

The root module input variable "app_db_password" is not set, and has no
default value. Use a -var or -var-file command line argument to provide a
value for this variable.

❌ Failed to generate plan in PR Terraform Plan #22

@JackHaeg
Copy link
Member

JackHaeg commented Nov 5, 2024

@angela-lee1 This is the PR I was mentioning on our call, which is linked to: hackforla/VRMS#1497 which you have been assigned to :)

@angela-lee1
Copy link
Member

VRMS issue #1497 has been placed in the icebox.
@JackHaeg @chelseybeck @ale210

@angela-lee1
Copy link
Member

I was unable to make another commit to refresh the PR due to branch protection. Could you disable branch protection temporarily? Could we close and reopen this pr to trigger the GitHub Action? If needed, I can also create a temporary PR.
@JackHaeg @chelseybeck @ale210

@ale210 ale210 marked this pull request as draft November 21, 2024 02:27
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.

5 participants