Re: [PR] Fix location requirement in DataflowTemplatedJobStartOperator [airflow]

2024-02-05 Thread via GitHub
potiuk merged PR #37069: URL: https://github.com/apache/airflow/pull/37069 -- 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.a

Re: [PR] Fix location requirement in DataflowTemplatedJobStartOperator [airflow]

2024-02-05 Thread via GitHub
boring-cyborg[bot] commented on PR #37069: URL: https://github.com/apache/airflow/pull/37069#issuecomment-1926897524 Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions.

Re: [PR] Fix location requirement in DataflowTemplatedJobStartOperator [airflow]

2024-02-05 Thread via GitHub
VladaZakharova commented on PR #37069: URL: https://github.com/apache/airflow/pull/37069#issuecomment-1926661951 Hi @dirrao @potiuk ! Can you please check the changes here? Thank you! -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

Re: [PR] Fix location requirement in DataflowTemplatedJobStartOperator [airflow]

2024-01-30 Thread via GitHub
shardbread commented on code in PR #37069: URL: https://github.com/apache/airflow/pull/37069#discussion_r1471186463 ## airflow/providers/google/cloud/operators/dataflow.py: ## @@ -677,6 +677,9 @@ def set_current_job(current_job): options = self.dataflow_default_options

Re: [PR] Fix location requirement in DataflowTemplatedJobStartOperator [airflow]

2024-01-29 Thread via GitHub
dirrao commented on code in PR #37069: URL: https://github.com/apache/airflow/pull/37069#discussion_r1469838242 ## airflow/providers/google/cloud/operators/dataflow.py: ## @@ -677,6 +677,9 @@ def set_current_job(current_job): options = self.dataflow_default_options

[PR] Fix location requirement in DataflowTemplatedJobStartOperator [airflow]

2024-01-29 Thread via GitHub
shardbread opened a new pull request, #37069: URL: https://github.com/apache/airflow/pull/37069 An error was encountered when calling the execute method on a DataflowTemplateJobStartOperator instance with an unspecified location. This fix resolves the issue with an empty location in the e

Re: [PR] Fix location requirement in DataflowTemplatedJobStartOperator [airflow]

2024-01-23 Thread via GitHub
shardbread closed pull request #36970: Fix location requirement in DataflowTemplatedJobStartOperator URL: https://github.com/apache/airflow/pull/36970 -- 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

Re: [PR] Fix location requirement in DataflowTemplatedJobStartOperator [airflow]

2024-01-23 Thread via GitHub
boring-cyborg[bot] commented on PR #36970: URL: https://github.com/apache/airflow/pull/36970#issuecomment-1905693793 Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution

[PR] Fix location requirement in DataflowTemplatedJobStartOperator [airflow]

2024-01-23 Thread via GitHub
shardbread opened a new pull request, #36970: URL: https://github.com/apache/airflow/pull/36970 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelin