SamWheating opened a new pull request, #34114: URL: https://github.com/apache/airflow/pull/34114
Follow-up to https://github.com/apache/airflow/pull/33632, which only tested the relevant code in isolation. While verifying this behaviour in order to test the noticed two things - 1) The `sync_perm --include-dags` CLI command was ignoring DAGs with empty (but not None) `access_control` 2) Empty access control blocks were being overwritten with `None` in DAG initialization. This PR fixes these two issues. I am still working on verifying that this works end-to-end (thus enabling the functionality defined in https://github.com/apache/airflow/pull/33632). -- 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
