Jorricks commented on a change in pull request #18438:
URL: https://github.com/apache/airflow/pull/18438#discussion_r719115970
##########
File path: airflow/www/views.py
##########
@@ -4203,7 +4203,8 @@ class
TaskInstanceModelView(AirflowPrivilegeVerifierModelView):
class_permission_name = permissions.RESOURCE_TASK_INSTANCE
method_permission_name = {
'list': 'read',
- 'action_clear': 'edit',
+ 'action_clear': 'delete',
Review comment:
I think you have a valid point here. In my opinion, the delete should
really only be about completely removing any trace of the run. While 'clearing'
is an option we would want to give to any user who can also set tasks to
another state with 'edit' privileges.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
