Skip to content

Add set of basic operations alerts - #17

Merged
haasad merged 3 commits into
masterfrom
feat/alerts
Nov 2, 2026
Merged

haasad merged 3 commits into
masterfrom
feat/alerts

Conversation

@haasad

@haasad haasad commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

This PR adds the following alerts:

  • ExternalSecretsWebhookErrors
  • ExternalSecretsReconcileErrorRateHigh
  • ExternalSecretsProviderAPIErrorRateHigh
  • ExternalSecretNotReady

It also adds the capability to define additional alerts and modify/patch the alerts bundled with the component.

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.

@haasad haasad added enhancement New feature or request bump:minor labels Oct 23, 2026
@github-actions

github-actions Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

🚀 This PR has been released as v2.3.0

Triggering workflows Release

🛠️ Auto tagging enabled with label bump:minor

@haasad
haasad requested a review from a team October 23, 2026 15:10

@HappyTetrahedron HappyTetrahedron left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM, queries look sensible.

@haasad
haasad requested a review from simu October 30, 2026 10:37

@simu simu left a comment

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.

Inline suggestion to use more of the utilities provided by alertpatching.libsonnet via https://github.com/projectsyn/jsonnet-libs/blob/main/alerts.libsonnet

Comment thread component/alerts.jsonnet Resolved
Comment thread component/alerts.jsonnet Resolved
Co-authored-by: Simon Gerber <simon.gerber@vshn.ch>
@haasad
haasad merged commit 6306a1d into master Nov 2, 2026
9 checks passed
@haasad haasad mentioned this pull request Nov 2, 2026
4 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bump:minor enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants