Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds audit state in Alert #461

Merged
merged 1 commit into from
Jun 21, 2023
Merged

Adds audit state in Alert #461

merged 1 commit into from
Jun 21, 2023

Conversation

eirsep
Copy link
Member

@eirsep eirsep commented Jun 21, 2023

Description

Adds audit state in Alert

Alerts are created in audit state when they are generated by delegate monitors of a workflow since chained alerts can be configured and acknowledged, the underlying monitors' alerts are simply for evaluating chained alert triggers and auditing purpose. Audit state alerts will be created in the history index and do not need to be acknowledged by users.

@codecov
Copy link

codecov bot commented Jun 21, 2023

Codecov Report

Merging #461 (050289b) into main (fcb853e) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               main     #461   +/-   ##
=========================================
  Coverage     73.80%   73.81%           
  Complexity      824      824           
=========================================
  Files           127      127           
  Lines          5353     5354    +1     
  Branches        675      675           
=========================================
+ Hits           3951     3952    +1     
  Misses         1104     1104           
  Partials        298      298           
Impacted Files Coverage Δ
...lin/org/opensearch/commons/alerting/model/Alert.kt 75.45% <100.00%> (+0.08%) ⬆️

Signed-off-by: Surya Sashank Nistala <[email protected]>
@eirsep eirsep merged commit f1c0f1d into opensearch-project:main Jun 21, 2023
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jun 21, 2023
Signed-off-by: Surya Sashank Nistala <[email protected]>
(cherry picked from commit f1c0f1d)
eirsep added a commit that referenced this pull request Jun 21, 2023
Signed-off-by: Surya Sashank Nistala <[email protected]>
(cherry picked from commit f1c0f1d)

Co-authored-by: Surya Sashank Nistala <[email protected]>
AWSHurneyt pushed a commit to AWSHurneyt/common-utils that referenced this pull request Apr 12, 2024
…t#462)

Signed-off-by: Surya Sashank Nistala <[email protected]>
(cherry picked from commit f1c0f1d)

Co-authored-by: Surya Sashank Nistala <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants