Bug 277049 - dns/powerdns-recursor rc.d init script
Summary: dns/powerdns-recursor rc.d init script
Status: Open
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-02-14 10:10 UTC by Jordan Ostreff
Modified: 2024-02-16 08:21 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (tremere)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jordan Ostreff 2024-02-14 10:10:21 UTC
1. The script doesn't check for existence of recursor.yml and only wants recursor.conf
2. The script doesn't care for existence of 
recursor:
  chroot: 
in new yml syntax or the equivalent into old .conf syntax. This changes pidfile= line ...
Comment 1 Ralf van der Enden 2024-02-14 12:43:11 UTC
(In reply to Jordan Ostreff from comment #0)
Hi Jordan,

I'm open to suggestions (or patch to the rc.d script). Right now I check for recursor.conf via required_files. If I add the yaml file to that, you'd need to have both present for the script to run. Been looking for a way to check for either of the files in other rc.d scripts, but so far no luck yet.

As for the pidfile when you run it in a jail... I have no experience with that myself and peeking inside a config file seems rather complex.

Both the configfile it looks for as the pidfile location and name can be set via rc.conf variables.

I might change the recursor.conf to recursor.yml in the near future to give people time to transition.
Comment 2 Jordan Ostreff 2024-02-15 10:35:46 UTC
Hi,

I agree that using rc.conf file it is possible to change pdns_recursor_conf variable to point to recursor.yml but the important part is with 
pidfile="/var/pdns/var/run/${name}.pid"

what about to implement pdns_recursor_jaildir variable and use it like pidfile="${pdns_recursor_jaildir}/var/run/${name}.pid"?