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

Added Loki Integration #1011

Merged
merged 36 commits into from
Dec 19, 2024
Merged

Added Loki Integration #1011

merged 36 commits into from
Dec 19, 2024

Conversation

bentonam
Copy link
Collaborator

  • Added Loki Integration
  • Updated Integrations default values schema
  • Added pod logs support to keep labels
  • Added pod logs support for structured metadata

@bentonam bentonam requested review from a team and petewall as code owners December 18, 2024 17:09
Copy link
Collaborator

@petewall petewall left a comment

Choose a reason for hiding this comment

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

This looks great. Can we add an integration test: /charkts/k8s-monitoring/tests/integration/integration-loki to test the loki integration is getting metrics and appropriately processing loki logs?

@petewall
Copy link
Collaborator

The integration-cert-manager test is failing because charts/k8s-monitoring/tests/integration/integration-cert-manager/values.yaml is still using metricsTuning instead of metrics.tuning

…art "feature-prometheus-operator-objects" to 17.0.1 (#1007)

Signed-off-by: Pete Wall <[email protected]>
Co-authored-by: petewall <[email protected]>
github-actions bot and others added 14 commits December 19, 2024 11:44
…art "k8s-monitoring-v1" to 17.0.1 (#1008)

Signed-off-by: Pete Wall <[email protected]>
Co-authored-by: petewall <[email protected]>
* Improved validators for OpenCost prometheus URLs and basic auth

Signed-off-by: Pete Wall <[email protected]>

* Sync the helpers template

Signed-off-by: Pete Wall <[email protected]>

---------

Signed-off-by: Pete Wall <[email protected]>
* Additional Cluster Events Processing

- Extracted default values from Cluster Events using logfmt
- Added static label source=kubernetes-events
- Added labelsToKeep support
- Added structuredMetadata support

* Normalized extraLogProcessingStages

Changed extraProcessingStages to extraLogProcessingStages for consistency across charts

* Additional Cluster Events Processing

- Extracted all systemd unit service discovery labels
- Added static label source=journal
- Added labelsToKeep support
- Added structuredMetadata support
- Determine log level

* Fixed Cluster Events Test and Rebuilt

* Add the ability to use a caFile in tls sections (#1013)

Signed-off-by: Pete Wall <[email protected]>

* Improved validators for OpenCost prometheus URLs and basic auth (#1012)

* Improved validators for OpenCost prometheus URLs and basic auth

Signed-off-by: Pete Wall <[email protected]>

* Sync the helpers template

Signed-off-by: Pete Wall <[email protected]>

---------

Signed-off-by: Pete Wall <[email protected]>

* Additional Cluster Events Processing

- Extracted default values from Cluster Events using logfmt
- Added static label source=kubernetes-events
- Added labelsToKeep support
- Added structuredMetadata support

* Sync'd and Rebuilt

* Scyn'd and Rebuilt

* Resolved Conflicts

* Regenerated Cluster Events Documentation

* Fixed Documentation and Unit Test

---------

Signed-off-by: Pete Wall <[email protected]>
Co-authored-by: Pete Wall <[email protected]>
…at/loki-integration

# Conflicts:
#	charts/k8s-monitoring/Chart.lock
#	charts/k8s-monitoring/charts/k8s-monitoring-feature-annotation-autodiscovery-1.0.0.tgz
#	charts/k8s-monitoring/charts/k8s-monitoring-feature-application-observability-1.0.0.tgz
#	charts/k8s-monitoring/charts/k8s-monitoring-feature-auto-instrumentation-1.0.0.tgz
#	charts/k8s-monitoring/charts/k8s-monitoring-feature-cluster-events-1.0.0.tgz
#	charts/k8s-monitoring/charts/k8s-monitoring-feature-cluster-metrics-1.0.0.tgz
#	charts/k8s-monitoring/charts/k8s-monitoring-feature-integrations-1.0.0.tgz
#	charts/k8s-monitoring/charts/k8s-monitoring-feature-node-logs-1.0.0.tgz
#	charts/k8s-monitoring/charts/k8s-monitoring-feature-pod-logs-1.0.0.tgz
#	charts/k8s-monitoring/charts/k8s-monitoring-feature-profiling-1.0.0.tgz
#	charts/k8s-monitoring/charts/k8s-monitoring-feature-prometheus-operator-objects-1.0.0.tgz
@bentonam bentonam merged commit bf3e3ea into main Dec 19, 2024
36 checks passed
@bentonam bentonam deleted the feat/loki-integration branch December 19, 2024 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants