Skip to content

Allow managing certificates and alerts from the component#193

Merged
bastjan merged 3 commits intomasterfrom
allow-manage-certificates
Feb 2, 2026
Merged

Allow managing certificates and alerts from the component#193
bastjan merged 3 commits intomasterfrom
allow-manage-certificates

Conversation

@bastjan
Copy link
Copy Markdown
Member

@bastjan bastjan commented Jan 30, 2026

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.

@bastjan bastjan added enhancement New feature or request bump:minor labels Jan 30, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 30, 2026

🚀 This PR has been released as v5.1.0

Triggering workflows Release

🛠️ Auto tagging enabled with label bump:minor

@bastjan bastjan requested a review from a team January 30, 2026 18:28
@bastjan bastjan changed the title Allow creating certificates from the component Allow managing certificates and alerts from the component Jan 30, 2026
@bastjan bastjan force-pushed the allow-manage-certificates branch 3 times, most recently from 2bd9864 to 45e593b Compare January 30, 2026 19:42
@bastjan bastjan force-pushed the allow-manage-certificates branch from 45e593b to 48040cc Compare January 30, 2026 19:42
Comment thread component/main.jsonnet Outdated
Comment thread component/main.jsonnet
@bastjan bastjan force-pushed the allow-manage-certificates branch from 2acbd45 to cb959a0 Compare February 2, 2026 08:45
Copy link
Copy Markdown
Member Author

@bastjan bastjan left a comment

Choose a reason for hiding this comment

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

@DebakelOrakel Implemented the namespacing but did forgo the alert patching since it's all user custom alerts - no upstream ones - and I don't think it warrants the complexity at this point in time. Custom alerts should be setup correctly by the users anyways.

Comment thread component/main.jsonnet Outdated
@bastjan bastjan requested a review from DebakelOrakel February 2, 2026 08:52
@bastjan bastjan merged commit 7255f12 into master Feb 2, 2026
11 checks passed
@bastjan bastjan deleted the allow-manage-certificates branch February 2, 2026 09:55
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.

2 participants