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

Fix pgadmin for deployment with vpc #378

Merged
merged 1 commit into from
Aug 28, 2024
Merged

Fix pgadmin for deployment with vpc #378

merged 1 commit into from
Aug 28, 2024

Conversation

dkatzz
Copy link
Contributor

@dkatzz dkatzz commented Aug 28, 2024

Description

We were getting an error when running pgadmin:

╷
│ Error: Invalid value for input variable
│ 
│   on main.tf line 169, in module "pgadmin":
│  169:   subnet_ids      = local.vpc_private_subnets
│ 
│ The given value is not suitable for module.pgadmin[0].var.subnet_ids
│ declared at ../../modules/pgadmin/variables.tf:47,1-22: element 0: string
│ required.

By pointing at the ID variable instead, we are successfully able to create pgadmin resources. Note: this doesn't change anything for non-vpc enabled deployments, since _private_subnets and _private_subnet_ids point to the same thing for those

vpc_public_subnets = local.enable_managed_vpc ? module.vpc[0].public_subnets : data.aws_subnets.external_public_subnets

Checklist

General

  • Added the correct label
  • Assigned to a specific person or civiform/deployment-system
  • Performed manual testing (at a minimum run bin/setup without your changes and then bin/deploy with your changes to ensure your changes don't break existing deployments)

Issue(s) this completes

civiform/civiform#8412

@dkatzz dkatzz requested review from a team and gwendolyngoetz and removed request for a team August 28, 2024 15:17
@dkatzz dkatzz added the bug Something isn't working label Aug 28, 2024
@dkatzz dkatzz merged commit a201d18 into main Aug 28, 2024
7 checks passed
@dkatzz dkatzz deleted the dkatz-pgadmin branch August 28, 2024 15:30
jcma-google pushed a commit that referenced this pull request Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants