Skip to content

Refactor component to use projectsyn/jsonnet-libs's alerts.libsonnet - #57

Merged
DebakelOrakel merged 1 commit into
masterfrom
change/alert-patching
Oct 1, 2026
Merged

DebakelOrakel merged 1 commit into
masterfrom
change/alert-patching

Conversation

@DebakelOrakel

@DebakelOrakel DebakelOrakel commented Sep 11, 2026 •

Copy link
Copy Markdown
Contributor

We've moved the core logic of openshift4-monitoring-alert-patching.libsonnet and prom.generateRules() to projectsyn/jsonnet-libs (cf. projectsyn/jsonnet-libs#13).

This commit updates the compnent to use that generic library.

Checklist

  • The PR has a meaningful title. It will be used to auto-generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • Update the documentation.
  • Categorize the PR by adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog.
  • Link this PR to related issues or PRs.

@github-actions

github-actions Bot commented Sep 11, 2026 •

Copy link
Copy Markdown

🚀 This PR has been released as v2.1.0

Triggering workflows Release

🛠️ Auto tagging enabled with label bump:minor

@github-actions

Copy link
Copy Markdown

🚀 Merging this PR will release v2.1.0

Merging will trigger workflows Release

🛠️ Auto tagging enabled with label bump:minor

@DebakelOrakel
DebakelOrakel requested a review from a team September 11, 2026 09:13
},
};

local registryAlerts = prom.generateRules('registry-cache', params.rules) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please keep this, this works for both components.

We've moved the core logic of openshift4-monitoring-alert-patching.libsonnet and prom.generateRules() to projectsyn/jsonnet-libs (cf. projectsyn/jsonnet-libs#13).

This commit updates the compnent to use that generic library.
@DebakelOrakel
DebakelOrakel force-pushed the change/alert-patching branch from ed3c563 to a0993f3 Compare October 1, 2026 06:36
@DebakelOrakel
DebakelOrakel merged commit 2fe798f into master Oct 1, 2026
7 checks passed
@DebakelOrakel
DebakelOrakel deleted the change/alert-patching branch October 1, 2026 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants