fix: do not send internal notification for backups and status jobs
This commit is contained in:
@@ -199,7 +199,7 @@ class DatabaseBackupJob implements ShouldBeEncrypted, ShouldQueue
|
|||||||
$databasesToBackup = data_get($this->backup, 'databases_to_backup');
|
$databasesToBackup = data_get($this->backup, 'databases_to_backup');
|
||||||
}
|
}
|
||||||
|
|
||||||
if (is_null($databasesToBackup)) {
|
if (filled($databasesToBackup)) {
|
||||||
if (str($databaseType)->contains('postgres')) {
|
if (str($databaseType)->contains('postgres')) {
|
||||||
$databasesToBackup = [$this->database->postgres_db];
|
$databasesToBackup = [$this->database->postgres_db];
|
||||||
} elseif (str($databaseType)->contains('mongodb')) {
|
} elseif (str($databaseType)->contains('mongodb')) {
|
||||||
@@ -320,12 +320,10 @@ class DatabaseBackupJob implements ShouldBeEncrypted, ShouldQueue
|
|||||||
'filename' => null,
|
'filename' => null,
|
||||||
]);
|
]);
|
||||||
}
|
}
|
||||||
send_internal_notification('DatabaseBackupJob failed with: '.$e->getMessage());
|
|
||||||
$this->team?->notify(new BackupFailed($this->backup, $this->database, $this->backup_output, $database));
|
$this->team?->notify(new BackupFailed($this->backup, $this->database, $this->backup_output, $database));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
} catch (\Throwable $e) {
|
} catch (\Throwable $e) {
|
||||||
send_internal_notification('DatabaseBackupJob failed with: '.$e->getMessage());
|
|
||||||
throw $e;
|
throw $e;
|
||||||
} finally {
|
} finally {
|
||||||
if ($this->team) {
|
if ($this->team) {
|
||||||
|
@@ -89,7 +89,6 @@ class DeleteResourceJob implements ShouldBeEncrypted, ShouldQueue
|
|||||||
$this->resource?->delete_connected_networks($this->resource->uuid);
|
$this->resource?->delete_connected_networks($this->resource->uuid);
|
||||||
}
|
}
|
||||||
} catch (\Throwable $e) {
|
} catch (\Throwable $e) {
|
||||||
send_internal_notification('ContainerStoppingJob failed with: '.$e->getMessage());
|
|
||||||
throw $e;
|
throw $e;
|
||||||
} finally {
|
} finally {
|
||||||
$this->resource->forceDelete();
|
$this->resource->forceDelete();
|
||||||
|
Reference in New Issue
Block a user