generated from cloudoperators/repository-template
-
Notifications
You must be signed in to change notification settings - Fork 3
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
[opentelemetry] adding logs label, fixing ceph parsing #478
Closed
+5,145
−153
Conversation
This file contains 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
Contributor
Kuckkuck
commented
Nov 4, 2024
- Fixed ceph log line parsing for resource.attributes["app.label.component"] == "cephobjectstores.ceph.rook.io"
- added label for log.type
- increase memory limit
- more Kubernetes labels from the Kubernetes metadata service
4 tasks
viennaa
reviewed
Nov 5, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small nit from my side.
* [opentelemetry] add alerting to plugin * [opentelemetry] add nl * Automatic application of license header * [opentelemetry] rm trailing spaces * [opentelemetry] add check for montioring crd * [opentelemetry] add playbooks, template labels and additioanlRuleLabels * [opentelemetry] add checking for APIversion * Update .github/licenserc.yaml Co-authored-by: Richard Tief <[email protected]> * Update opentelemetry/chart/values.yaml Co-authored-by: Richard Tief <[email protected]> * Update opentelemetry/chart/values.yaml Co-authored-by: Richard Tief <[email protected]> * Update opentelemetry/chart/templates/_helpers.tpl Co-authored-by: Richard Tief <[email protected]> * [opentelemetry] incorporate changes from review --------- Co-authored-by: License Bot <[email protected]> Co-authored-by: Richard Tief <[email protected]>
* docs(kube-monitoring): add more context and a quick start guide * docs(kube-monitoring): highlight proper names, imporved wording * docs(kube-monitoring): add global options * docs(kube-monitoring): add extention guide, contribution note and disclaimer * docs(kube-monitoring): source and type added * docs(kube-monitoring): add cluster onboarding link * chore(kube-monitoring): Apply suggestions from code review Co-authored-by: Akshay Iyyadurai Balasundaram <[email protected]> * docs(kube-monitoring): add short link refs * chore(licenserc): ignore kube-monitoring folder * docs(kube-monitoring): add arch drawing * docs(kube-monitoring): move architecture drawing and remove section --------- Co-authored-by: Akshay Iyyadurai Balasundaram <[email protected]>
* docs(plutono): add more context * docs(plutono): Apply suggestions from code review Co-authored-by: Akshay Iyyadurai Balasundaram <[email protected]> * docs(plutono): add architecture drawing * chore(licenserc): ignore all plutono files * docs(plutono): add disclaimer --------- Co-authored-by: Akshay Iyyadurai Balasundaram <[email protected]>
* chore(deps): update helm release kube-prometheus-stack to v65.4.0 * chore(deps): Update alerts plugindefinition.yaml * chore(deps): Update kube-monitoring plugindefinition.yaml --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Richard Tief <[email protected]>
There are no UI components in here anymore.
* chore(deps): update helm release kubernetes-operations to v1.1.1 * chore(deps): Update kube-monitoring plugindefinition.yaml --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Richard Tief <[email protected]>
* docs(alerts): add more context and quick start guide * docs(alerts): add arch drawing * chore(licenserc): ignore all alerts files * docs(alerts): refresh drawing * docs(alerts): Apply suggestions from code review Co-authored-by: Tommy Sauer <[email protected]> * docs(alerts): rewrite prometheus-operator part Co-authored-by: Tommy Sauer <[email protected]> * docs(alerts): prometheus-operator dependency Co-authored-by: Tommy Sauer <[email protected]> * docs(alerts): remove Architecture section * docs(alerts): Apply suggestions from code review Co-authored-by: Tommy Sauer <[email protected]> * docs(alerts): use absolute urls to render the images for websites --------- Co-authored-by: Tommy Sauer <[email protected]>
* docs(plutono|kube-monitoring): use absolute image urls * docs(alerts): fix image url
* docs(thanos): add more context and quick start guide * docs(thanos): img formatting * docs(thanos): Apply suggestions from review * chore(licenserc): ignore thanos dir * docs(thanos): remove intro paragraph Should go to a dedicated document describing overarching concepts. * docs(thanos): refresh arch drawings * docs(thanos): update arch drawing * docs(thanos): add component description and interplay + Thanos Compact volume size recommendation * docs(thanos): move arch drawing to the top * docs(thanos): Apply suggestions from code review Co-authored-by: Tommy Sauer <[email protected]> * docs(thanos): use absolute urls for images To show up on the website correctly * docs(thanos): to the point Co-authored-by: Tommy Sauer <[email protected]> * docs(thanos): image url to main branch --------- Co-authored-by: Tommy Sauer <[email protected]>
* docs(opentelemetry): add more context * docs(opentelemetry) add more information about the receivers, add steps, restructure * docs(opentelemetry) rm typo --------- Co-authored-by: I504010 <[email protected]>
* chore(deps): run helm dep up for kube-monitoring and alerts * chore(alerts|kube-monitoring): update plugindefinitions
* github-guard: init * helm-release: add github guard * Update github-guard/plugindefinition.yaml Co-authored-by: David Gogl <[email protected]> * Update github-guard/charts/templates/manager-rbac.yaml Co-authored-by: David Gogl <[email protected]> * Update github-guard/charts/templates/deployment.yaml Co-authored-by: David Gogl <[email protected]> * Update github-guard/charts/templates/serviceaccount.yaml Co-authored-by: David Gogl <[email protected]> --------- Co-authored-by: David Gogl <[email protected]>
* feat(renovate) update versions in plugindefinition * rm orphan
revert the depNameTemplate fir lookup in extractVersionTemplate
* chore(kube-moniotring): enable prometheus self alerting * chore(deps): update kubernetes-operations to remove prometheus dashboard * fix(kube-monitoring): disable not needed rule groups * chore(kube-monitoring): Apply suggestions for values.yaml Co-authored-by: Tommy Sauer <[email protected]> * chore(kube-monitoring): remove data source template for prometheus dashboard --------- Co-authored-by: Tommy Sauer <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.