-
Notifications
You must be signed in to change notification settings - Fork 4
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
kprobe needs a startup script #10
Comments
@jryburn, are you referring to the "service" file which would allow starting/stopping it as a service using systemctl? I think that is really needed. |
Correct. Most (all?) users that deploy kprobe will want to run it as a service that starts when the server boots up so that it starts collecting flow data without someone having to log into the server and start kprobe. |
240418: @dpajin-kentik @jryburn Is this on the roadmap? If so is there an expected completion date? |
I will have to defer to Dusan on the roadmap question. My thought would be we "fix" this by rolling the kprobe capabilities into universal agent. Not ideal for customers in the interim but they have lived this long without a startup script. |
Charter asked about this today when installing kprobe on their DNS servers. "Ryan – do you have a recommended systemd or init script so these daemons restart on a reboot ?" |
One thing though - and apologies for invading this thread - I've recently read feedback from a customer complaining that they lost a ton of visibility because |
I created a PR with service file that is waiting to be merged. #41 I understand that there might be need for service file for DNS mode. |
I approved the PR that adds the service file to the package. Another way we could address the DNS mode (OTT enrichment) is to just have a note in the README file or a comment in the |
I think we could largely base a startup script on what we have with kproxy.
https://packagecloud.io/kentik/kproxy/packages/debian/wheezy/kproxy-latest-debian-7_7.17_amd64.deb
Just rename the files to kprobe and make sure the options are correct.
The text was updated successfully, but these errors were encountered: