[pve-devel] [PATCH v4 qemu-server 2/12] qemuserver: foreach_volid: include pending volumes

Thomas Lamprecht t.lamprecht at proxmox.com
Fri Jun 16 14:37:53 CEST 2023


Am 16/06/2023 um 14:25 schrieb Fiona Ebner:
>> @@ -4902,6 +4904,11 @@ sub foreach_volid {
>>      };
>>  
>>      PVE::QemuConfig->foreach_volume_full($conf, $include_opts, $test_volid);
>> +
>> +    if ($include_pending && defined($conf->{pending}) && $conf->{pending}->%*) {
>> +	PVE::QemuConfig->foreach_volume_full($conf->{pending}, $include_opts, $test_volid, undef, 1);
>> +    }
>
> Style nit: wouldn't be 101 chars as a post-if 😉

OTOH, the check is a bit complex and for complex ones the "normal" if block
often feels a bit easier to read, but not really hard feelings either way ^^





More information about the pve-devel mailing list