From 116a596a5431373e36d151fe933d6541d7f02ae6 Mon Sep 17 00:00:00 2001 From: Julie Qiu Date: Wed, 2 Sep 2020 11:08:51 -0400 Subject: internal/postgres: do not reprocess 490s and 491s Bad modules and alternative modules do not need to be reprocessed, and are no longer marked for reprocessing. Modules in the reprocess state for those statuses (540 and 541s) will continue be requeued so that we don't end up having two status codes that indicate the same thing. Once all 540s and 541s have been updated, we can delete those error codes. Change-Id: I804c021421e8a8e19be49992d9113ea832d35a9b Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/252679 Run-TryBot: Julie Qiu TryBot-Result: kokoro Reviewed-by: Jonathan Amsterdam --- internal/postgres/requeue.go | 2 -- 1 file changed, 2 deletions(-) (limited to 'internal/postgres/requeue.go') diff --git a/internal/postgres/requeue.go b/internal/postgres/requeue.go index d9ecf5a4..bea82d2a 100644 --- a/internal/postgres/requeue.go +++ b/internal/postgres/requeue.go @@ -24,8 +24,6 @@ func (db *DB) UpdateModuleVersionStatesForReprocessing(ctx context.Context, appV for _, status := range []int{ http.StatusOK, derrors.ToStatus(derrors.HasIncompletePackages), - derrors.ToStatus(derrors.BadModule), - derrors.ToStatus(derrors.AlternativeModule), derrors.ToStatus(derrors.DBModuleInsertInvalid), } { if err := db.UpdateModuleVersionStatesWithStatus(ctx, status, appVersion); err != nil { -- cgit v1.3-5-g9baa