Skip to content

feat: add IsTriggerAlert field to RuntimeAlert struct#593

Merged
slashben merged 1 commit into
mainfrom
feature/add-is-trigger-alert-to-runtime-alert
Jan 18, 2026
Merged

feat: add IsTriggerAlert field to RuntimeAlert struct#593
slashben merged 1 commit into
mainfrom
feature/add-is-trigger-alert-to-runtime-alert

Conversation

@slashben
Copy link
Copy Markdown
Member

Summary

Add IsTriggerAlert boolean field to RuntimeAlert struct to indicate whether the alert is a trigger alert.

Changes

  • Added IsTriggerAlert bool field to RuntimeAlert struct in armotypes/runtimeincidents.go
  • Field uses omitempty for JSON/BSON tags to maintain backward compatibility

Purpose

This field is copied from RuntimeRule.IsTriggerAlert during alert creation, allowing downstream consumers to identify trigger alerts. The field defaults to false when not present.

Backward Compatibility

  • Uses omitempty tags so existing alerts without the field will continue to work
  • No breaking changes to existing APIs

Add IsTriggerAlert boolean field to RuntimeAlert to indicate whether
the alert is a trigger alert. This field is copied from RuntimeRule
during alert creation, allowing downstream consumers to identify
trigger alerts.
Signed-off-by: Ben <ben@armosec.io>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 18, 2026

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


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

@slashben slashben merged commit 3398437 into main Jan 18, 2026
3 checks passed
@slashben slashben deleted the feature/add-is-trigger-alert-to-runtime-alert branch January 18, 2026 06:50
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