ac9eca3c05
Exited containers don't run health checks, so showing "(unhealthy)" is misleading. This fix ensures exited status displays without health suffixes across all monitoring systems (SSH, Sentinel, services, etc.) and at the UI layer for backward compatibility with existing data. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
88 lines
3.5 KiB
PHP
88 lines
3.5 KiB
PHP
<?php
|
|
|
|
namespace App\Actions\Shared;
|
|
|
|
use App\Models\Application;
|
|
use App\Services\ContainerStatusAggregator;
|
|
use App\Traits\CalculatesExcludedStatus;
|
|
use Lorisleiva\Actions\Concerns\AsAction;
|
|
|
|
class ComplexStatusCheck
|
|
{
|
|
use AsAction;
|
|
use CalculatesExcludedStatus;
|
|
|
|
public function handle(Application $application)
|
|
{
|
|
$servers = $application->additional_servers;
|
|
$servers->push($application->destination->server);
|
|
foreach ($servers as $server) {
|
|
$is_main_server = $application->destination->server->id === $server->id;
|
|
if (! $server->isFunctional()) {
|
|
if ($is_main_server) {
|
|
$application->update(['status' => 'exited']);
|
|
|
|
continue;
|
|
} else {
|
|
$application->additional_servers()->updateExistingPivot($server->id, ['status' => 'exited']);
|
|
|
|
continue;
|
|
}
|
|
}
|
|
$containers = instant_remote_process(["docker container inspect $(docker container ls -q --filter 'label=coolify.applicationId={$application->id}' --filter 'label=coolify.pullRequestId=0') --format '{{json .}}'"], $server, false);
|
|
$containers = format_docker_command_output_to_json($containers);
|
|
|
|
if ($containers->count() > 0) {
|
|
$statusToSet = $this->aggregateContainerStatuses($application, $containers);
|
|
|
|
if ($is_main_server) {
|
|
$statusFromDb = $application->status;
|
|
if ($statusFromDb !== $statusToSet) {
|
|
$application->update(['status' => $statusToSet]);
|
|
}
|
|
} else {
|
|
$additional_server = $application->additional_servers()->wherePivot('server_id', $server->id);
|
|
$statusFromDb = $additional_server->first()->pivot->status;
|
|
if ($statusFromDb !== $statusToSet) {
|
|
$additional_server->updateExistingPivot($server->id, ['status' => $statusToSet]);
|
|
}
|
|
}
|
|
} else {
|
|
if ($is_main_server) {
|
|
$application->update(['status' => 'exited']);
|
|
|
|
continue;
|
|
} else {
|
|
$application->additional_servers()->updateExistingPivot($server->id, ['status' => 'exited']);
|
|
|
|
continue;
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
private function aggregateContainerStatuses($application, $containers)
|
|
{
|
|
$dockerComposeRaw = data_get($application, 'docker_compose_raw');
|
|
$excludedContainers = $this->getExcludedContainersFromDockerCompose($dockerComposeRaw);
|
|
|
|
// Filter non-excluded containers
|
|
$relevantContainers = collect($containers)->filter(function ($container) use ($excludedContainers) {
|
|
$labels = data_get($container, 'Config.Labels', []);
|
|
$serviceName = data_get($labels, 'com.docker.compose.service');
|
|
|
|
return ! ($serviceName && $excludedContainers->contains($serviceName));
|
|
});
|
|
|
|
// If all containers are excluded, calculate status from excluded containers
|
|
// but mark it with :excluded to indicate monitoring is disabled
|
|
if ($relevantContainers->isEmpty()) {
|
|
return $this->calculateExcludedStatus($containers, $excludedContainers);
|
|
}
|
|
|
|
// Use ContainerStatusAggregator service for state machine logic
|
|
$aggregator = new ContainerStatusAggregator;
|
|
|
|
return $aggregator->aggregateFromContainers($relevantContainers);
|
|
}
|
|
}
|