Skip to content

Commit

Permalink
Remove max limit of alerting.rules.maxScheduledPerMinute (elastic#194723
Browse files Browse the repository at this point in the history
)

Resolves: elastic#194626

Co-authored-by: Elastic Machine <[email protected]>
  • Loading branch information
ersin-erdal and elasticmachine authored Oct 3, 2024
1 parent 6827ba4 commit 7d29cb1
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
9 changes: 9 additions & 0 deletions x-pack/plugins/alerting/server/config.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -114,4 +114,13 @@ describe('config validation', () => {
);
});
});

test('maxScheduledPerMinute allows more than 32000', () => {
const config: Record<string, unknown> = {
rules: {
maxScheduledPerMinute: 50000,
},
};
expect(() => configSchema.validate(config)).not.toThrow();
});
});
2 changes: 1 addition & 1 deletion x-pack/plugins/alerting/server/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ const rulesSchema = schema.object({
}),
enforce: schema.boolean({ defaultValue: false }), // if enforce is false, only warnings will be shown
}),
maxScheduledPerMinute: schema.number({ defaultValue: 32000, max: 32000, min: 0 }),
maxScheduledPerMinute: schema.number({ defaultValue: 32000, min: 0 }),
overwriteProducer: schema.maybe(
schema.oneOf([
schema.literal('observability'),
Expand Down

0 comments on commit 7d29cb1

Please sign in to comment.