Skip to content

feat: bloom-dev AWS deployment#5580

Merged
avrittrohwer merged 1 commit into
mainfrom
avritt/bloom-dev
Dec 3, 2025
Merged

feat: bloom-dev AWS deployment#5580
avrittrohwer merged 1 commit into
mainfrom
avritt/bloom-dev

Conversation

@avrittrohwer

@avrittrohwer avrittrohwer commented Nov 16, 2025

Copy link
Copy Markdown
Contributor

This PR addresses #5450

  • Addresses the issue in full
  • Addresses only certain aspects of the issue

This PR addresses #5438

  • Addresses the issue in full
  • Addresses only certain aspects of the issue

Description

Bloom-dev AWS deployment initial infra. Contains:

  1. Changes to the bloom-dev-deployer permission set required (already deployed).
  2. A bloom-deployment reusable Open Tofu module that deploys Bloom to an AWS account.
  3. A bloom-dev root Open Tofu module that uses the bloom-deployment module.

The deployment is currently live at:

https://core-dev.bloomhousing.dev/
https://partners.core-dev.bloomhousing.dev/

I manually seeded the database with the staging seed command. I will followup automation of that when I add the deployment automation.

How Can This Be Tested/Reviewed?

  1. From repo root: cd infra/tofu_root_modules/bloom_dev
  2. Run tofu destroy -target=module.bloom_deployment
  3. Run tofu apply
  4. Go to https://d-9067ac8222.awsapps.com/start/#/?tab=accounts and log in to bloom-dev as SystemAdministrator.
  5. Inspect the VPC, RDS, and ECS pages.

Author Checklist:

  • Added QA notes to the issue with applicable URLs
  • Reviewed in a desktop view
  • Reviewed in a mobile view
  • Reviewed considering accessibility
  • Added tests covering the changes
  • Made corresponding changes to the documentation
  • Ran yarn generate:client and/or created a migration when required

Review Process:

  • Read and understand the issue
  • Ensure the author has added QA notes
  • Review the code itself from a style point of view
  • Pull the changes down locally and test that the acceptance criteria is met
  • Either (1) explicitly ask a clarifying question, (2) request changes, or (3) approve the PR, even if there are very small remaining changes, if you don't need to re-review after the updates

@avrittrohwer avrittrohwer added this to the bloom core aws preprod milestone Nov 16, 2025
@avrittrohwer avrittrohwer added the infra A way to track issues that are specific to bloom infrastructural changes label Nov 16, 2025
@netlify

netlify Bot commented Nov 16, 2025

Copy link
Copy Markdown

Deploy Preview for partners-bloom-dev ready!

Name Link
🔨 Latest commit ec87c45
🔍 Latest deploy log https://app.netlify.com/projects/partners-bloom-dev/deploys/692f249e7358d70008202b1a
😎 Deploy Preview https://deploy-preview-5580--partners-bloom-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Nov 16, 2025

Copy link
Copy Markdown

Deploy Preview for bloom-public-seeds ready!

Name Link
🔨 Latest commit 0a3868a
🔍 Latest deploy log https://app.netlify.com/projects/bloom-public-seeds/deploys/692e31b4defd3a000847ae37
😎 Deploy Preview https://deploy-preview-5580--bloom-public-seeds.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Nov 16, 2025

Copy link
Copy Markdown

Deploy Preview for bloom-angelopolis canceled.

Name Link
🔨 Latest commit ec87c45
🔍 Latest deploy log https://app.netlify.com/projects/bloom-angelopolis/deploys/692f249ee96f4e0008239b3f

@netlify

netlify Bot commented Nov 16, 2025

Copy link
Copy Markdown

Deploy Preview for bloom-exygy-dev ready!

Name Link
🔨 Latest commit 0a3868a
🔍 Latest deploy log https://app.netlify.com/projects/bloom-exygy-dev/deploys/692e31b4ce047400075d66ad
😎 Deploy Preview https://deploy-preview-5580--bloom-exygy-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Nov 16, 2025

Copy link
Copy Markdown

Deploy Preview for bloom-flagly canceled.

Name Link
🔨 Latest commit ec87c45
🔍 Latest deploy log https://app.netlify.com/projects/bloom-flagly/deploys/692f249ee96f4e0008239b3b

@netlify

