Re: [pbs-devel] [PATCH proxmox-backup 08/12] api2/admin/datastore: add get/set_protection

[Fabian Ebner <f.ebner@proxmox.com>]

Am 13.09.21 um 10:34 schrieb Dominik Csapak:
> On 9/10/21 14:43, Fabian Ebner wrote:
>> Am 06.09.21 um 12:57 schrieb Dominik Csapak:
>>> for gettin/setting the protected flag for snapshots (akin to notes)
>>>
>>
>> For notes, there is a 'show' command, but not for protected. Is there 
>> an other way to query the current protection of a single snapshot via 
>> proxmox-backup-client?
>>
>> I'm working on bug #3307, which is the same feature for PVE and for 
>> integration with the PBS plugin it would be nicer to query only the 
>> single snapshot. Or should I list the whole group and extract the info 
>> from there instead?
>>
> 
> well, i did not think we would need a 'show' command yet for a single 
> snapshot, but if you'll need it, i can either add it later
> or in a v2 (if the series needs some changes)
> 
> the api already contains the 'get' method anyway
> 

Using the API instead of the client also makes distinguishing between 
"not supported" (i.e. 404 status code) and some other error easy. To do 
the same with proxmox-backup-client, it would be necessary to match the 
error message, but that would require refactoring run_client_cmd (in 
PBSPlugin.pm) AFAICT.

Anyways, during testing I noticed that the GET call does not fail if the 
requested snapshot doesn't even exist.