You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/08/29 11:21:00 UTC
[GitHub] [airflow] bhirsz opened a new pull request, #26036: Migrate Cloud Storage Transfer Service system tests to AIP-47
bhirsz opened a new pull request, #26036:
URL: https://github.com/apache/airflow/pull/26036
From #22447 and #22430
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] bhirsz commented on a diff in pull request #26036: Migrate Cloud Storage Transfer Service system tests to AIP-47
Posted by GitBox <gi...@apache.org>.
bhirsz commented on code in PR #26036:
URL: https://github.com/apache/airflow/pull/26036#discussion_r957224022
##########
tests/system/providers/google/cloud/cloud_sql/example_cloud_sql.py:
##########
@@ -141,7 +141,7 @@
with models.DAG(
DAG_ID,
- schedule=None,
+ schedule="@once",
Review Comment:
Not related to this PR but it's a very minor fix to other system tests that wasn't worth separate PR - "None" schedule made this tests to always pass
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] bhirsz closed pull request #26036: Migrate Cloud Storage Transfer Service system tests to AIP-47
Posted by GitBox <gi...@apache.org>.
bhirsz closed pull request #26036: Migrate Cloud Storage Transfer Service system tests to AIP-47
URL: https://github.com/apache/airflow/pull/26036
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] bhirsz commented on pull request #26036: Migrate Cloud Storage Transfer Service system tests to AIP-47
Posted by GitBox <gi...@apache.org>.
bhirsz commented on PR #26036:
URL: https://github.com/apache/airflow/pull/26036#issuecomment-1230205150
Static checks are failing on file that wasn't modified in this PR.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org