Merge pull request #9852 from appwrite/fix-double-logs

Fix: double logs on failure
This commit is contained in:
Matej Bačo 2025-05-21 16:59:24 +02:00 committed by GitHub
commit 6b7071ca94
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -1218,13 +1218,22 @@ class Builds extends Action
$message .= "\n" . $error;
}
// Combine with previous logs if deployment got past build process
$previousLogs = '';
if (!empty($deployment->getAttribute('buildEndedAt', ''))) {
$previousLogs = $deployment->getAttribute('buildLogs', '');
if (!empty($previousLogs)) {
$message = $previousLogs . "\n" . $message;
}
}
$endTime = DateTime::now();
$durationEnd = \microtime(true);
$deployment->setAttribute('buildEndedAt', $endTime);
$deployment->setAttribute('buildDuration', \intval(\ceil($durationEnd - $durationStart)));
$deployment->setAttribute('status', 'failed');
$deployment->setAttribute('buildLogs', $deployment->getAttribute('buildLogs', '') . "\n" . $message);
$deployment->setAttribute('buildLogs', $message);
$deployment = $dbForProject->updateDocument('deployments', $deploymentId, $deployment);
if ($deployment->getInternalId() === $resource->getAttribute('latestDeploymentInternalId', '')) {