Skip to content

Export metrics in prometheus format #60181 #18528

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

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Conversation

top4ek
Copy link
Contributor

@top4ek top4ek commented Apr 3, 2025

Ticket

#60181

What are you trying to accomplish?

Add Prometheus metrics endpoints. Some OP instances are running inside closed networks without ability to use appsignal.

What approach did you choose and why?

Add yabeda gems to get metrics by Prometheus when OPENPROJECT_PROMETHEUS_EXPORT environment set to 'true'.

@top4ek top4ek changed the title Add Yabeda integration Export metrics in prometheus format #60181 Apr 3, 2025
Copy link
Member

@oliverguenther oliverguenther left a comment

Choose a reason for hiding this comment

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

Some minor changes on the ENV.

Also, I would like to see some documentation on what is being exported with this defualt configuration. Could you add a draft document at https://github.com/opf/openproject/tree/dev/docs/installation-and-operations/operation for what metrics are being exported?

I will take another look at that and fine to merge from me afterwards

@top4ek
Copy link
Contributor Author

top4ek commented Apr 3, 2025

I would like to see some documentation on what is being exported with this defualt configuration.

English is not my native, so could you check it? And do we need to add some short docs from gems used?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants