[pve-devel] [qemu-container] Add unused mountpoints.

[Dietmar Maurer]

> On June 13, 2017 at 8:47 AM Wolfgang Link <w.link at proxmox.com> wrote:
> 
> 
> On 06/13/2017 08:44 AM, Dietmar Maurer wrote:
> >> On 06/12/2017 04:01 PM, Dietmar Maurer wrote:
> >>>> +    # add 'unusedX' volumes to volhash
> >>>> +    foreach my $key (keys %$conf) {
> >>>> +	my $value = $conf->{$key};
> >>>> +	if ($key =~ m/^unused/) {
> >>>> +	    my $drive = PVE::QemuServer::parse_drive($key, $value);
> >>>
> >>> AFAIR $value is already a volid, so what is the purpose of this
> >>> parse_drive
> >>> call?
> >>>
> >>
> >> In this String there is normally the size included.
> >> So it is better to parse the string before we use it.
> >
> > Ah, I see. I am still unsure if we should really replicate unused disks...
> >
> What if we sync if replica attribute is presents in the unused string?

Wait. After reading the code, I cannot see that we store attributes for unused 
volumes. Size is also not stored.