Skip to content

fix: switch to lh handler#604

Merged
liu-zhipeng merged 2 commits intomainnet-stagingfrom
fix/switch-to-lh-handler
Mar 24, 2026
Merged

fix: switch to lh handler#604
liu-zhipeng merged 2 commits intomainnet-stagingfrom
fix/switch-to-lh-handler

Conversation

@otsybizov
Copy link
Contributor

🤖 Linear

Closes CONG-XXX

Copilot AI review requested due to automatic review settings March 23, 2026 22:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Re-enables the Lighthouse “handler” path in mainnet staging by turning back on the Lighthouse queue Redis + PrivateLink plumbing and deploying the Lighthouse handler ECS service (instead of the prior lambda poller approach).

Changes:

  • Enable lighthouse_handler ECS service in core staging and include it in CI deploy/health-check flow.
  • Provision Lighthouse queue Redis + PrivateLink provider (core) and consumer endpoint (backend) for cross-VPC access.
  • Expose core Terraform outputs needed by backend remote-state to configure Redis connectivity.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
ops/mainnet/staging/core/outputs.tf Re-enables outputs needed for backend to consume Lighthouse queue Redis/PrivateLink details.
ops/mainnet/staging/core/main.tf Turns on lighthouse handler ECS service and Lighthouse queue Redis + PrivateLink provider.
ops/mainnet/staging/core/config.tf Reintroduces lighthouse_handler_env_vars including direct Redis URL for the handler.
ops/mainnet/staging/backend/main.tf Re-enables PrivateLink consumer endpoint to reach core’s Lighthouse queue Redis.
ops/mainnet/staging/backend/config.tf Re-adds REDIS_URL for cartographer-handler via PrivateLink endpoint (with TLS SNI).
.github/workflows/build-test-deploy.yml Adds lighthouse-handler back into ECS service list and ensures handler image build precedes core staging apply.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@liu-zhipeng liu-zhipeng merged commit bc5e780 into mainnet-staging Mar 24, 2026
23 checks passed
@liu-zhipeng liu-zhipeng deleted the fix/switch-to-lh-handler branch March 24, 2026 03:05
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.

3 participants