[GitHub] [airflow] xinbinhuang edited a comment on pull request #15743: Redirect forked process output to logger

2021-05-10 Thread GitBox
xinbinhuang edited a comment on pull request #15743: URL: https://github.com/apache/airflow/pull/15743#issuecomment-837266936 > I'm not yet convinced this is an actual problem in a task run by airflow -- I haven't been able to reproduce this with a python operator Can you reproduce

[GitHub] [airflow] boring-cyborg[bot] commented on issue #15768: PythonVirtualenvOperator fails with error from pip execution: Can not perform a '--user' install.

2021-05-10 Thread GitBox
boring-cyborg[bot] commented on issue #15768: URL: https://github.com/apache/airflow/issues/15768#issuecomment-837296054 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] NickPancakes opened a new issue #15768: PythonVirtualenvOperator fails with error from pip execution: Can not perform a '--user' install.

2021-05-10 Thread GitBox
NickPancakes opened a new issue #15768: URL: https://github.com/apache/airflow/issues/15768 **Apache Airflow version**: 2.0.2 **Environment**: - **Hardware configuration**: Macbook Pro 2017 - **OS**: macOS X Catalina 10.15.7 - **Kernel**: Darwin 19.6.0 - **Others**:

[GitHub] [airflow] kaxil commented on a change in pull request #9973: Improve compatibility with mssql

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #9973: URL: https://github.com/apache/airflow/pull/9973#discussion_r629695358 ## File path: airflow/migrations/versions/e9304a3141f0_make_xcom_pkey_columns_non_nullable.py ## @@ -0,0 +1,76 @@ +# +# Licensed to the Apache Software

[GitHub] [airflow] khilawar4 commented on a change in pull request #15727: templatize_awsconnectionid

2021-05-10 Thread GitBox
khilawar4 commented on a change in pull request #15727: URL: https://github.com/apache/airflow/pull/15727#discussion_r629762502 ## File path: airflow/providers/amazon/aws/operators/s3_bucket.py ## @@ -37,12 +37,16 @@ class S3CreateBucketOperator(BaseOperator): running

[GitHub] [airflow] xinbinhuang edited a comment on pull request #15743: Redirect forked process output to logger

2021-05-10 Thread GitBox
xinbinhuang edited a comment on pull request #15743: URL: https://github.com/apache/airflow/pull/15743#issuecomment-837266936 > I'm not yet convinced this is an actual problem in a task run by airflow -- I haven't been able to reproduce this with a python operator Can you reproduce

[GitHub] [airflow] kaxil closed issue #14302: Helm Chart: Review Architecture

2021-05-10 Thread GitBox
kaxil closed issue #14302: URL: https://github.com/apache/airflow/issues/14302 -- 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. For queries about this service, please

[GitHub] [airflow] khilawar4 commented on a change in pull request #15727: templatize_awsconnectionid

2021-05-10 Thread GitBox
khilawar4 commented on a change in pull request #15727: URL: https://github.com/apache/airflow/pull/15727#discussion_r629760349 ## File path: airflow/providers/amazon/aws/operators/cloud_formation.py ## @@ -67,9 +67,11 @@ class CloudFormationDeleteStackOperator(BaseOperator):

[GitHub] [airflow] khilawar4 commented on a change in pull request #15727: templatize_awsconnectionid

2021-05-10 Thread GitBox
khilawar4 commented on a change in pull request #15727: URL: https://github.com/apache/airflow/pull/15727#discussion_r629760514 ## File path: airflow/providers/amazon/aws/operators/emr_modify_cluster.py ## @@ -32,6 +32,8 @@ class EmrModifyClusterOperator(BaseOperator):

[airflow] branch master updated (3edcd12 -> 34cf525)

