On Fri, Jul 04, 2025 at 03:23:14PM +0200, Stefan Hanreich wrote:
> On 7/4/25 15:14, Wolfgang Bumiller wrote:
> 
> [snip]
> 
> >> +
> >> +    /// Class method: Return all FRR daemons that need to be enabled for 
> >> this fabric configuration
> >> +    /// instance.
> > 
> > Method*
> > 
> > Daemons? Or would "services" make more sense (and a `.service` suffix?)
> 
> It's a bit weird with FRR, there's only frr.service and it starts
> daemons based on the config in /etc/frr/daemons - hence why I think
> daemons is more fitting.

Ah okay, didn't realize this was about a config file. (Maybe link the
docs or mention the file in the doc comment?)

> 
> [snip]


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to