Skip to content

fix: update ALERT_PIPELINE_MODE from legacy to dual in production configuration and add status field to MonitorEventV1#602

Merged
preethamr merged 1 commit intomainnet-prodfrom
fix/reenable_alerts
Mar 19, 2026
Merged

fix: update ALERT_PIPELINE_MODE from legacy to dual in production configuration and add status field to MonitorEventV1#602
preethamr merged 1 commit intomainnet-prodfrom
fix/reenable_alerts

Conversation

@preethamr
Copy link
Contributor

🤖 Linear

Closes CONG-XXX

Copilot AI review requested due to automatic review settings March 19, 2026 02:55
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

This PR updates production monitoring configuration to run the alert pipeline in a migration “dual” mode and extends the emitted monitor event shape to include a status indicator.

Changes:

  • Add an optional status field to MonitorEventV1 and set it to 'firing' when building events.
  • Update prod Terraform configuration to set ALERT_PIPELINE_MODE to "dual".
  • Expand EventEmitterConfig.environment to also accept "production".

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
packages/utils/src/helpers/events.ts Extends the event payload type and emitter config typing; sets status on emitted events.
ops/mainnet/prod/core/config.tf Switches production ALERT_PIPELINE_MODE from legacy to dual in env var definitions.

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

You can also share your feedback on Copilot code review. Take the survey.

@preethamr preethamr force-pushed the fix/reenable_alerts branch 2 times, most recently from b596a01 to e5bfde9 Compare March 19, 2026 03:04
@preethamr preethamr force-pushed the fix/reenable_alerts branch from e5bfde9 to 20c66ae Compare March 19, 2026 03:16
@preethamr preethamr merged commit 56f67b0 into mainnet-prod Mar 19, 2026
42 of 43 checks passed
@preethamr preethamr deleted the fix/reenable_alerts branch March 19, 2026 03:46
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