Skip to content

[Fix] CES alarm rule v2 resources tag#934

Merged
otc-zuul[bot] merged 1 commit into
develfrom
fix-alarms-v2-resources
May 14, 2026
Merged

[Fix] CES alarm rule v2 resources tag#934
otc-zuul[bot] merged 1 commit into
develfrom
fix-alarms-v2-resources

Conversation

@anton-sidelnikov
Copy link
Copy Markdown
Member

@anton-sidelnikov anton-sidelnikov commented May 13, 2026

What this PR does / why we need it

API wants explicitly to receive empty list if nothing are set

Which issue this PR fixes

Special notes for your reviewer

=== RUN   TestAlarmsCRUD
    alarms_test.go:17: Attempting to create alarm rule
    alarms_test.go:57: Attempting to get alarm rule
    alarms_test.go:66: Attempting to enable alarm rule
    alarms_test.go:73: Attempting to verify alarm rule is enabled
    alarms_test.go:80: Attempting to disable alarm rule
    alarms_test.go:50: Attempting to delete alarm rule
--- PASS: TestAlarmsCRUD (2.12s)
=== RUN   TestAlarmsCreateEventSysWithEmptyResources
    alarms_test.go:94: Attempting to create EVENT.SYS alarm rule with empty resources
    alarms_test.go:128: Attempting to verify EVENT.SYS alarm rule
    alarms_test.go:121: Attempting to delete EVENT.SYS alarm rule
--- PASS: TestAlarmsCreateEventSysWithEmptyResources (1.20s)
PASS

Process finished with the exit code 0

@anton-sidelnikov anton-sidelnikov added the gate Merge PR label May 14, 2026
@otc-zuul otc-zuul Bot merged commit 701926b into devel May 14, 2026
13 checks passed
@otc-zuul otc-zuul Bot deleted the fix-alarms-v2-resources branch May 14, 2026 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working gate Merge PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants