Schedule command issue when using backup:run (Laravel/backup package)

[sys-auditing]

Hi, when using
php artisan tenants:artisan "backup:run --only-db --only-to-disk=s3"
all works well, but when using schedule command the backup failed

$schedule->command("backup:run --only-db --only-to-disk=s3")
        ->weeklyOn(3, '03:00')
        ->runInBackground()
        ->onFailure(function () {
            Log::error("error");
        });

N.B i'm using a cron

* * * * * cd /var/www/html/site/ && /usr/bin/php artisan schedule:run 
* * * * * cd /var/www/html/site/ && /usr/bin/php artisan tenants:artisan schedule:run

Any help will be appreciated
Thank you

[masterix21]

Take a look here: https://spatie.be/docs/laravel-multitenancy/v2/advanced-usage/executing-code-for-tenants-and-landlords#content-executing-a-delayed-callback-in-the-correct-tenant-context

[sys-auditing]

Thank you for your response.

Should i keep those cron job commands?

* * * * * cd /var/www/html/site/ && /usr/bin/php artisan schedule:run 
* * * * * cd /var/www/html/site/ && /usr/bin/php artisan tenants:artisan schedule:run

Noting that i'm using others schedule command without issues by using the above cron job commands like :

//Tenant aware
if (Tenant::current()) { 
    $schedule->job(GenerateStats::class)
                ->dailyAt('1:30')
                ->onFailure(function () {
                    Log::error("GenerateStats error");
                });
}

however, when using backup:run command, the process failed

//Tenant aware
if (Tenant::current()) { 
    $schedule->command("backup:run --only-db --only-to-disk=s3")
            ->weeklyOn(1, '03:00')
            ->runInBackground()
            ->onFailure(function () {
                Log::error("error");
            });
}

also landlord database backup works well using the same code

//Tenant aware
if (Tenant::current()){
 .....
 
}else{  //landlord
$schedule->command("backup:run --only-db --only-to-disk=spaces")
        ->weeklyOn(1, '03:30')
        ->runInBackground()
        ->onFailure(function (){
            Log::error( error");
        });
}

I want some help.
Thank you

[masterix21]

You could use only

* * * * * cd /var/www/html/site/ && /usr/bin/php artisan schedule:run 

[sys-auditing]

Using what i have explained in my last reply, i observed that schedule command works and looping through all tenants by using the landlord connection instead of tenant connection, so instead of making tenant db backup, the command backup the landlord db multiple time (tenants db count).

If i use your last proposal how so can run tenant aware jobs ... like

if (Tenant::current()) { 
    $schedule->job(GenerateStats::class)
                ->dailyAt('1:30')
                ->onFailure(function () {
                    Log::error("GenerateStats error");
                });

 return;
}

In fact i'm confused.

Thank you

[sys-auditing]

I solved the issue, by using schedule jobs instead of command.

//BackupTenantDatabases.php Job

 public function handle()
    {
        Artisan::call('tenants:artisan', [
            'artisanCommand' => 'backup:run --only-db --only-to-disk=spaces'
        ]);
    }

//Kernel.php

if (Tenant::current()) {
$schedule->job(BackupTenantDatabases::class)
            ->weeklyOn(1, '03:00');

return;
}