Dev-iL commented on code in PR #60270:
URL: https://github.com/apache/airflow/pull/60270#discussion_r2673692170
##########
airflow-core/src/airflow/dag_processing/bundles/base.py:
##########
@@ -76,6 +76,34 @@ def get_bundle_version_path(bundle_name: str, version: str)
-> Path:
return base_folder / version
+def get_bundle_permissions() -> tuple[int, int]:
+ """
+ Return configured permissions for bundle directories and files.
+
+ When using user impersonation (run_as_user), bundle directories and files
+ should be group-writable so that impersonated users can access them.
+
+ :return: Tuple of (folder_permissions, file_permissions) as integers
+ """
+ folder_perms = int(conf.get("dag_processor",
"dag_bundle_new_folder_permissions", fallback="0o775"), 8)
Review Comment:
Maybe I don't understand what you mean - the default for files is 664 and
folders 775.
##########
airflow-core/src/airflow/dag_processing/bundles/base.py:
##########
@@ -76,6 +76,34 @@ def get_bundle_version_path(bundle_name: str, version: str)
-> Path:
return base_folder / version
+def get_bundle_permissions() -> tuple[int, int]:
+ """
+ Return configured permissions for bundle directories and files.
+
+ When using user impersonation (run_as_user), bundle directories and files
+ should be group-writable so that impersonated users can access them.
+
+ :return: Tuple of (folder_permissions, file_permissions) as integers
+ """
+ folder_perms = int(conf.get("dag_processor",
"dag_bundle_new_folder_permissions", fallback="0o775"), 8)
Review Comment:
Maybe I don't understand what you mean - the default for files is 664 and
folders 775.
--
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]
