On Tue 30/04/2024 14:50, Solene Rapenne wrote:
> this diff adds a new check in daily(8) using rcctl ls rogue
> 
> maybe wording could be merged better with the current text
> to avoid repeating rc.conf.local(8) twice.

Just saw that this has been committed. Instead of replying to the commit
message I think it makes sense to have the discussion here. Your commit
does exactly what is intended. Point of attention is that
'rcctl ls rogue' does not play nicely with "meta" scripts from ports.
There are a couple:

mail/grommunio/gromox/pkg/gromox.rc
mail/kopano/core/pkg/kopano.rc
net/netatalk/pkg/netatalk.rc
net/samba/pkg/samba.rc
sysutils/bacula/pkg/bacula.rc
sysutils/cfengine/pkg/cfengine.rc
sysutils/nut/pkg/nut.rc

When such a meta script is used, services started by the child scripts
are seen as rogue. In case of net/samba:

# rcctl ls on
samba
# rcctl ls rogue
nmbd
smbd

Using samba's child scripts instead of its meta script:

# rcctl ls on
nmbd
smbd
# rcctl ls rogue
samba

The only way to get rid of this annoyance is to enable a meta script and
its child scripts, which does not make sense.

# rcctl ls on
nmbd
samba
smbd
# rcctl ls rogue


I'm not sure if there is a good solution for this: getting 'rcctl' to
play nicely with meta scripts seems not trivial, and removing meta
scripts will likely cause some user head scratches. Until there is a
solution, meta script users will receive a daily report with rogue
services.