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 <flachman@redhat.com>
packit · Mar 12, 2020 · 7b6e7c7 · 7b6e7c7
1 parent 3589947
commit 7b6e7c7
Showing 1 changed file with 2 additions and 2 deletions.
diff --git 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()