Skip to content

fix(planner): prioritize configured metric schemas - #731

Merged
milindsrivastava1997 merged 1 commit into
mainfrom
fix/planner-hint-first-schema
Sep 22, 2026
Merged

fix(planner): prioritize configured metric schemas#731
milindsrivastava1997 merged 1 commit into
mainfrom
fix/planner-hint-first-schema

Conversation

@milindsrivastava1997

@milindsrivastava1997 milindsrivastava1997 commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Treat configured metrics label hints as authoritative during PromQL controller setup.
  • Discover labels only for workload metrics without a configured hint.
  • Preserve fatal discovery errors for unhinted metrics.

Testing

  • cargo test -p asap_planner

@milindsrivastava1997
milindsrivastava1997 marked this pull request as ready for review September 22, 2026 23:59
@milindsrivastava1997
milindsrivastava1997 merged commit 633fbeb into main Sep 22, 2026
13 checks passed
@milindsrivastava1997
milindsrivastava1997 deleted the fix/planner-hint-first-schema branch September 23, 2026 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@milindsrivastava1997