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-cpp for Prometheus Metrics #67

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

stateisbad
Copy link

linuxserver.io


  • I have read the contributing guideline and understand that I have made the correct modifications

Description:

This PR adds support for Prometheus metrics by adding the prometheus-cpp library, and enabling the Prometheus Luanti build flag. Build steps were adapted from the steps outlined on the official wiki.

Benefits of this PR and context:

Some users may want the ability to gather server metrics beyond CPU/memory. For instance, creating Grafana dashboards with Luanti-specific data.

How Has This Been Tested?

This has been tested by building the images and running them with a backup of an existing world and database.

Source / References:

https://wiki.minetest.net/Server_Metrics_using_Prometheus
https://github.com/jupp0r/prometheus-cpp

@LinuxServer-CI
Copy link
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/minetest/5.10.0-pkg-9b23c729-dev-3a501d917adb00914e8be40b222252c421f91268-pr-67/index.html
https://ci-tests.linuxserver.io/lspipepr/minetest/5.10.0-pkg-9b23c729-dev-3a501d917adb00914e8be40b222252c421f91268-pr-67/shellcheck-result.xml

Tag Passed
amd64-5.10.0-pkg-9b23c729-dev-3a501d917adb00914e8be40b222252c421f91268-pr-67
arm64v8-5.10.0-pkg-9b23c729-dev-3a501d917adb00914e8be40b222252c421f91268-pr-67

@stateisbad
Copy link
Author

Is there anything else I should or need to do for this to get reviewed and merged? Thanks!

@thespad thespad self-assigned this Dec 1, 2024
@thespad
Copy link
Member

thespad commented Dec 1, 2024

Please add a changelog entry to the readme-vars and then I'm happy to merge.

@stateisbad
Copy link
Author

stateisbad commented Dec 2, 2024

Please add a changelog entry to the readme-vars and then I'm happy to merge.

Oh nice catch! Done.

@LinuxServer-CI
Copy link
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/minetest/5.10.0-pkg-fe65d7ef-dev-88fdd188753c8bdbc051563892fc35667c8edf83-pr-67/index.html
https://ci-tests.linuxserver.io/lspipepr/minetest/5.10.0-pkg-fe65d7ef-dev-88fdd188753c8bdbc051563892fc35667c8edf83-pr-67/shellcheck-result.xml

Tag Passed
amd64-5.10.0-pkg-fe65d7ef-dev-88fdd188753c8bdbc051563892fc35667c8edf83-pr-67
arm64v8-5.10.0-pkg-fe65d7ef-dev-88fdd188753c8bdbc051563892fc35667c8edf83-pr-67

@thespad thespad merged commit 5dd693c into linuxserver:master Dec 2, 2024
2 checks passed
@stateisbad stateisbad deleted the add-prometheus-cpp branch December 2, 2024 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants