[pdm-devel] [PATCH datacenter-manager 0/2] support eligibility checking in the cli

Maximiliano Sandoval m.sandoval at proxmox.com
Tue Dec 16 09:57:33 CET 2025


"Lukas Wagner" <l.wagner at proxmox.com> writes:

> On Mon Dec 15, 2025 at 5:21 PM CET, Shannon Sterz wrote:
>> these two patches add commands for query and updating the enterprise
>> support eligibility status of pdm itself as well as updating it. it also
>> includes a patch to add such a command to the system report. for a
>> reasoning on why this naming was chosen see commit 1.
>>
>> Shannon Sterz (2):
>>   cli: admin: add `support-eligibility` subcommand
>>   server: add the new `support-eligibility` subcommand to the report
>>
>
> Code looks good from what I can tell.
>
> Maybe we should go with a simpler (albeit, more generic)
> `support-status` command? 'eligibilty' could be a bit hard to
> type/remember for some.
>
> Reviewed-by: Lukas Wagner <l.wagner at proxmox.com>

Looks good to me.

I do agree that it might be a bit harder to type, but I guess one can
use auto-complete too.

Tested-by: Maximiliano Sandoval <m.sandoval at proxmox.com>

-- 
Maximiliano




More information about the pdm-devel mailing list