[pve-devel] [PATCH guest-common] GuestHelpers: allow overriding $lockdir in test environments

Dietmar Maurer dietmar at proxmox.com
Fri Jun 9 11:30:34 CEST 2017


No, I don't want to generate multiple lock directories ...
AFAIK pve-storage also create locks there.

> On June 9, 2017 at 10:51 AM Fabian Grünbichler <f.gruenbichler at proxmox.com>
> wrote:
> 
> 
> On Fri, Jun 09, 2017 at 09:55:58AM +0200, Wolfgang Bumiller wrote:
> > ---
> >  PVE/GuestHelpers.pm | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/PVE/GuestHelpers.pm b/PVE/GuestHelpers.pm
> > index d066527..16e27a9 100644
> > --- a/PVE/GuestHelpers.pm
> > +++ b/PVE/GuestHelpers.pm
> > @@ -8,11 +8,12 @@ use PVE::Tools;
> >  # We use a separate lock to block migration while a replication job
> >  # is running.
> >  
> > +our $lockdir = '/var/lock/pve-manager';
> > +
> >  sub guest_migration_lock {
> >      my ($vmid, $timeout, $func, @param) = @_;
> >  
> >      my $lockid = "pve-migrate-$vmid";
> > -    my $lockdir = "/var/lock/pve-manager";
> 
> could we rename this? it's in pve-guest-common, not pve-manager..
> "pve-guests"?
> 
> >  
> >      mkdir $lockdir;
> >  
> > -- 
> > 2.11.0
> > 
> > 
> > _______________________________________________
> > pve-devel mailing list
> > pve-devel at pve.proxmox.com
> > https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 
> _______________________________________________
> 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