From bf0906eb9500942baff5011197ad0d20634b3f90 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Fri, 3 Feb 2023 16:36:53 +0100 Subject: [PATCH] Manager: avoid logging an error when requesting a non-existent job This is expected to happen every once in a while, especially now that Flamenco supports job deletion. It's not something to log at error level. --- internal/manager/persistence/jobs.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/internal/manager/persistence/jobs.go b/internal/manager/persistence/jobs.go index 61695041..5a414899 100644 --- a/internal/manager/persistence/jobs.go +++ b/internal/manager/persistence/jobs.go @@ -211,7 +211,8 @@ func (db *DB) StoreAuthoredJob(ctx context.Context, authoredJob job_compilers.Au func (db *DB) FetchJob(ctx context.Context, jobUUID string) (*Job, error) { dbJob := Job{} findResult := db.gormDB.WithContext(ctx). - First(&dbJob, "uuid = ?", jobUUID) + Limit(1). + Find(&dbJob, "uuid = ?", jobUUID) if findResult.Error != nil { return nil, jobError(findResult.Error, "fetching job") }