netlify Bot commented Nov 16, 2025

Copy link
Copy Markdown

Deploy Preview for bloom-lakeview ready!

Name Link
🔨 Latest commit 0a3868a
🔍 Latest deploy log https://app.netlify.com/projects/bloom-lakeview/deploys/692e31b4e8eb4a0008749887
😎 Deploy Preview https://deploy-preview-5580--bloom-lakeview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@avrittrohwer avrittrohwer marked this pull request as draft November 16, 2025 02:04
@avrittrohwer avrittrohwer force-pushed the avritt/bloom-dev branch 2 times, most recently from 041c19e to 2203397 Compare November 19, 2025 00:51
@avrittrohwer avrittrohwer marked this pull request as ready for review November 19, 2025 00:51
@avrittrohwer avrittrohwer changed the title feat: bloom-dev AWS config: network, database, and API task feat: bloom-dev AWS deployment Nov 19, 2025
@avrittrohwer avrittrohwer force-pushed the avritt/bloom-dev branch 3 times, most recently from 5b8329d to e7eac30 Compare November 19, 2025 22:04

@nb1701 nb1701 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One critical change, the others are just suggestions.

Comment thread infra/tofu_importable_modules/bloom_deployment/db.tf Outdated
Comment thread infra/tofu_importable_modules/bloom_deployment/ecs.tf Outdated
Comment thread infra/tofu_importable_modules/bloom_deployment/ecs.tf
Comment thread infra/tofu_importable_modules/bloom_deployment/ecs_site_public_task.tf Outdated
Comment thread infra/tofu_importable_modules/bloom_deployment/lb.tf
Comment thread infra/tofu_importable_modules/bloom_deployment/lb.tf
Comment thread infra/tofu_importable_modules/bloom_deployment/ecs_api_task.tf
Comment thread infra/tofu_importable_modules/bloom_deployment/db.tf

@ludtkemorgan ludtkemorgan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love this setup. It was easier to understand than I thought it would be. No issues found, mostly just questions.

Comment thread infra/tofu_importable_modules/bloom_deployment/db.tf Outdated
Comment thread infra/tofu_importable_modules/bloom_deployment/ecs.tf Outdated
Comment thread infra/tofu_importable_modules/bloom_deployment/ecs_site_public_task.tf Outdated
Comment thread infra/tofu_importable_modules/bloom_deployment/ecs_site_public_task.tf Outdated
Comment thread infra/tofu_importable_modules/bloom_deployment/db.tf
Comment thread infra/tofu_importable_modules/bloom_deployment/ecs.tf Outdated
@avrittrohwer avrittrohwer force-pushed the avritt/bloom-dev branch 3 times, most recently from f03e071 to bb37738 Compare December 2, 2025 02:44
@YazeedLoonat

Copy link
Copy Markdown
Collaborator

Hey @avrittrohwer this looks good and I think you're good to merge, but you should double check the gitLeaks hook that is failing on this pr to make sure its not something we need to address

@YazeedLoonat YazeedLoonat left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@YazeedLoonat YazeedLoonat added the ready to merge Should be applied when a PR has been reviewed and approved label Dec 3, 2025
@avrittrohwer

Copy link
Copy Markdown
Contributor Author

Hey @avrittrohwer this looks good and I think you're good to merge, but you should double check the gitLeaks hook that is failing on this pr to make sure its not something we need to address

yeah the gitleak is triggering on the docker image git sha ghcr.io/bloom-housing/bloom/api:gitsha-f642fc1f3f056b9fa53429c4fa81689c5e856e5a"

@avrittrohwer avrittrohwer merged commit 02df977 into main Dec 3, 2025
33 of 34 checks passed
@avrittrohwer avrittrohwer deleted the avritt/bloom-dev branch December 3, 2025 17:39
ludtkemorgan pushed a commit to metrotranscom/doorway that referenced this pull request Dec 16, 2025
ludtkemorgan pushed a commit to metrotranscom/doorway that referenced this pull request Dec 23, 2025
ludtkemorgan pushed a commit to metrotranscom/doorway that referenced this pull request Jan 6, 2026
ludtkemorgan pushed a commit to metrotranscom/doorway that referenced this pull request Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra A way to track issues that are specific to bloom infrastructural changes ready to merge Should be applied when a PR has been reviewed and approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants