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
prometheus-node-exporter: initial addon #5549
Conversation
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.
tx for PR, just initial cleanup
didn't tested the changes but should work
packages/addons/service/prometheus-node-exporter/source/bin/prometheus-node-exporter.start
Outdated
Show resolved
Hide resolved
packages/addons/service/prometheus-node-exporter/source/default.py
Outdated
Show resolved
Hide resolved
packages/addons/service/prometheus-node-exporter/source/settings-default.xml
Outdated
Show resolved
Hide resolved
packages/addons/service/prometheus-node-exporter/source/resources/settings.xml
Outdated
Show resolved
Hide resolved
...ns/service/prometheus-node-exporter/source/system.d/service.prometheus-node-exporter.service
Outdated
Show resolved
Hide resolved
Thx for review ! |
@Celedhrim 3x Icon suggestions attached. |
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.
I had no time yet to test it, but just found some additional stuff :)
@heitbaum could you have a look about the go compile stuff ?
packages/addons/service/prometheus-node-exporter/source/bin/prometheus-node-exporter.start
Outdated
Show resolved
Hide resolved
packages/addons/service/prometheus-node-exporter/source/settings-default.xml
Outdated
Show resolved
Hide resolved
...ns/service/prometheus-node-exporter/source/system.d/service.prometheus-node-exporter.service
Outdated
Show resolved
Hide resolved
Yep. Just finished work. Will have a look after dinner ✅ |
Many thx ! I'll take the first one ! :D |
...ns/service/prometheus-node-exporter/source/system.d/service.prometheus-node-exporter.service
Outdated
Show resolved
Hide resolved
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.
LGTM without having tested anything.
Icon is not very good , but :
https://forum.libreelec.tv/thread/24362-addon-icons/
Addon tested on pi4 without issue.