From 4c156f484096fb70d9440afa26f93c6e228758f3 Mon Sep 17 00:00:00 2001
From: Tomas Kopecek <tkopecek@redhat.com>
Date: Jun 13 2019 17:43:14 +0000
Subject: add 'target' permission


---

diff --git a/docs/source/permissions.rst b/docs/source/permissions.rst
index 2d90f04..e259e9e 100644
--- a/docs/source/permissions.rst
+++ b/docs/source/permissions.rst
@@ -37,6 +37,9 @@ Administration
 ``tag``
   Permission for adding/deleting/editing tags
 
+``target``
+  Permission for adding/deleting/editing targets
+
 Tasks
 -----
 
@@ -61,8 +64,8 @@ Tasks
 ``regen-repo``
   same as ``repo`` for now
 
-Data
-----
+Data Import
+-----------
 ``image-import``
   used for importing external maven artifacts
   (``koji import-archive --type maven``)
diff --git a/hub/kojihub.py b/hub/kojihub.py
index a4bbc18..e2006e3 100644
--- a/hub/kojihub.py
+++ b/hub/kojihub.py
@@ -2797,7 +2797,7 @@ def _validate_build_target_name(name):
 def create_build_target(name, build_tag, dest_tag):
     """Create a new build target"""
 
-    context.session.assertPerm('admin')
+    context.session.assertPerm('target')
     return _create_build_target(name, build_tag, dest_tag)
 
 
@@ -2833,7 +2833,7 @@ def _create_build_target(name, build_tag, dest_tag):
 
 def edit_build_target(buildTargetInfo, name, build_tag, dest_tag):
     """Set the build_tag and dest_tag of an existing build_target to new values"""
-    context.session.assertPerm('admin')
+    context.session.assertPerm('target')
     _edit_build_target(buildTargetInfo, name, build_tag, dest_tag)
 
 
@@ -2885,7 +2885,7 @@ def _edit_build_target(buildTargetInfo, name, build_tag, dest_tag):
 def delete_build_target(buildTargetInfo):
     """Delete the build target with the given name.  If no build target
     exists, raise a GenericError."""
-    context.session.assertPerm('admin')
+    context.session.assertPerm('target')
     _delete_build_target(buildTargetInfo)