Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add component parameter for configuring additional Prometheus rules #127

Merged
merged 1 commit into from
Jun 30, 2023

Conversation

simu
Copy link
Member

@simu simu commented Jun 30, 2023

This makes use of the new openshift4-monitoring library function introduced in appuio/component-openshift4-monitoring#162

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.

This makes use of the new openshift4-monitoring library function
introduced in appuio/component-openshift4-monitoring#162
@simu simu added the enhancement New feature or request label Jun 30, 2023
Comment on lines +97 to +99
std.trace(
'Prometheus object helper library not available on non-OCP4, additional rules may be configured incorrectly',
{
Copy link
Member Author

Choose a reason for hiding this comment

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

This is ugly, but necessary since the component nominally supports non-OCP4

@simu simu requested a review from a team June 30, 2023 12:51
@simu simu merged commit 2c46612 into master Jun 30, 2023
8 checks passed
@simu simu deleted the feat/configurable-custom-rules branch June 30, 2023 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants