[BlueOnyx:27283] Re: Easy Migrate errors : Exceptions not handled ...kindof

[Rodrigo Ordoñez Licona Xnet.mx]

THanks for the quick turnaround, 

But the error went just a Little further 

Oct 12 14:38:56 web4 cced(smd)[943817]: client 0:[0:943810]: SET succeeded
Oct 12 14:38:56 web4 cced(smd)[943817]: client 0:[0:943810]: SET  345 . Disk refresh = "" quota = 20 over_quota = "" used = "" lastmailed = 0
Oct 12 14:38:56 web4 cced(smd)[943817]: client 0:[0:943810]: SET 345.Disk failed (-5)


It needs the same a few lines down I think

Regards

Rodrigo O
Xnet


-----Mensaje original-----
De: Michael Stauber <mstauber at blueonyx.it> 
Enviado el: viernes, 11 de octubre de 2024 11:41 a. m.
Para: Rodrigo Ordoñez Licona Xnet.mx <rordonez at xnet.com.mx>; 'BlueOnyx General Mailing List' <blueonyx at mail.blueonyx.it>
CC: programacion at xnet.mx
Asunto: Re: Easy Migrate errors : Exceptions not handled ...kindof

Hi Rodrigo,

> We are migrating from a 5209 to a 5211, running easymigrate on the new 
> server,
> 
> And during the migration a user fails with disk used as "empty"
> 
> Log output of failure:
> 0:handlers/base/disk/userusage.pl: SET  456 . Disk used = ""
> Oct  5 19:07:53 web cced(smd)[105707]: client
> 0:handlers/base/disk/userusage.pl: SET 456.Disk failed (-5)

That's weird.

> However we noticed the easy-migrate.pl  has these lines if ($key eq 
> 'Disk') {
>                      if ($User_NameSpace->{'used'} eq "") {
>                          $User_NameSpace->{'used'} = '0';
>                      }
>                      if ($User_NameSpace->{'over_quota'} eq "") {
>                          $User_NameSpace->{'over_quota'} = '0';
>                      }
>                  }
> 
> But they are not changing  a "" to a '0'

 From the looks of it $User_NameSpace->{'used'} wasn't just empty in your case. The key 'used' probably wasn't even set in first place. Which is kinda unusual.

I looked at the code in the broader context and I think that should fix it:

if ($key eq 'Disk') {
     $User_NameSpace->{'used'} //= '0';
     $User_NameSpace->{'over_quota'} //= '0'; }

if (($key eq 'Sites') && (!defined($User_NameSpace->{'used'}) || $User_NameSpace->{'used'} eq "")) {
     $User_NameSpace->{'used'} = '0';
}

See: https://devel.blueonyx.it/trac/changeset/5418/

I just published an updated Easy-Migrate to the BlueOnyx 5211R Testing YUM repository.

To install it do this:

dnf config-manager --set-enabled BlueOnyx-5211R-Testing dnf clean all dnf update dnf config-manager --set-disabled BlueOnyx-5211R-Testing

There is (currently) nothing else in the Testing repository but this newer Easy-Migrate.

Let me know if that fixes your issue. If so, I'll move it to the normal repository.

--
With best regards

Michael Stauber