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