joaopamaral commented on code in PR #40703:
URL: https://github.com/apache/airflow/pull/40703#discussion_r1675797557
##########
airflow/providers/fab/auth_manager/security_manager/override.py:
##########
@@ -1102,81 +1103,103 @@ def sync_perm_for_dag(
:param dag_id: the ID of the DAG whose permissions should be updated
:param access_control: a dict where each key is a role name and
- each value is a set() of action names (e.g.,
+ each value is a set() of action names in case of DAGs resource
(e.g.,
{'can_read'}
+ or the value can be a dict where each key is a resource name and
+ each value is a set() of action names (e.g., {'DAG Runs':
{'can_read'}})
Review Comment:
Tried to make it more clear in
https://github.com/apache/airflow/pull/40703/commits/c7ebe69d6fe41e37262ace566d70d489a895fc75
. Please let me know if it's ok now.
--
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]
