[pve-devel] [PATCH 1/2] Simplify image/template upload check.

Wolfgang Link w.link at proxmox.com
Mon May 14 09:17:17 CEST 2018


It is also true for GlusterFS
See
'Gluster' => {
                                    'content' => {
                                                   'images' => 1,
                                                   'iso' => 1
                                                 },
                                    'volume' => 'gv0',
                                    'shared' => 1,
                                    'path' => '/mnt/pve/Gluster',
                                    'type' => 'glusterfs',
                                    'server' => '127.0.0.1'


> Dietmar Maurer <dietmar at proxmox.com> hat am 14. Mai 2018 um 08:48 geschrieben:
> 
> 
> AFAIK this is not really true with glusterfs?
> 
> > On May 14, 2018 at 7:23 AM Wolfgang Link <w.link at proxmox.com> wrote:
> > 
> > 
> > All storage's witch are able to store images must have a path.
> > ---
> >  PVE/API2/Storage/Status.pm | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/PVE/API2/Storage/Status.pm b/PVE/API2/Storage/Status.pm
> > index ab07146..afb8dcc 100644
> > --- a/PVE/API2/Storage/Status.pm
> > +++ b/PVE/API2/Storage/Status.pm
> > @@ -335,7 +335,7 @@ __PACKAGE__->register_method ({
> >  	my $scfg = PVE::Storage::storage_check_enabled($cfg, $param->{storage},
> > $node);
> >  
> >  	die "cant upload to storage type '$scfg->{type}'\n" 
> > -	    if !($scfg->{type} eq 'dir' || $scfg->{type} eq 'nfs' || $scfg->{type}
> > eq 'glusterfs');
> > +	    if !defined($scfg->{path});
> >  
> >  	my $content = $param->{content};
> >  
> > -- 
> > 2.11.0
> > 
> > 
> > _______________________________________________
> > pve-devel mailing list
> > pve-devel at pve.proxmox.com
> > https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel



More information about the pve-devel mailing list