-
Notifications
You must be signed in to change notification settings - Fork 16.4k
Description
Apache Airflow Provider(s)
Versions of Apache Airflow Providers
apache-airflow-providers-google: version 8.9.0
Apache Airflow version
2.5.1
Operating System
Linux - official airflow image from docker hub apache/airflow:slim-2.5.1
Deployment
Official Apache Airflow Helm Chart
Deployment details
No response
What happened
Not sure if it's a bug or a feature.
Originally I have used LocalFilesystemToGoogleDriveOperator to try uploading a file into a shared Google Drive without success.
Provider didn't find a directory with a given name so it created a new one without browsing shared drives. Method call that doesn't allow to upload it is here:
https://github.com/apache/airflow/blob/main/airflow/providers/google/suite/hooks/drive.py#L223
What you think should happen instead
It would be nice if there was an optional parameter to provide a drive_id into which user would like to upload a file. With the same directory check behaviour that already exists but extended to shared drives
How to reproduce
- Create a directory on the shared google drive
- Fill
LocalFilesystemToGoogleDriveOperatorconstructor with the arguments. - Execute the function
Anything else
I am willing to submit a PR but I would need to know more details, your thoughts, expectations of the implementation to make as little iteration on it possible.
Are you willing to submit PR?
- Yes I am willing to submit a PR!
Code of Conduct
- I agree to follow this project's Code of Conduct