From 7b6e7c7994a28967a2fbb7af11fa740816b2758d Mon Sep 17 00:00:00 2001 From: Frantisek Lachman Date: Thu, 12 Mar 2020 10:05:20 +0100 Subject: [PATCH] Use successful result for permission problems It's not an error and we don't want to have it in the sentry. Signed-off-by: Frantisek Lachman --- packit_service/worker/handlers/github_handlers.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packit_service/worker/handlers/github_handlers.py b/packit_service/worker/handlers/github_handlers.py index 7ba1f8c2c..2fc87c10f 100644 --- a/packit_service/worker/handlers/github_handlers.py +++ b/packit_service/worker/handlers/github_handlers.py @@ -358,7 +358,7 @@ def run(self) -> HandlerResults: state=CommitStatus.failure, ) return HandlerResults( - success=False, details={"msg": PERMISSIONS_ERROR_WRITE_OR_ADMIN} + success=True, details={"msg": PERMISSIONS_ERROR_WRITE_OR_ADMIN} ) return super().run() @@ -523,7 +523,7 @@ def run(self) -> HandlerResults: state=CommitStatus.failure, ) return HandlerResults( - success=False, details={"msg": PERMISSIONS_ERROR_WRITE_OR_ADMIN} + success=True, details={"msg": PERMISSIONS_ERROR_WRITE_OR_ADMIN} ) return super().run()