[pve-devel] applied: [PATCH-SERIES v3 storage/manager] factoring out RE for backup extension

Thomas Lamprecht t.lamprecht at proxmox.com
Wed Mar 16 17:16:40 CET 2022


On 04.11.21 15:40, Dominik Csapak wrote:
> Series LGTM, looked closely at the regexes,
> and they match (semantically), tested around a bit
> and all seemed to work.
> 
> one super small nit (not a blocker IMHO)
> 
> the format/extension/compression handling in 2/2
> could maybe use a comment what we do there
> (though maybe it's just me who's confused ^^)
> 

can you send a follow up with what you have in mind?

> On 10/22/21 14:23, Lorenz Stechauner wrote:
>> changes to v2:
>> * rebased to current master
>> * kept old $vztmpl_extension_re to not break pve-manager and added
>>      # FIXME [...]
>>
>> changes to v1:
>> * also renamed iso/vztmpl REs
>> * new naming schema: $<X>_EXT_RE_<# of capture groups>
>>      for example: $BACKUP_EXT_RE_2
>>
>> pve-storage
>> Lorenz Stechauner (2):
>>    storage: rename REs for iso and vztmpl extensions
>>    storage/plugin: factoring out regex for backup extension re
>>
>>   PVE/API2/Storage/Status.pm |  8 ++++----
>>   PVE/Storage.pm             | 26 +++++++++++++++++---------
>>   PVE/Storage/Plugin.pm      | 12 ++++++------
>>   3 files changed, 27 insertions(+), 19 deletions(-)

applied the storage part, thanks!





More information about the pve-devel mailing list