From 82907196e1a697dfa5040d2a157381fe8d40bdc3 Mon Sep 17 00:00:00 2001
From: Mike McLean <mikem@redhat.com>
Date: May 03 2018 00:08:31 +0000
Subject: PR#691: option for notifications in untagBuildBypass


Merges #691
https://pagure.io/koji/pull-request/691

Fixes: #690
https://pagure.io/koji/issue/690
Tag notifications don't work for untagging ops by koji-gc

---

diff --git a/hub/kojihub.py b/hub/kojihub.py
index 814d89e..6c30357 100644
--- a/hub/kojihub.py
+++ b/hub/kojihub.py
@@ -9249,8 +9249,8 @@ class RootExports(object):
         context.session.assertPerm('admin')
         add_external_rpm(rpminfo, external_repo, strict=strict)
 
-    def tagBuildBypass(self, tag, build, force=False):
-        """Tag a build without running post checks or notifications
+    def tagBuildBypass(self, tag, build, force=False, notify=True):
+        """Tag a build without running post checks
 
         This is a short circuit function for imports.
         Admin permission required.
@@ -9261,6 +9261,8 @@ class RootExports(object):
         """
         context.session.assertPerm('admin')
         _tag_build(tag, build, force=force)
+        if notify:
+            tag_notification(True, None, tag, build, context.session.user_id)
 
     def tagBuild(self, tag, build, force=False, fromtag=None):
         """Request that a build be tagged
@@ -9347,8 +9349,8 @@ class RootExports(object):
             tag_notification(False, None, tag, build, user_id, False, "%s: %s" % (exctype, value))
             raise
 
-    def untagBuildBypass(self, tag, build, strict=True, force=False):
-        """Untag a build without any checks or notifications
+    def untagBuildBypass(self, tag, build, strict=True, force=False, notify=True):
+        """Untag a build without any checks
 
         Admins only. Intended for syncs/imports.
 
@@ -9356,6 +9358,8 @@ class RootExports(object):
         No return value"""
         context.session.assertPerm('admin')
         _untag_build(tag, build, strict=strict, force=force)
+        if notify:
+            tag_notification(True, None, tag, build, context.session.user_id)
 
     def moveBuild(self, tag1, tag2, build, force=False):
         """Move a build from tag1 to tag2