Skip to content

Commit

Permalink
Make PublicRuleMonitoringService; not optional
Browse files Browse the repository at this point in the history
  • Loading branch information
jpdjere committed Dec 5, 2022
1 parent fe89ec2 commit c0407b1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion x-pack/plugins/alerting/server/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ export interface RuleExecutorServices<
alertFactory: PublicAlertFactory<State, Context, ActionGroupIds>;
shouldWriteAlerts: () => boolean;
shouldStopExecution: () => boolean;
ruleMonitoringService?: PublicRuleMonitoringService;
ruleMonitoringService: PublicRuleMonitoringService;
}

export interface RuleExecutorOptions<
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -343,6 +343,7 @@ export const createSecurityRuleTypeWrapper: CreateSecurityRuleTypeWrapper =
secondaryTimestamp,
ruleExecutionLogger,
aggregatableTimestampField,
alertTimestampOverride,
},
});

Expand Down

0 comments on commit c0407b1

Please sign in to comment.