Skip to content

Syncing development with main#45

Merged
NickSavino merged 37 commits intodevelopmentfrom
main
Jan 31, 2026
Merged

Syncing development with main#45
NickSavino merged 37 commits intodevelopmentfrom
main

Conversation

@NickSavino
Copy link
Copy Markdown
Contributor

@NickSavino NickSavino commented Jan 31, 2026

Summary by CodeRabbit

  • Chores
    • Improved deployment process with integrated health verification checks that validate application responsiveness after deployment.
    • Refactored deployment execution to use structured command patterns for more reliable infrastructure updates and improved error handling during deployment cycles.
    • Updated infrastructure provider dependencies to current versions for enhanced compatibility and stability across deployment environments.

✏️ Tip: You can customize this high-level summary in your review settings.

NickSavino and others added 30 commits January 27, 2026 18:11
HOTFIX: add backend.tf file to track remote state
Added VPC to allow SSM access to EC2
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 31, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The deployment workflow refactors image URI handling to compute values locally instead of passing through outputs, introduces a JSON-encoded command payload for orchestrating EC2 deployment steps, and adds health verification with polling and retry logic. Terraform provider dependencies are updated in the production environment lock file.

Changes

Cohort / File(s) Summary
GitHub Actions Deployment Workflow
.github/workflows/deploy.yml
Refactored to compute IMAGE_URI and ECR_REGISTRY locally, replaced SSM command invocation with structured JSON COMMANDS payload, added command completion polling with success/failure handling, and introduced health verification flow with repeated health checks and retries.
Terraform Provider Lock File
infra/env/prod/.terraform.lock.hcl
Added local and tls providers (versions 2.6.1 and 4.1.0 respectively), updated random provider from 3.8.0 to 3.8.1 with new hash values.

Sequence Diagram

sequenceDiagram
    participant GHA as GitHub Actions
    participant EC2 as EC2 Instance
    participant App as Application Health Endpoint
    
    GHA->>GHA: Compute IMAGE_URI & ECR_REGISTRY locally
    GHA->>GHA: Construct COMMANDS JSON payload
    GHA->>EC2: Send SSM command with COMMANDS payload
    EC2->>EC2: Execute pull, docker login, container mgmt
    GHA->>GHA: Poll for command completion
    EC2-->>GHA: Command execution status
    GHA->>EC2: Retrieve public IP
    GHA->>App: Health check request
    App-->>GHA: Health status response
    alt Health check succeeds
        GHA->>GHA: Deployment verified
    else Health check fails/retries exhausted
        GHA->>GHA: Deployment failed
    end
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Poem

🐰 The workflow hops with newfound grace,
Computing URIs in their rightful place,
Commands in JSON now neatly bound,
Health checks hop and poll around,
Providers updated, lock files tight—
Deployments dance through day and night! 🚀


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@NickSavino NickSavino merged commit 7197bdc into development Jan 31, 2026
2 of 3 checks passed
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.

2 participants