Re: [PATCH manager 1/1] cephx: rename 'consumer' to 'client' in output and comments

[Jonas Theisen <j.theisen@proxmox.com>]

On 9/15/26 16:50, Michael Köppl wrote:
> 'Client' is the term more often used in the Proxmox VE documentation and
> 'consumer' has been replaced by 'client' in the Ceph documentation to
> avoid confusion [0]. `pveceph auth status` and the cephx health hints in
> the GUI still used the old 'consumer'. Update output strings and
> comments accordingly. This change is made to conform to the updated
> documentation and the naming change in the cephx migration script.
>
> [0] https://docs.ceph.com/en/latest/dev/cephx/
>
> Signed-off-by: Michael Köppl <m.koeppl@proxmox.com>
> ---
> Sent as a separate patch but meant as a follow-up to [1].
>
> [1] https://lore.proxmox.com/pve-devel/20260915131651.515520-1-m.koeppl@proxmox.com
>
> <snip>

Looks good to me.
Nearly all the remaining occurrences of 'consumer' are
in variable or function names.
There are still two 'consumers' in Services.pm line 938 and 1019
but since these are code comments i don't see that as a blocker.

Consider this:
Reviewed-by: Jonas Theisen <j.theisen@proxmox.com>