Skip to content

feat: serve Prometheus label values as a curated command - #191

Merged
ysyneu merged 2 commits into
mainfrom
feat/prometheus-label-values
Sep 14, 2026
Merged

ysyneu merged 2 commits into
mainfrom
feat/prometheus-label-values

Conversation

@ysyneu

@ysyneu ysyneu commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds the curated command flashduty monit prometheus-api-v1-label-{label_name}-values <label_name> --data-source-id <id> on top of the new DataSourcesService.ReadPrometheusLabelValues SDK method. The operation has a path parameter and a non-enveloped response, so it is excluded from generation.

  • The command sits at the operation's path-name under monit, so TestEveryOperationHasPathCommand finds it. curatedOperationIDs lists the operation.
  • The data source's own error text surfaces as a command error.
  • The base commit bumps go-flashduty and regenerates commands and cards.

Tests

  • monit_prometheus_label_values_test.go covers path, method and header, the missing-flag rejection, and a data source error.
  • make test (including TestEveryOperationHasPathCommand and TestGeneratorTargetsFullSpec), make check-cards, make build and make lint pass.
  • Checked against a live Prometheus data source: __name__ and job return real label values.

monit-prometheus-read-label-values carries a path parameter and a
non-enveloped response, so it is excluded from generation. Bump
go-flashduty to the commit that adds the hand-written SDK method, and
attach a curated command at the operation's path-name under the monit
group: a positional label name, a required --data-source-id flag for
the X-DSID header, and standard result rendering.
@ysyneu
ysyneu merged commit 5edf1d7 into main Sep 14, 2026
12 checks passed
@ysyneu
ysyneu deleted the feat/prometheus-label-values branch September 22, 2026 10:03
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