-
-
Notifications
You must be signed in to change notification settings - Fork 174
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 steps to create systemd services #63
base: master
Are you sure you want to change the base?
Conversation
… Call out the fact that NVMe drives require --cap-add SYS_ADMIN in addition to SYS_RAWIO.
…the frontend breaks. Added more robust frontend display code to handle this case. fixes AnalogJ#53
Hey @nithinphilips I like the idea of adding systemd documentation, but your PR is based on an older version of the doc (I've updated the instructions to use Thanks! |
Sure thing. I've updated the doc. Not sure if I rebased it correctly, but let me know if I need to redo it. |
- [ ] freenas/truenas | ||
- [ ] unraid | ||
- [ ] ESXI | ||
- [ ] Proxmox | ||
- [ ] Synology | ||
- [ ] OMV | ||
- [ ] Amahi | ||
- [ ] Running in a LXC container |
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.
Are these meant to be "tasks" and not just an unsorted list?
``` | ||
sudo cat > /etc/systemd/system/scrutiny-collector.service <<EOF | ||
[Unit] | ||
Description=Scrutiny disk health data collector | ||
|
||
[Service] | ||
Type=idle | ||
ExecStart=/opt/scrutiny/bin/scrutiny-collector-metrics-linux-amd64 run --api-endpoint "http://localhost:8080" | ||
TimeoutStartSec=600 | ||
TimeoutStopSec=600 | ||
EOF | ||
``` | ||
|
||
Configure collector systemd timer: | ||
|
||
``` |
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.
Migt make sense to specify lang
(use ```\sh) and then make it a long list with comments in there like
# Create this file
sudo cat file <EOF
...
EOF
# Create that
sudo cat file <EOF
...
EOF
# Enable timer
systemctl this
systemctl that
systemctl enable scrutiny-collector.timer | ||
systemctl start scrutiny-collector.timer |
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.
could shorten with systemctl enable --now scutiny-collector.timer
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.
Just added a type correction 🙂 systemctl enable --now scrutiny-collector.timer
Over the weekend I created Arch packages for the main server and the collector: https://aur.archlinux.org/packages/scrutiny/ Which includes systemd units:
The server units here are much more restrictive than the ones proposed in this PR, which is important given the collector must be run with root privileges. |
IIRC you do not need I'm not sure if the same functionality is present with the |
Hi, i was planning on adding the same systemd configuration. Can I do something to contribute in this PR? Or should I open my own PR? |
Adding steps to optionally run Scrutiny as a systemd service.
scrutiny
user.root
. Scheduling is done by a systemd timer (every 15 mins, same as it is documented now)