refactor: Remove unnecessary logging from various jobs and views

This commit is contained in:
Andras Bacsai 2025-12-03 15:47:58 +01:00
parent b66988943b
commit b321abbe2e
6 changed files with 2 additions and 44 deletions

View File

@ -472,7 +472,6 @@ class DatabaseBackupJob implements ShouldBeEncrypted, ShouldQueue
throw new \Exception('MongoDB credentials not found. Ensure MONGO_INITDB_ROOT_USERNAME and MONGO_INITDB_ROOT_PASSWORD environment variables are available in the container.');
}
}
\Log::info('MongoDB backup URL configured', ['has_url' => filled($url), 'using_env_vars' => blank($this->database->internal_db_url)]);
if ($databaseWithCollections === 'all') {
$commands[] = 'mkdir -p '.$this->backup_dir;
if (str($this->database->image)->startsWith('mongo:4')) {

View File

@ -35,12 +35,6 @@ class ValidateAndInstallServerJob implements ShouldQueue
// Mark validation as in progress
$this->server->update(['is_validating' => true]);
Log::info('ValidateAndInstallServer: Starting validation', [
'server_id' => $this->server->id,
'server_name' => $this->server->name,
'attempt' => $this->numberOfTries + 1,
]);
// Validate connection
['uptime' => $uptime, 'error' => $error] = $this->server->validateConnection();
if (! $uptime) {
@ -92,13 +86,6 @@ class ValidateAndInstallServerJob implements ShouldQueue
return;
}
Log::info('ValidateAndInstallServer: Installing prerequisites', [
'server_id' => $this->server->id,
'attempt' => $this->numberOfTries + 1,
'missing_commands' => $validationResult['missing'],
'found_commands' => $validationResult['found'],
]);
// Install prerequisites
$this->server->installPrerequisites();
@ -128,11 +115,6 @@ class ValidateAndInstallServerJob implements ShouldQueue
return;
}
Log::info('ValidateAndInstallServer: Installing Docker', [
'server_id' => $this->server->id,
'attempt' => $this->numberOfTries + 1,
]);
// Install Docker
$this->server->installDocker();
@ -158,12 +140,6 @@ class ValidateAndInstallServerJob implements ShouldQueue
return;
}
// Validation successful!
Log::info('ValidateAndInstallServer: Validation successful', [
'server_id' => $this->server->id,
'server_name' => $this->server->name,
]);
// Start proxy if needed
if (! $this->server->isBuildServer()) {
$proxyShouldRun = CheckProxy::run($this->server, true);

View File

@ -2,8 +2,6 @@
namespace App\Notifications\Dto;
use Illuminate\Support\Facades\Log;
class PushoverMessage
{
public function __construct(
@ -43,8 +41,6 @@ class PushoverMessage
$payload['message'] .= "&nbsp;<a href='".$buttonUrl."'>".$text.'</a>';
}
Log::info('Pushover message', $payload);
return $payload;
}
}

View File

@ -2966,8 +2966,6 @@ function getHelperVersion(): string
function loadConfigFromGit(string $repository, string $branch, string $base_directory, int $server_id, int $team_id): ?array
{
Log::info("coolify.json: Starting detection for {$repository} branch {$branch} base_dir {$base_directory}");
$server = Server::where('id', $server_id)->where('team_id', $team_id)->first();
if (! $server) {
Log::warning("coolify.json: Server not found - server_id: {$server_id}, team_id: {$team_id}");
@ -2986,8 +2984,6 @@ function loadConfigFromGit(string $repository, string $branch, string $base_dire
}
$pathsToCheck[] = 'coolify.json';
Log::info('coolify.json: Checking paths - '.implode(', ', $pathsToCheck));
// Build sparse-checkout file list
$fileList = collect($pathsToCheck)->map(fn ($path) => "./{$path}")->implode(' ');
@ -3010,11 +3006,7 @@ function loadConfigFromGit(string $repository, string $branch, string $base_dire
try {
$output = instant_remote_process($commands, $server);
Log::info('coolify.json: Raw output length - '.strlen($output ?? ''));
if (empty($output)) {
Log::info('coolify.json: No output from remote process');
return null;
}
@ -3038,11 +3030,6 @@ function loadConfigFromGit(string $repository, string $branch, string $base_dire
// Warn about unknown top-level fields
$knownFields = ['version', 'name', 'description', 'build', 'domains', 'environment_variables', 'health_check', 'limits', 'settings'];
$unknownFields = array_diff(array_keys($config), $knownFields);
if (! empty($unknownFields)) {
Log::info('coolify.json: Unknown fields detected (will be ignored) - '.implode(', ', $unknownFields));
}
Log::info('coolify.json: Successfully loaded config with keys - '.implode(', ', array_keys($config)));
return $config;
} catch (\Exception $e) {

View File

@ -14,7 +14,7 @@
<div class="pb-2 text-red-500">{{ $parseError }}</div>
@endif
<x-forms.textarea useMonacoEditor monacoEditorLanguage="json" monacoJsonSchema="/schemas/coolify.schema.json"
<x-forms.textarea useMonacoEditor monacoEditorLanguage="json"
rows="20" id="coolifyJson" autofocus wire:model.live.debounce.500ms="coolifyJson"
placeholder='{
"version": "1.0",

View File

@ -1,5 +1,5 @@
<form wire:submit="uploadConfig" class="flex flex-col gap-2 w-full">
<x-forms.textarea id="config" monacoEditorLanguage="json" monacoJsonSchema="/schemas/coolify.schema.json"
<x-forms.textarea id="config" monacoEditorLanguage="json"
useMonacoEditor />
<x-forms.button type="submit">
Upload