diff --git a/changelogs/unreleased/zj-sentry-ignore-not-found.yml b/changelogs/unreleased/zj-sentry-ignore-not-found.yml new file mode 100644 index 0000000000000000000000000000000000000000..c7c31811df5f047ced55938ad0d42642cb18264e --- /dev/null +++ b/changelogs/unreleased/zj-sentry-ignore-not-found.yml @@ -0,0 +1,5 @@ +--- +title: Ignore NotFound errors in Sentry +merge_request: 1156 +author: +type: changed diff --git a/internal/middleware/sentryhandler/sentryhandler.go b/internal/middleware/sentryhandler/sentryhandler.go index 47991a82aeac84cc0e18934f32bcdf89e61ad2bc..4cdb79af1f9404de2b09ba06234c971c90b610eb 100644 --- a/internal/middleware/sentryhandler/sentryhandler.go +++ b/internal/middleware/sentryhandler/sentryhandler.go @@ -16,6 +16,8 @@ import ( var ignoredCodes = []codes.Code{ // OK means there was no error codes.OK, + // NotFound means the (pool) repository was not present + codes.NotFound, // Canceled and DeadlineExceeded indicate clients that disappeared or lost interest codes.Canceled, codes.DeadlineExceeded, diff --git a/internal/middleware/sentryhandler/sentryhandler_test.go b/internal/middleware/sentryhandler/sentryhandler_test.go index 88888acb90a8c0a473bd705b603663a705d6b189..2e5b1670f12074e7ee4212159b42189255698380 100644 --- a/internal/middleware/sentryhandler/sentryhandler_test.go +++ b/internal/middleware/sentryhandler/sentryhandler_test.go @@ -32,13 +32,11 @@ func Test_generateRavenPacket(t *testing.T) { wantCulprit: "SSHService::SSHUploadPack", }, { - name: "GRPC error", - method: "/gitaly.RepoService/RepoExists", - sinceStart: 500 * time.Millisecond, - err: status.Errorf(codes.NotFound, "Something failed"), - wantCode: codes.NotFound, - wantMessage: "rpc error: code = NotFound desc = Something failed", - wantCulprit: "RepoService::RepoExists", + name: "GRPC error", + method: "/gitaly.RepoService/RepoExists", + sinceStart: 500 * time.Millisecond, + err: status.Errorf(codes.NotFound, "Something failed"), + wantNil: true, }, { name: "nil",