Просмотр исходного кода

Simplify error identification

GitOrigin-RevId: 6dae1ed99269deb26e15ec921ba173f85f5e6fc9
Andrew Rumble 1 год назад
Родитель
Сommit
0b387c5116
1 измененных файлов с 4 добавлено и 22 удалено
  1. 4 22
      services/history-v1/backupVerifier/ProjectVerifier.mjs

+ 4 - 22
services/history-v1/backupVerifier/ProjectVerifier.mjs

@@ -40,31 +40,13 @@ export function setWriteMetrics(writeMetrics) {
  * @param {string} historyId
  */
 function handleVerificationError(error, historyId) {
-  let errorType = 'unknown'
-  // Additional detail for verbose/development
-  if (error instanceof Error) {
-    if (error instanceof BackupCorruptedInvalidBlobError) {
-      errorType = 'invalid-blob'
-    } else if (error instanceof BackupCorruptedMissingBlobError) {
-      errorType = 'missing-blob'
-    } else if (error instanceof BackupCorruptedError) {
-      errorType = 'backup-corrupted-unknown'
-    } else if (error instanceof BackupRPOViolationChunkNotBackedUpError) {
-      errorType = 'rpo-violation-chunk-not-backed-up'
-    } else if (error instanceof BackupRPOViolationError) {
-      errorType = 'rpo-violation'
-    }
-  }
-
-  logger.error(
-    { errorType, historyId, error },
-    'error verifying project backup'
-  )
+  const name = error instanceof Error ? error.name : 'UnknownError'
+  logger.error({ historyId, error, name }, 'error verifying project backup')
 
   WRITE_METRICS &&
-    metrics.inc(METRICS.backup_project_verification_failed, 1, { errorType })
+    metrics.inc(METRICS.backup_project_verification_failed, 1, { name })
 
-  return errorType
+  return name
 }
 
 /**