hussein-awala opened a new pull request, #34942:
URL: https://github.com/apache/airflow/pull/34942

   This PR updates the required permissions for some endpoints:
   -  `update_dag_run_state`,  `clear_dag_run` and `set_dag_run_note` -> it 
replaces `can read dag` by `can edit dag` similar to what we have in the other 
endpoints (for example the one used to clear TI state)
   -  `post_clear_task_instances` -> it replaces `can read dagrun` by `can edit 
dagrun`, because the endpoint accepts a parameter `reset_dag_runs` to reset the 
dagruns states, which should check for edit on dagrun resource.
   - `set_task_instance_note` -> it removes `can read dag` because it is 
useless, the edit permission should be sufficient
   
   I'm not sure if we consider some of these changes as breaking changes, I can 
create different PRs if needed. 


-- 
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: commits-unsubscr...@airflow.apache.org

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

Reply via email to