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 Prometheus #23

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Add Prometheus #23

wants to merge 5 commits into from

Conversation

Janni2006
Copy link

This pull request introduces several updates and new features to the opencast-ca-display project, including Go module updates, Prometheus metrics integration, and improved error handling. The most important changes are summarized below:

Go Module Updates:

  • Updated the Go version from 1.20 to 1.21 and added toolchain go1.24.0.
  • Added new dependencies such as github.com/beorn7/perks, github.com/cespare/xxhash/v2, and several Prometheus-related packages.

Prometheus Metrics Integration:

  • Introduced Prometheus metrics support by adding necessary imports and defining new structs and functions for metrics collection. [1] [2]
  • Added a new Metrics section to the Config struct and updated the loadConfig function to handle metrics configuration. [1] [2]
  • Implemented a new setupMetricsRouter function to handle Prometheus metrics endpoint.

Error Handling Improvements:

  • Improved error handling in the setupRouter function by adding checks for potential errors and logging them appropriately.

Configuration File Update:

  • Updated the opencast-ca-display.yml configuration file to include a new metrics section for enabling Prometheus metrics.

@Janni2006 Janni2006 changed the title Prometheus Add Prometheus Feb 20, 2025
@mheyen mheyen self-requested a review February 21, 2025 12:57
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