Skip to content

converter: support more PrometheusRule properties conversion#1487

Merged
AndrewChubatiuk merged 1 commit intomasterfrom
more-prom-rules-properties-support
Jul 26, 2025
Merged

converter: support more PrometheusRule properties conversion#1487
AndrewChubatiuk merged 1 commit intomasterfrom
more-prom-rules-properties-support

Conversation

@AndrewChubatiuk
Copy link
Contributor

@AndrewChubatiuk AndrewChubatiuk commented Jul 26, 2025

fixes #1485
added support given below PrometheusRule properties:

  • spec.limit
  • spec.query_offset -> spec.eval_offset
  • spec.labels
  • spec.group[*].keep_firing_for

added tests for PrometheusRule and rewrote all converted tests using f-tests pattern

…c.group[*].keep_firing_for PrometheusRule properties conversion
@AndrewChubatiuk AndrewChubatiuk force-pushed the more-prom-rules-properties-support branch from 688eba0 to ca163e7 Compare July 26, 2025 07:45
Copy link
Collaborator

@f41gh7 f41gh7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AndrewChubatiuk AndrewChubatiuk merged commit 8272559 into master Jul 26, 2025
6 checks passed
@AndrewChubatiuk AndrewChubatiuk deleted the more-prom-rules-properties-support branch July 26, 2025 09:05
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.

PrometheusRule conversion omits keep_firing_for from rule definitions

2 participants