[GitHub] [airflow] Piatachock commented on pull request #25158: YandexCloud provider: Support new Yandex SDK features for DataProc

2022-07-27 Thread GitBox
Piatachock commented on PR #25158: URL: https://github.com/apache/airflow/pull/25158#issuecomment-1196493344 Oh, makes sense, thanks for the explanation. Please let me know if I can help in any way. On the other hand, we at Yandex would be happy to pack this PR in next providers release;

[GitHub] [airflow] Piatachock commented on pull request #25158: YandexCloud provider: Support new Yandex SDK features for DataProc

2022-07-26 Thread GitBox
Piatachock commented on PR #25158: URL: https://github.com/apache/airflow/pull/25158#issuecomment-1195640508 Not sure if fails are related to changes of this PR; ``` Unknown warnings generated: [1393](https://github.com/apache/airflow/runs/7520766090?check_suite_focus=true#step:

[GitHub] [airflow] Piatachock commented on pull request #25158: YandexCloud provider: Support new Yandex SDK features for DataProc

2022-07-21 Thread GitBox
Piatachock commented on PR #25158: URL: https://github.com/apache/airflow/pull/25158#issuecomment-1191737660 That makes sense, thanks! I guess I was not sure if pre-commit should happen on my side or on CI side. I'll see if there is other way to phrase it in README -- This is an automa

[GitHub] [airflow] Piatachock commented on pull request #25158: YandexCloud provider: Support new Yandex SDK features for DataProc

2022-07-20 Thread GitBox
Piatachock commented on PR #25158: URL: https://github.com/apache/airflow/pull/25158#issuecomment-1190254380 @potiuk hey Jarek, I'm not sure what to do with CI, guide me please. I've got a dependency version update in this PR, so I updated provider.yaml file. Now CI fails on check on `gen