Skip to content

COBS-1290: Add Insights alert rules OpenAPI - #1225

Open
dmamidala-glitch wants to merge 4 commits into
digitalocean:mainfrom
dmamidala-glitch:cobs-1290-insights-alert-rules
Open

dmamidala-glitch wants to merge 4 commits into
digitalocean:mainfrom
dmamidala-glitch:cobs-1290-insights-alert-rules

Conversation

@dmamidala-glitch

@dmamidala-glitch dmamidala-glitch commented Sep 16, 2026

Copy link
Copy Markdown

Summary

  • Adds public OpenAPI for Insights alert rule CRUD under /v2/insights/alert-rules.
  • Aligned to the Alerts Public Insights API RFC wire contract: lowercase enums (active, greater_than, critical), duration strings (5m, 4h), dotted OTel metrics, links/meta list envelope, page/per_page (default 20, max 200), scopes insights:read|create|update|delete.
  • Documents 422 for underscored Prometheus-style metric names.

Test plan

  • make lint passes (0 Spectral errors)
  • Examples match RFC request/response shapes
  • Confirm api-service public DTO emits RFC wire format (staging currently still returns proto-style enums / pagination object)
  • After merge, confirm docs regen publishes the new endpoints

Blocked by / ships after: COBS-1287 Edge routing.

Related: notification-channels draft #1224 (COBS-1289).

dmamidala-glitch and others added 4 commits September 16, 2026 12:27
Document public CRUD for /v2/insights/alert-rules against the api-service
contract and Edge scopes from COBS-1287.

Co-authored-by: Cursor <cursoragent@cursor.com>
Match staging: protobuf omitempty drops total when zero.

Co-authored-by: Cursor <cursoragent@cursor.com>
Use lowercase enums, duration strings, links/meta list envelope, and
dotted OTel metric guidance from the Alerts Public Insights API RFC.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant