From 8a518deac2802bd7b767d71eeeade248d339ea75 Mon Sep 17 00:00:00 2001
From: Tomas Hrcka <thrcka@redhat.com>
Date: Jul 22 2021 09:34:58 +0000
Subject: Make the comment less cryptic


Signed-off-by: Tomas Hrcka <thrcka@redhat.com>

---

diff --git a/fedscm_admin/bugzilla.py b/fedscm_admin/bugzilla.py
index 82ffd33..9654239 100644
--- a/fedscm_admin/bugzilla.py
+++ b/fedscm_admin/bugzilla.py
@@ -169,8 +169,8 @@ class BugzillaClient(object):
                             'can\'t be validated.'.format(bug.assigned_to))
                     if not FAS_CLIENT.user_member_of(fas_reviewer, 'packager'):
                         raise ValidationError('The Bugzilla bug\'s review '
-                                              'is approved by a user that is '
-                                              'not a packager')
+                                              'is approved by a user "{0}" that is '
+                                              'not a packager'.format(bug.assigned_to))
                     fas_submitter = self.get_fas_user_by_bz_email(bug.creator)
                     if not fas_submitter:
                         raise ValidationError(
@@ -178,9 +178,8 @@ class BugzillaClient(object):
                             'is not tied to a user in FAS. Group membership '
                             'can\'t be validated.'.format(bug.creator))
                     if not FAS_CLIENT.user_member_of(fas_submitter, 'packager'):
-                        raise ValidationError('The Bugzilla bug\'s review '
-                                              'is submitted by a user that is '
-                                              'not a packager')
+                        raise ValidationError('The Bugzilla reporter "{0}"'
+                                              'is not a packager'.format(bug.creator))
                 # Setter will be an empty string and emails will not be shown
                 # if the user is not logged in. This is why we check for
                 # authentication here.