You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/04/27 19:40:07 UTC

[GitHub] [airflow] kaxil edited a comment on pull request #11260: Add DAG permissions based on DAG tags

kaxil edited a comment on pull request #11260:
URL: https://github.com/apache/airflow/pull/11260#issuecomment-827866937


   Temporarily added it to 2.1 Milestone to revisit this at a later time.
   
   Assigning permissions to a group of DAGs is a good future -- I am not yet sure if "Tags" are the best way for it for the reason James already mentioned. Probably better would be some sort of "wildcard" on permissions -- not sure at the moment.
   
   
   Also, we should just have a single way to add/edit permissions via UI & API -- not via DAGs themselves -- I know currently we allow it but it is "anti-pattern" in my opinion and is already causing some headaches.


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org