Re: [pdm-devel] [PATCH datacenter-manager 4/5] cli: client: add `delete-token` option to delete token from remote

["Shan Shaji" <s.shaji@proxmox.com>]

On Tue Dec 9, 2025 at 3:52 PM CET, Michael Köppl wrote:
> shouldn't this also be implemented for the admin CLI?

Yes, I missed to include that patch. Will send it in v2.Thank you for
pointing it out. 

> On Fri Dec 5, 2025 at 7:04 PM CET, Shan Shaji wrote:
>> Signed-off-by: Shan Shaji <s.shaji@proxmox.com>
>> ---
>>  cli/client/src/remotes.rs | 11 ++++++++---
>>  1 file changed, 8 insertions(+), 3 deletions(-)
>>
>> diff --git a/cli/client/src/remotes.rs b/cli/client/src/remotes.rs
>> index 4145dac..0b4df3c 100644
>> --- a/cli/client/src/remotes.rs
>> +++ b/cli/client/src/remotes.rs
>> @@ -116,12 +116,17 @@ async fn update_remote(id: String, updater: RemoteUpdater) -> Result<(), Error>
>>      input: {
>>          properties: {
>>              id: { schema: REMOTE_ID_SCHEMA },
>> +            "delete-token": {
>> +                type: bool,
>> +                optional: true,
>> +                description: "Remove the API-Token from remote."
>> +            }
>>          }
>>      }
>>  )]
>> -/// Add a new remote.
>> -async fn delete_remote(id: String) -> Result<(), Error> {
>> -    client()?.delete_remote(&id).await?;
>> +/// Delete a remote.
>> +async fn delete_remote(id: String, delete_token: Option<bool>) -> Result<(), Error> {
>> +    client()?.delete_remote(&id, &delete_token).await?;
>>      Ok(())
>>  }
>>  
>
>
>
> _______________________________________________
> pdm-devel mailing list
> pdm-devel@lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel



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