fix(nextcloud): lower outdated apps prometheus alertrule to info - #784
Merged
Conversation
Signed-off-by: Josef Hofer <me@jhofer.de>
wrenix
changed the title
wrenix
enabled auto-merge (squash)
October 4, 2025 07:07
4 tasks
wrenix
approved these changes
Oct 4, 2025
Contributor
Author
|
thanks ❤️ |
Collaborator
|
No Problem, i do not care over this severinty. If somebody else, we need to make it configurable over the values. |
Contributor
Author
sounds good to me :) |
MichaelSp
pushed a commit
to MichaelSp/nextcloud-helm
that referenced
this pull request
Nov 8, 2025
* fix(nextcloud): lower outdated apps prometheus alertrule to info nextcloud#784 Signed-off-by: Josef Hofer <me@jhofer.de> * feat(nextcloud): allow imagePullSecrets for cronjob nextcloud#781 Signed-off-by: Txomin Itoiz <txomin.itoiz@mazins.fr> * feat(nextcloud): add topologySpreadConstraints support nextcloud#782 Signed-off-by: Txomin Itoiz <txomin.itoiz@mazins.fr> * fix(nextcloud): cronjob not working with no $.Values.podLabels set nextcloud#785 wrong type for value; expected map[string]interface {}; got interface {} i think this issue got introduced with nextcloud#779 Signed-off-by: Josef Hofer <me@jhofer.de> * feat(pvc): add extra labels on pvcs nextcloud#793 Signed-off-by: DrummyFloyd <jonathan.monnet28@gmail.com> * fix(nextcloud): update AppVersion to v32.0.1 nextcloud#795 Signed-off-by: WrenIX <133280015+wrenix@users.noreply.github.com> * chore(deps): bump helm/kind-action from 1.12.0 to 1.13.0 Bumps [helm/kind-action](https://github.com/helm/kind-action) from 1.12.0 to 1.13.0. - [Release notes](https://github.com/helm/kind-action/releases) - [Commits](helm/kind-action@v1.12.0...v1.13.0) --- updated-dependencies: - dependency-name: helm/kind-action dependency-version: 1.13.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * fix(docs): typo in README.md for ipFamilyPolicy nextcloud#796 Signed-off-by: Roger Roger <me@rogerrogert.de> --------- Signed-off-by: Josef Hofer <me@jhofer.de> Signed-off-by: Txomin Itoiz <txomin.itoiz@mazins.fr> Signed-off-by: DrummyFloyd <jonathan.monnet28@gmail.com> Signed-off-by: WrenIX <133280015+wrenix@users.noreply.github.com> Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Roger Roger <me@rogerrogert.de> Co-authored-by: Josef Hofer <me@jhofer.de> Co-authored-by: Mazins <133565773+mazinsfr@users.noreply.github.com> Co-authored-by: DrummyFloyd <jonathan.monnet28@gmail.com> Co-authored-by: WrenIX <133280015+wrenix@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: V Aretakis <vassilis@aretakis.eu> Co-authored-by: Roger Roger <me@rogerrogert.de>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of the change
I lowered the severity from warning to info for the outdated apps prometheus alert rule
Benefits
imo the outdated apps are not relevant enough that it should be a warning, im not sure how most people have the severities set up, but i send alerts for warning and critical, and usually i just leave the apps outdated until i do a nextcloud update, which will update the apps anyway.
Possible drawbacks
Applicable issues
Additional information
This is the only thing bugging me about the default alert rules, so i thought I'd just propose the change and see what you think :) Thanks in advance for looking over it.
Checklist
Chart.yamlaccording to semver.