Skip to content

Support configuration of event rules for generic sources#471

Merged
nilmerg merged 2 commits into
mainfrom
configure-rules-without-integration
Jun 3, 2026
Merged

Support configuration of event rules for generic sources#471
nilmerg merged 2 commits into
mainfrom
configure-rules-without-integration

Conversation

@BastianLedererIcinga

@BastianLedererIcinga BastianLedererIcinga commented May 7, 2026

Copy link
Copy Markdown
Contributor

For event rules of generic sources a simple SearchEditor without enrichment, validation and suggestions is created.

resolve #450
require Icinga/ipl-web#384
require Icinga/ipl-web#358

@cla-bot cla-bot Bot added the cla/signed CLA is signed by all contributors of a PR label May 7, 2026
@BastianLedererIcinga
BastianLedererIcinga force-pushed the configure-rules-without-integration branch from 457f0fa to 889baa6 Compare May 7, 2026 11:59
@BastianLedererIcinga BastianLedererIcinga linked an issue May 7, 2026 that may be closed by this pull request
@BastianLedererIcinga
BastianLedererIcinga force-pushed the rule-filter-handling branch 2 times, most recently from 015bfb8 to d8a98b8 Compare May 19, 2026 07:46
@BastianLedererIcinga
BastianLedererIcinga force-pushed the rule-filter-handling branch 2 times, most recently from 1fb39c8 to a4e2aa2 Compare May 22, 2026 14:02
Base automatically changed from rule-filter-handling to main May 22, 2026 14:15
@BastianLedererIcinga
BastianLedererIcinga force-pushed the configure-rules-without-integration branch 3 times, most recently from 52667d2 to d57ae09 Compare May 26, 2026 09:33
Comment thread public/css/form.less Outdated
Comment thread application/controllers/EventRuleController.php Outdated
Comment thread application/controllers/EventRuleController.php Outdated
Comment thread application/controllers/EventRuleController.php Outdated

This comment was marked as resolved.

Comment thread application/controllers/EventRuleController.php Outdated
Comment thread application/controllers/EventRuleController.php Outdated
Comment thread application/controllers/EventRuleController.php
Comment thread application/forms/SourceForm.php Outdated
@BastianLedererIcinga

Copy link
Copy Markdown
Contributor Author

The failing tests are unrelated to this PR, they are caused by the API tests being incompatible with recent changes to the schema.

Comment thread application/controllers/EventRuleController.php
Comment thread application/forms/SourceForm.php
@nilmerg
nilmerg dismissed sukhwinder33445’s stale review June 2, 2026 09:20

all comments were resolved

Comment thread application/forms/SourceForm.php Outdated
Comment thread application/forms/SourceForm.php Outdated
Comment thread application/forms/SourceForm.php Outdated
Comment thread application/forms/SourceForm.php Outdated
Comment thread application/forms/SourceForm.php Outdated
Comment thread application/controllers/EventRuleController.php Outdated
Comment thread application/forms/SourceForm.php Outdated
Comment thread application/forms/SourceForm.php Outdated
Comment thread public/css/form.less Outdated
@BastianLedererIcinga
BastianLedererIcinga force-pushed the configure-rules-without-integration branch from 88d3200 to 0f2929c Compare June 3, 2026 08:34
Comment thread application/forms/SourceForm.php
@nilmerg nilmerg added this to the 0.3.0 milestone Jun 3, 2026
Allow the user to fisrt select `generic` or `integrated`,
only śhow the source identifier field for `integrated` and adjust
the descriptions.
Create a simple `SearchEditor` without suggestions, enrichment
and validation when editing the filter of a genric rule.
@BastianLedererIcinga
BastianLedererIcinga force-pushed the configure-rules-without-integration branch from 920bdf6 to 742d4f5 Compare June 3, 2026 09:54
@nilmerg
nilmerg merged commit b9fe30b into main Jun 3, 2026
13 checks passed
@nilmerg
nilmerg deleted the configure-rules-without-integration branch June 3, 2026 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla/signed CLA is signed by all contributors of a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow to configure event rules without a source integration

4 participants