2021-05-10 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 3edcd12 Update croniter to 1.0.x series (#15769) add 34cf525 Clean up unnecessary Airflow config in helm

[GitHub] [airflow] kaxil merged pull request #15729: Clean up unnecessary Airflow config in helm chart

2021-05-10 Thread GitBox
kaxil merged pull request #15729: URL: https://github.com/apache/airflow/pull/15729 -- 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. For queries about this service,

[GitHub] [airflow] kaxil commented on a change in pull request #9973: Improve compatibility with mssql

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #9973: URL: https://github.com/apache/airflow/pull/9973#discussion_r629691560 ## File path: airflow/migrations/versions/83f031fd9f1c_improve_mssql_compatibility.py ## @@ -0,0 +1,256 @@ +# +# Licensed to the Apache Software

[airflow] branch master updated (25caeda -> 3edcd12)

2021-05-10 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 25caeda Remove unused dependency (#15762) add 3edcd12 Update croniter to 1.0.x series (#15769) No new

[GitHub] [airflow] mik-laj commented on pull request #15755: Save pod name to xcom for KubernetesPodOperator

2021-05-10 Thread GitBox
mik-laj commented on pull request #15755: URL: https://github.com/apache/airflow/pull/15755#issuecomment-837350803 Tests faield. Can you look at it? -- 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

[GitHub] [airflow] dstandish commented on pull request #14105: Add Google leveldb hook (#13109)

2021-05-10 Thread GitBox
dstandish commented on pull request #14105: URL: https://github.com/apache/airflow/pull/14105#issuecomment-837350318 with this change it seems leveldb is a required system dependency for the google provider. it doesn't seem right to require users to install leveldb on their

[GitHub] [airflow] kaxil commented on a change in pull request #9973: Improve compatibility with mssql

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #9973: URL: https://github.com/apache/airflow/pull/9973#discussion_r629691901 ## File path: airflow/migrations/versions/83f031fd9f1c_improve_mssql_compatibility.py ## @@ -0,0 +1,256 @@ +# +# Licensed to the Apache Software

[GitHub] [airflow] kaxil commented on a change in pull request #9973: Improve compatibility with mssql

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #9973: URL: https://github.com/apache/airflow/pull/9973#discussion_r629692470 ## File path: airflow/migrations/versions/bbf4a7ad0465_remove_id_column_from_xcom.py ## @@ -35,6 +37,61 @@ depends_on = None +def

[GitHub] [airflow] kaxil commented on a change in pull request #9973: Improve compatibility with mssql

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #9973: URL: https://github.com/apache/airflow/pull/9973#discussion_r629692289 ## File path: airflow/migrations/versions/98271e7606e2_add_scheduling_decision_to_dagrun_and_.py ## @@ -35,12 +35,32 @@ depends_on = None +def

[GitHub] [airflow] kaxil commented on a change in pull request #9973: Improve compatibility with mssql

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #9973: URL: https://github.com/apache/airflow/pull/9973#discussion_r629703321 ## File path: airflow/migrations/versions/83f031fd9f1c_improve_mssql_compatibility.py ## @@ -0,0 +1,256 @@ +# +# Licensed to the Apache Software

[GitHub] [airflow] kaxil opened a new issue #15771: Automate docs for `values.yaml` via pre-commit config and break them into logical groups

2021-05-10 Thread GitBox
kaxil opened a new issue #15771: URL: https://github.com/apache/airflow/issues/15771 -- 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. For queries about this service,

[GitHub] [airflow] dvd9604 opened a new issue #15767: Webserver changes the base path for the log on rerun of task 404

2021-05-10 Thread GitBox
dvd9604 opened a new issue #15767: URL: https://github.com/apache/airflow/issues/15767 **Apache Airflow version**: 2.0.0 **Environment**: - **OS** (e.g. from /etc/os-release): RHEL7 - **Python**: 3.8.6 - **Executor**: CeleryExecutor - **Workers**: 4 worker nodes -

[GitHub] [airflow] boring-cyborg[bot] commented on issue #15767: Webserver changes the base path for the log on rerun of task 404

2021-05-10 Thread GitBox
boring-cyborg[bot] commented on issue #15767: URL: https://github.com/apache/airflow/issues/15767#issuecomment-837160531 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] SamWheating commented on issue #15759: flask error: ValueError: instance() only accepts datetime objects.

2021-05-10 Thread GitBox
SamWheating commented on issue #15759: URL: https://github.com/apache/airflow/issues/15759#issuecomment-837261174 Fixed in https://github.com/apache/airflow/pull/14416 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [airflow] malthe commented on a change in pull request #15581: Add bindvars Xcom result to Oracle operator

2021-05-10 Thread GitBox
malthe commented on a change in pull request #15581: URL: https://github.com/apache/airflow/pull/15581#discussion_r629657710 ## File path: airflow/providers/oracle/operators/oracle.py ## @@ -62,4 +68,30 @@ def __init__( def execute(self, context) -> None:

[GitHub] [airflow] xinbinhuang edited a comment on pull request #15743: Redirect forked process output to logger

2021-05-10 Thread GitBox
xinbinhuang edited a comment on pull request #15743: URL: https://github.com/apache/airflow/pull/15743#issuecomment-837266936 > I'm not yet convinced this is an actual problem in a task run by airflow -- I haven't been able to reproduce this with a python operator Can you reproduce

[GitHub] [airflow] khilawar4 commented on a change in pull request #15727: templatize_awsconnectionid

2021-05-10 Thread GitBox
khilawar4 commented on a change in pull request #15727: URL: https://github.com/apache/airflow/pull/15727#discussion_r629770582 ## File path: airflow/providers/amazon/aws/operators/s3_delete_objects.py ## @@ -45,6 +45,8 @@ class S3DeleteObjectsOperator(BaseOperator):

[GitHub] [airflow] lsowen commented on issue #13853: Clearing of historic Task or DagRuns leads to failed DagRun

2021-05-10 Thread GitBox
lsowen commented on issue #13853: URL: https://github.com/apache/airflow/issues/13853#issuecomment-837226456 Is this or #14265 a candidate for https://github.com/apache/airflow/milestone/21? -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow] xinbinhuang commented on pull request #15743: Redirect forked process output to logger

2021-05-10 Thread GitBox
xinbinhuang commented on pull request #15743: URL: https://github.com/apache/airflow/pull/15743#issuecomment-837266936 > I'm not yet convinced this is an actual problem in a task run by airflow -- I haven't been able to reproduce this with a python operator Can you reproduce the

[GitHub] [airflow] xinbinhuang edited a comment on pull request #15743: Redirect forked process output to logger

2021-05-10 Thread GitBox
xinbinhuang edited a comment on pull request #15743: URL: https://github.com/apache/airflow/pull/15743#issuecomment-837266936 > I'm not yet convinced this is an actual problem in a task run by airflow -- I haven't been able to reproduce this with a python operator Can you reproduce

[GitHub] [airflow] xinbinhuang edited a comment on pull request #15743: Redirect forked process output to logger

2021-05-10 Thread GitBox
xinbinhuang edited a comment on pull request #15743: URL: https://github.com/apache/airflow/pull/15743#issuecomment-837266936 > I'm not yet convinced this is an actual problem in a task run by airflow -- I haven't been able to reproduce this with a python operator Can you reproduce

[GitHub] [airflow] dstandish edited a comment on pull request #14105: Add Google leveldb hook (#13109)

2021-05-10 Thread GitBox
dstandish edited a comment on pull request #14105: URL: https://github.com/apache/airflow/pull/14105#issuecomment-837350318 with this change it seems leveldb is a required system dependency for the google provider. it doesn't seem right to require users to install leveldb on their

[GitHub] [airflow] mik-laj commented on pull request #14105: Add Google leveldb hook (#13109)

2021-05-10 Thread GitBox
mik-laj commented on pull request #14105: URL: https://github.com/apache/airflow/pull/14105#issuecomment-837368681 @dstandish can you create a issue? -- 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

[GitHub] [airflow] khilawar4 commented on a change in pull request #15727: templatize_awsconnectionid

2021-05-10 Thread GitBox
khilawar4 commented on a change in pull request #15727: URL: https://github.com/apache/airflow/pull/15727#discussion_r629761961 ## File path: airflow/providers/amazon/aws/operators/cloud_formation.py ## @@ -67,9 +67,11 @@ class CloudFormationDeleteStackOperator(BaseOperator):

[GitHub] [airflow] kaxil commented on a change in pull request #9973: Improve compatibility with mssql

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #9973: URL: https://github.com/apache/airflow/pull/9973#discussion_r629692470 ## File path: airflow/migrations/versions/bbf4a7ad0465_remove_id_column_from_xcom.py ## @@ -35,6 +37,61 @@ depends_on = None +def

[GitHub] [airflow] kaxil commented on a change in pull request #9973: Improve compatibility with mssql

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #9973: URL: https://github.com/apache/airflow/pull/9973#discussion_r629701355 ## File path: airflow/migrations/versions/bbf4a7ad0465_remove_id_column_from_xcom.py ## @@ -35,6 +37,61 @@ depends_on = None +def

[GitHub] [airflow] Smithx10 commented on issue #15724: mysql cluster import_error waits for a lock

2021-05-10 Thread GitBox
Smithx10 commented on issue #15724: URL: https://github.com/apache/airflow/issues/15724#issuecomment-837479909 @potiuk, I made a bit of progress today. I forked the percona-healthcheck script, ported it to go, and added raft. It still needs to be cleaned up, Logging, Config,

[GitHub] [airflow] github-actions[bot] commented on pull request #15012: PostgreSQL operator returns query result

2021-05-10 Thread GitBox
github-actions[bot] commented on pull request #15012: URL: https://github.com/apache/airflow/pull/15012#issuecomment-837524479 This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank

[GitHub] [airflow] github-actions[bot] commented on pull request #13832: removing try-catch block to fix timeout exception getting ignored in aws batch operator

2021-05-10 Thread GitBox
github-actions[bot] commented on pull request #13832: URL: https://github.com/apache/airflow/pull/13832#issuecomment-837524537 This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank

[GitHub] [airflow] khilawar4 commented on a change in pull request #15727: templatize_awsconnectionid

2021-05-10 Thread GitBox
khilawar4 commented on a change in pull request #15727: URL: https://github.com/apache/airflow/pull/15727#discussion_r629762975 ## File path: airflow/providers/amazon/aws/operators/s3_list.py ## @@ -65,7 +67,7 @@ class S3ListOperator(BaseOperator): ) """ -

[GitHub] [airflow] kaxil merged pull request #15769: Update croniter to 1.0.x series

2021-05-10 Thread GitBox
kaxil merged pull request #15769: URL: https://github.com/apache/airflow/pull/15769 -- 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. For queries about this service,

[GitHub] [airflow] malthe opened a new pull request #15769: Update croniter to 1.0.x series

2021-05-10 Thread GitBox
malthe opened a new pull request #15769: URL: https://github.com/apache/airflow/pull/15769 The [croniter](https://pypi.org/project/croniter/#changelog) project transitioned to semver2 as part of recently making it to 1.0.x. -- This is an automated message from the Apache Git

[GitHub] [airflow] dstandish edited a comment on pull request #14105: Add Google leveldb hook (#13109)

2021-05-10 Thread GitBox
dstandish edited a comment on pull request #14105: URL: https://github.com/apache/airflow/pull/14105#issuecomment-837350318 with this change it seems leveldb is a required system dependency for the google provider. it doesn't seem right to require users to install leveldb on their

[GitHub] [airflow] kaxil commented on a change in pull request #9973: Improve compatibility with mssql

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #9973: URL: https://github.com/apache/airflow/pull/9973#discussion_r629693129 ## File path: airflow/migrations/versions/bbf4a7ad0465_remove_id_column_from_xcom.py ## @@ -35,6 +37,61 @@ depends_on = None +def

[GitHub] [airflow] kaxil commented on a change in pull request #9973: Improve compatibility with mssql

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #9973: URL: https://github.com/apache/airflow/pull/9973#discussion_r629693037 ## File path: airflow/migrations/versions/bbf4a7ad0465_remove_id_column_from_xcom.py ## @@ -35,6 +37,61 @@ depends_on = None +def

[GitHub] [airflow] kaxil commented on a change in pull request #9973: Improve compatibility with mssql

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #9973: URL: https://github.com/apache/airflow/pull/9973#discussion_r629693212 ## File path: airflow/migrations/versions/bbf4a7ad0465_remove_id_column_from_xcom.py ## @@ -35,6 +37,61 @@ depends_on = None +def

[GitHub] [airflow] dstandish opened a new issue #15770: google provider requires leveldb in host

2021-05-10 Thread GitBox
dstandish opened a new issue #15770: URL: https://github.com/apache/airflow/issues/15770 google provider now requires plyvel, which is used by leveldb hook. to install plyvel, the user needs leveldb headers installed in the system, otherwise it will fail: ```

[GitHub] [airflow] khilawar4 commented on a change in pull request #15727: templatize_awsconnectionid

2021-05-10 Thread GitBox
khilawar4 commented on a change in pull request #15727: URL: https://github.com/apache/airflow/pull/15727#discussion_r629762975 ## File path: airflow/providers/amazon/aws/operators/s3_list.py ## @@ -65,7 +67,7 @@ class S3ListOperator(BaseOperator): ) """ -

[GitHub] [airflow] kaxil commented on pull request #15695: Add integration test for other executors in kubernetes cluster

2021-05-10 Thread GitBox
kaxil commented on pull request #15695: URL: https://github.com/apache/airflow/pull/15695#issuecomment-837577849 @ephraimbuddy We should also now integrate this in CI to run Helm Chart integration test with Celery & Local Executor too -- This is an automated message from the Apache Git

[GitHub] [airflow] kaxil merged pull request #15695: Add integration test for other executors in kubernetes cluster

2021-05-10 Thread GitBox
kaxil merged pull request #15695: URL: https://github.com/apache/airflow/pull/15695 -- 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. For queries about this service,

[GitHub] [airflow] dstandish opened a new pull request #15772: Fix OdbcHook handling of port

2021-05-10 Thread GitBox
dstandish opened a new pull request #15772: URL: https://github.com/apache/airflow/pull/15772 When port was set in connection, it was not added to connection string. This PR fixes that. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow] kaxil commented on a change in pull request #15726: Add more integration tests to the kubernetes executor

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #15726: URL: https://github.com/apache/airflow/pull/15726#discussion_r629773272 ## File path: kubernetes_tests/test_kubernetes_executor.py ## @@ -266,3 +266,65 @@ def test_integration_run_dag_with_scheduler_failure(self): )

[GitHub] [airflow] EunSeop commented on issue #15196: ExternalTaskSensor not working python3.6 & airflow2.0.1

2021-05-10 Thread GitBox
EunSeop commented on issue #15196: URL: https://github.com/apache/airflow/issues/15196#issuecomment-837608306 @DenisOgr I never saw that message. Airflow 2.0 is not perfectly support python 3.6, so I decided upgrade python version using pyenv. Please check airflow and python versions.

[GitHub] [airflow] ephraimbuddy commented on a change in pull request #15726: Add more integration tests to the kubernetes executor

2021-05-10 Thread GitBox
ephraimbuddy commented on a change in pull request #15726: URL: https://github.com/apache/airflow/pull/15726#discussion_r629804088 ## File path: kubernetes_tests/test_kubernetes_executor.py ## @@ -266,3 +266,65 @@ def test_integration_run_dag_with_scheduler_failure(self):

[GitHub] [airflow] dstandish commented on pull request #14521: Add Asana Provider

2021-05-10 Thread GitBox
dstandish commented on pull request #14521: URL: https://github.com/apache/airflow/pull/14521#issuecomment-837774480 seems that the recent change to `get_all_changes_for_package` in docs generation may not handle the "new provider" case? ``` File

[GitHub] [airflow] dstandish edited a comment on pull request #14521: Add Asana Provider

2021-05-10 Thread GitBox
dstandish edited a comment on pull request #14521: URL: https://github.com/apache/airflow/pull/14521#issuecomment-837774480 seems that the recent change to `get_all_changes_for_package` in docs generation may not handle the "new provider" case? ``` File

[GitHub] [airflow] kaxil commented on a change in pull request #15602: Update KylinHook docstring

2021-05-10 Thread GitBox
kaxil commented on a change in pull request #15602: URL: https://github.com/apache/airflow/pull/15602#discussion_r629777234 ## File path: airflow/providers/apache/kylin/hooks/kylin.py ## @@ -26,6 +26,8 @@ class KylinHook(BaseHook): """ +Interact with Kylin to run

[GitHub] [airflow] kaxil commented on pull request #15719: Refactor tests/www/test_views.py (Part 2/2)

2021-05-10 Thread GitBox
kaxil commented on pull request #15719: URL: https://github.com/apache/airflow/pull/15719#issuecomment-837578992 @uranusjr There are some conflicts -- can you fix them please -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] vikramcse commented on issue #15524: Improve test coverage of task_command

2021-05-10 Thread GitBox
vikramcse commented on issue #15524: URL: https://github.com/apache/airflow/issues/15524#issuecomment-837783667 Raised a PR for improving test coverage of task_command.py file https://github.com/apache/airflow/pull/15760 -- This is an automated message from the Apache Git Service. To

[GitHub] [airflow] boring-cyborg[bot] commented on issue #15773: [SambaHook] Can not extract domain from the username

2021-05-10 Thread GitBox
boring-cyborg[bot] commented on issue #15773: URL: https://github.com/apache/airflow/issues/15773#issuecomment-837807022 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] edwinkong opened a new issue #15773: [SambaHook] Can not extract domain from the username

2021-05-10 Thread GitBox
edwinkong opened a new issue #15773: URL: https://github.com/apache/airflow/issues/15773 **Apache Airflow version**: 2.0.1 **Kubernetes version (if you are using kubernetes)** (use `kubectl version`): **Environment**: docker-compose + custom built docker

[GitHub] [airflow] ferruzzi commented on pull request #15727: templatize_awsconnectionid

2021-05-10 Thread GitBox
ferruzzi commented on pull request #15727: URL: https://github.com/apache/airflow/pull/15727#issuecomment-837577270 I'm not a Committer so I can't merge it, but other that my comments about style consistency - which is admittedly subjective - it looks good to me. -- This is an automated

[airflow] branch master updated (34cf525 -> cbc3cb8)

2021-05-10 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 34cf525 Clean up unnecessary Airflow config in helm chart (#15729) add cbc3cb8 Add integration test for

[GitHub] [airflow-site] ecerulm opened a new issue #418: Failed to clone .github/actions/JaimeIves-github-pages-deploy-action

2021-05-10 Thread GitBox
ecerulm opened a new issue #418: URL: https://github.com/apache/airflow-site/issues/418 I just cloned the repo and did `git submodule update --init --recursive` as instructed in the README.md ``` git submodule update --init --recursive Submodule

[GitHub] [airflow-ci-infra] ashb opened a new pull request #27: Perform a docker login before starting the actions runner script

2021-05-10 Thread GitBox
ashb opened a new pull request #27: URL: https://github.com/apache/airflow-ci-infra/pull/27 This was done in the cloud-init, but missed from the migration to packer build scripts. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [airflow-site] ecerulm opened a new issue #417: Link "Suggest a change on this page" is broken

2021-05-10 Thread GitBox
ecerulm opened a new issue #417: URL: https://github.com/apache/airflow-site/issues/417 For example [KubernetesPodOperator](https://airflow.apache.org/docs/apache-airflow-providers-cncf-kubernetes/stable/operators.html) has "Suggest a change on this page" button at the bottom that points

[GitHub] [airflow] github-actions[bot] commented on pull request #15683: Feature qubole hook support headers

2021-05-10 Thread GitBox
github-actions[bot] commented on pull request #15683: URL: https://github.com/apache/airflow/pull/15683#issuecomment-836271743 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not

[GitHub] [airflow] potiuk commented on issue #15753: Airflow scheduler getting stop with sys.exit(os.EX_OK)

2021-05-10 Thread GitBox
potiuk commented on issue #15753: URL: https://github.com/apache/airflow/issues/15753#issuecomment-836298456 I think you might have sys.exit() in your DAG file. The logs you have sent are just normal logs when you exit scheduler. And this will happen if your dag calls sys.exit()

[GitHub] [airflow] JavierLopezT commented on pull request #14415: SnowflakeToS3Operator

2021-05-10 Thread GitBox
JavierLopezT commented on pull request #14415: URL: https://github.com/apache/airflow/pull/14415#issuecomment-836335751 commemt -- 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

[GitHub] [airflow] armandleopold commented on issue #15720: Airflow LDAP search returned multiple results

2021-05-10 Thread GitBox
armandleopold commented on issue #15720: URL: https://github.com/apache/airflow/issues/15720#issuecomment-836372434 @potiuk Thanks for the reply, i used the bitnami docker image : `bitnami/airflow:2.0.2`

[GitHub] [airflow-ci-infra] ashb merged pull request #22: Install node in the AMI

2021-05-10 Thread GitBox
ashb merged pull request #22: URL: https://github.com/apache/airflow-ci-infra/pull/22 -- 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. For queries about this service,

[airflow-ci-infra] branch main updated: Install node in the AMI (#22)

2021-05-10 Thread ash
This is an automated email from the ASF dual-hosted git repository. ash pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow-ci-infra.git The following commit(s) were added to refs/heads/main by this push: new b92388f Install node in the AMI (#22)

[airflow-ci-infra] branch docker-login-worker-pre-start created (now 86a19d7)

2021-05-10 Thread ash
This is an automated email from the ASF dual-hosted git repository. ash pushed a change to branch docker-login-worker-pre-start in repository https://gitbox.apache.org/repos/asf/airflow-ci-infra.git. at 86a19d7 Perform a docker login before starting the actions runner script This

[airflow-ci-infra] 01/01: Perform a docker login before starting the actions runner script

2021-05-10 Thread ash
This is an automated email from the ASF dual-hosted git repository. ash pushed a commit to branch docker-login-worker-pre-start in repository https://gitbox.apache.org/repos/asf/airflow-ci-infra.git commit 86a19d79f8fc6c1bfbf6b5f011d19c66eeb78509 Author: Ash Berlin-Taylor AuthorDate: Mon May 10

[GitHub] [airflow] amatellanes commented on a change in pull request #15680: MongoToS3Operator failed when running with a single query (not aggregate pipeline)

2021-05-10 Thread GitBox
amatellanes commented on a change in pull request #15680: URL: https://github.com/apache/airflow/pull/15680#discussion_r629077291 ## File path: airflow/providers/amazon/aws/transfers/mongo_to_s3.py ## @@ -49,8 +49,8 @@ class MongoToS3Operator(BaseOperator): :type

[GitHub] [airflow] potiuk commented on issue #15753: Airflow scheduler getting stop with sys.exit(os.EX_OK)

2021-05-10 Thread GitBox
potiuk commented on issue #15753: URL: https://github.com/apache/airflow/issues/15753#issuecomment-836299715 In any case the infromation you posted is not enough to make any investigation - we need a reproducible case to be able to do anything with it. I am closing it for now, if you

[GitHub] [airflow] potiuk closed issue #15753: Airflow scheduler getting stop with sys.exit(os.EX_OK)

2021-05-10 Thread GitBox
potiuk closed issue #15753: URL: https://github.com/apache/airflow/issues/15753 -- 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. For queries about this service, please

[GitHub] [airflow] eladkal commented on pull request #15510: OdbcHook returns None. Related to #15016 issue.

2021-05-10 Thread GitBox
eladkal commented on pull request #15510: URL: https://github.com/apache/airflow/pull/15510#issuecomment-836320972 @Goodkat can you fix the failing test? -- 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

[GitHub] [airflow] Junnplus commented on issue #15742: Save pod name in xcom for KubernetesPodOperator.

2021-05-10 Thread GitBox
Junnplus commented on issue #15742: URL: https://github.com/apache/airflow/issues/15742#issuecomment-836375162 hi @mik-laj , Could you please assign this to me? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] potiuk commented on issue #15720: Airflow LDAP search returned multiple results

2021-05-10 Thread GitBox
potiuk commented on issue #15720: URL: https://github.com/apache/airflow/issues/15720#issuecomment-836440958 Yeah - they still use FAB 3.2.2 (just checked). I think you have few options :) 1) Wait for 2.0.3 from bitnami 2) switch to using Airlfow's official/reference docker image

[GitHub] [airflow] potiuk edited a comment on issue #15720: Airflow LDAP search returned multiple results

2021-05-10 Thread GitBox
potiuk edited a comment on issue #15720: URL: https://github.com/apache/airflow/issues/15720#issuecomment-836440958 Yeah - they still use FAB 3.2.2 (just checked). I think you have few options :) 1) Wait for 2.0.3 release/ image released by bitnami 2) switch to using Airlfow's

[GitHub] [airflow] DenisOgr commented on issue #15196: ExternalTaskSensor not working python3.6 & airflow2.0.1

2021-05-10 Thread GitBox
DenisOgr commented on issue #15196: URL: https://github.com/apache/airflow/issues/15196#issuecomment-836660611 @EunSeop how did you set DAG schedule so that it worked? I set up `schedule_interval='@weekly',` cleaned database, and turned on the DAGs:

[GitHub] [airflow] github-actions[bot] commented on pull request #15756: Mark test_mark_success_on_success_callback as quarantined

2021-05-10 Thread GitBox
github-actions[bot] commented on pull request #15756: URL: https://github.com/apache/airflow/pull/15756#issuecomment-836661794 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and

[GitHub] [airflow] potiuk commented on pull request #14521: Add Asana Provider

2021-05-10 Thread GitBox
potiuk commented on pull request #14521: URL: https://github.com/apache/airflow/pull/14521#issuecomment-836668381 No problem, rebases are tricky :) -- 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

[GitHub] [airflow] maryampashmi edited a comment on issue #15451: No module named 'airflow.providers.google.common.hooks.leveldb'

2021-05-10 Thread GitBox
maryampashmi edited a comment on issue #15451: URL: https://github.com/apache/airflow/issues/15451#issuecomment-836684818 > airflow.providers.google.leveldb.hooks.LevelDBHook This is not solved. I am getting the same error. Just tried to create a docker to install the module

[GitHub] [airflow] potiuk commented on issue #15451: No module named 'airflow.providers.google.common.hooks.leveldb'

2021-05-10 Thread GitBox
potiuk commented on issue #15451: URL: https://github.com/apache/airflow/issues/15451#issuecomment-836703445 It is solved in the most recent version of the google provider (3.0.0). Currently when you install 2.0.2 you get the previous version of provider installed, but you can manually

[GitHub] [airflow-site] potiuk closed issue #418: Failed to clone .github/actions/JaimeIves-github-pages-deploy-action

2021-05-10 Thread GitBox
potiuk closed issue #418: URL: https://github.com/apache/airflow-site/issues/418 -- 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. For queries about this service, please

[GitHub] [airflow-site] potiuk commented on issue #418: Failed to clone .github/actions/JaimeIves-github-pages-deploy-action

2021-05-10 Thread GitBox
potiuk commented on issue #418: URL: https://github.com/apache/airflow-site/issues/418#issuecomment-836718312 As discussed on slack, the submodule is added using ssh and you need to have SSH configured to update the submodules.

[GitHub] [airflow-site] potiuk commented on issue #63: Setup automatic deployment of latest documentation

2021-05-10 Thread GitBox
potiuk commented on issue #63: URL: https://github.com/apache/airflow-site/issues/63#issuecomment-836719358 This is done. -- 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.

[GitHub] [airflow-site] potiuk closed issue #63: Setup automatic deployment of latest documentation

2021-05-10 Thread GitBox
potiuk closed issue #63: URL: https://github.com/apache/airflow-site/issues/63 -- 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. For queries about this service, please

[GitHub] [airflow-site] potiuk commented on issue #59: Spell checker

2021-05-10 Thread GitBox
potiuk commented on issue #59: URL: https://github.com/apache/airflow-site/issues/59#issuecomment-836719005 This is done -- 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.

[GitHub] [airflow-site] potiuk closed issue #59: Spell checker

2021-05-10 Thread GitBox
potiuk closed issue #59: URL: https://github.com/apache/airflow-site/issues/59 -- 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. For queries about this service, please

[GitHub] [airflow-site] potiuk closed issue #301: add more tools that are integrated w/ Airflow

2021-05-10 Thread GitBox
potiuk closed issue #301: URL: https://github.com/apache/airflow-site/issues/301 -- 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. For queries about this service, please

[GitHub] [airflow-site] potiuk commented on issue #301: add more tools that are integrated w/ Airflow

2021-05-10 Thread GitBox
potiuk commented on issue #301: URL: https://github.com/apache/airflow-site/issues/301#issuecomment-836724793 We have now "Ecosytem" page. I think we can close this one. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[jira] [Commented] (AIRFLOW-4922) If a task crashes, host name is not committed to the database so logs aren't able to be seen in the UI

2021-05-10 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-4922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17341883#comment-17341883 ] ASF GitHub Bot commented on AIRFLOW-4922: - Deathblade6 commented on pull request #6722: URL:

[GitHub] [airflow] Deathblade6 commented on pull request #6722: [AIRFLOW-4922]Fix task get log by Web UI

2021-05-10 Thread GitBox
Deathblade6 commented on pull request #6722: URL: https://github.com/apache/airflow/pull/6722#issuecomment-836647596 Yeah this still exists( Running Airflow 2.0.1 in multiple vm's in Azure with Celery Executor.) -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] jmelot commented on a change in pull request #14521: Add Asana Provider

2021-05-10 Thread GitBox
jmelot commented on a change in pull request #14521: URL: https://github.com/apache/airflow/pull/14521#discussion_r629335910 ## File path: setup.py ## @@ -198,8 +198,9 @@ def get_sphinx_theme_version() -> str: 'watchtower~=0.7.3', ] apache_beam = [ -

[GitHub] [airflow] vikramcse opened a new pull request #15760: Improve test coverage of task_command.py FIXES: #15524

2021-05-10 Thread GitBox
vikramcse opened a new pull request #15760: URL: https://github.com/apache/airflow/pull/15760 improving test coverage of the tasks from task_command.py file -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] potiuk commented on pull request #9973: Improve compatibility with mssql

2021-05-10 Thread GitBox
potiuk commented on pull request #9973: URL: https://github.com/apache/airflow/pull/9973#issuecomment-836698500 And I just merged marking a test that often failed (for this and other PRs) as "quarantined", so next time you rebase it should be even "greener". -- This is an automated

[GitHub] [airflow] ryanahamilton opened a new pull request #15762: Remove unused dependency

2021-05-10 Thread GitBox
ryanahamilton opened a new pull request #15762: URL: https://github.com/apache/airflow/pull/15762 Removes an unused handlebars dependency that contains [a vulnerability](https://github.com/advisories/GHSA-f2jv-r9rf-7988). It appears to have been added in #7284 to resolve another

[GitHub] [airflow] ashb commented on pull request #15743: Redirect forked process output to logger

2021-05-10 Thread GitBox
ashb commented on pull request #15743: URL: https://github.com/apache/airflow/pull/15743#issuecomment-836663985 I'm not yet convinced this is an actual problem in a task run by airflow -- I haven't been able to reproduce this with a python operator -- This is an automated message from

  1   2   >