[GitHub] [airflow-client-python] msumit closed pull request #16: Prepare release 2.0.1 due to Openapi generator upgrade

2021-09-13 Thread GitBox
msumit closed pull request #16: URL: https://github.com/apache/airflow-client-python/pull/16 -- 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: commi

[GitHub] [airflow-client-python] msumit closed pull request #30: Issue #27 : Fixed dag details to accept None type for dag_run_timeout

2021-09-13 Thread GitBox
msumit closed pull request #30: URL: https://github.com/apache/airflow-client-python/pull/30 -- 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: commi

[airflow] branch main updated (27144bd -> fe6a769)

2021-09-13 Thread uranusjr
This is an automated email from the ASF dual-hosted git repository. uranusjr pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 27144bd Doc change: XCOM / Taskflow (#18212) add fe6a769 Add some basic metrics to the Triggerer (#18214) No

[GitHub] [airflow] uranusjr merged pull request #18214: Add some basic metrics to the Triggerer

2021-09-13 Thread GitBox
uranusjr merged pull request #18214: URL: https://github.com/apache/airflow/pull/18214 -- 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-uns

[GitHub] [airflow] uranusjr commented on pull request #18209: Upgrade importlib-resources version

2021-09-13 Thread GitBox
uranusjr commented on pull request #18209: URL: https://github.com/apache/airflow/pull/18209#issuecomment-918851329 This broke tests in main, see #18225. -- 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 t

[GitHub] [airflow] uranusjr opened a new issue #18225: tests/core/test_providers_manager.py::TestProviderManager::test_hooks broken on 3.6

2021-09-13 Thread GitBox
uranusjr opened a new issue #18225: URL: https://github.com/apache/airflow/issues/18225 ### Apache Airflow version main (development) ### Operating System Any ### Versions of Apache Airflow Providers _No response_ ### Deployment Other #

[GitHub] [airflow] malthe edited a comment on pull request #18112: Optional method name in task deferred

2021-09-13 Thread GitBox
malthe edited a comment on pull request #18112: URL: https://github.com/apache/airflow/pull/18112#issuecomment-918843087 @andrewgodwin a different take on this issue could be to add the following method to `BaseOperator`: ```python def execute_complete(self, context, event=None):

[GitHub] [airflow] malthe commented on pull request #18112: Optional method name in task deferred

2021-09-13 Thread GitBox
malthe commented on pull request #18112: URL: https://github.com/apache/airflow/pull/18112#issuecomment-918843087 @andrewgodwin a different take on this issue could be to add the following method to `BaseOperator`: ```python def execute_complete(self, context, event=None): self

[GitHub] [airflow] uranusjr commented on pull request #18222: make current_user_has_permissions backwards compatible

2021-09-13 Thread GitBox
uranusjr commented on pull request #18222: URL: https://github.com/apache/airflow/pull/18222#issuecomment-918828337 This blows up db query counts. Backward compatibility is valuable, but perferrably we should keep the nicer current behaviour by default and only fall back to compatibility b

[GitHub] [airflow] uranusjr commented on issue #18188: NoResultFound raises in UI when access to Task Instance Details

2021-09-13 Thread GitBox
uranusjr commented on issue #18188: URL: https://github.com/apache/airflow/issues/18188#issuecomment-918824179 Also, what are the links that brought you to the page? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [airflow] uranusjr commented on issue #18188: NoResultFound raises in UI when access to Task Instance Details

2021-09-13 Thread GitBox
uranusjr commented on issue #18188: URL: https://github.com/apache/airflow/issues/18188#issuecomment-918823472 Again, what would show in the *Task Instance Attributes* fields? Only your first screenshot is related to the question, and unfortunately the fields are cropped so I can’t see wha

[GitHub] [airflow] uranusjr edited a comment on issue #18188: NoResultFound raises in UI when access to Task Instance Details

2021-09-13 Thread GitBox
uranusjr edited a comment on issue #18188: URL: https://github.com/apache/airflow/issues/18188#issuecomment-917952074 Could you describe what would show in the *Task Instance Attributes* field in that case? (I think I know, but it’d be nice to have someone to confirm this.) -- This is a

[GitHub] [airflow] uranusjr opened a new pull request #18224: Emit explicit exception if user fails to add due to db errors

2021-09-13 Thread GitBox
uranusjr opened a new pull request #18224: URL: https://github.com/apache/airflow/pull/18224 Fix #18204. -- 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-

[GitHub] [airflow] uranusjr commented on pull request #17552: AIP 39: Documentation

2021-09-13 Thread GitBox
uranusjr commented on pull request #17552: URL: https://github.com/apache/airflow/pull/17552#issuecomment-918742831 I think this is ready for review. In addition to the new timetable how-to page, I also rewrote many *execution date* and `execution_date` occurrences to refer to the data int

[GitHub] [airflow] iostreamdoth commented on a change in pull request #18197: Add gcs yml config to create dataproc cluster

2021-09-13 Thread GitBox
iostreamdoth commented on a change in pull request #18197: URL: https://github.com/apache/airflow/pull/18197#discussion_r707831115 ## File path: airflow/providers/google/cloud/operators/dataproc.py ## @@ -48,6 +54,83 @@ ) +class DataProcCreateWorkflowBaseOperator(BaseOpera

[GitHub] [airflow] iostreamdoth commented on a change in pull request #18197: Add gcs yml config to create dataproc cluster

2021-09-13 Thread GitBox
iostreamdoth commented on a change in pull request #18197: URL: https://github.com/apache/airflow/pull/18197#discussion_r707831115 ## File path: airflow/providers/google/cloud/operators/dataproc.py ## @@ -48,6 +54,83 @@ ) +class DataProcCreateWorkflowBaseOperator(BaseOpera

[GitHub] [airflow] iostreamdoth commented on a change in pull request #18197: Add gcs yml config to create dataproc cluster

2021-09-13 Thread GitBox
iostreamdoth commented on a change in pull request #18197: URL: https://github.com/apache/airflow/pull/18197#discussion_r707831115 ## File path: airflow/providers/google/cloud/operators/dataproc.py ## @@ -48,6 +54,83 @@ ) +class DataProcCreateWorkflowBaseOperator(BaseOpera

[GitHub] [airflow] iostreamdoth commented on a change in pull request #18197: Add gcs yml config to create dataproc cluster

2021-09-13 Thread GitBox
iostreamdoth commented on a change in pull request #18197: URL: https://github.com/apache/airflow/pull/18197#discussion_r707831115 ## File path: airflow/providers/google/cloud/operators/dataproc.py ## @@ -48,6 +54,83 @@ ) +class DataProcCreateWorkflowBaseOperator(BaseOpera

[GitHub] [airflow] iostreamdoth commented on a change in pull request #18197: Add gcs yml config to create dataproc cluster

2021-09-13 Thread GitBox
iostreamdoth commented on a change in pull request #18197: URL: https://github.com/apache/airflow/pull/18197#discussion_r707831115 ## File path: airflow/providers/google/cloud/operators/dataproc.py ## @@ -48,6 +54,83 @@ ) +class DataProcCreateWorkflowBaseOperator(BaseOpera

[GitHub] [airflow] iostreamdoth commented on a change in pull request #18197: Add gcs yml config to create dataproc cluster

2021-09-13 Thread GitBox
iostreamdoth commented on a change in pull request #18197: URL: https://github.com/apache/airflow/pull/18197#discussion_r707831115 ## File path: airflow/providers/google/cloud/operators/dataproc.py ## @@ -48,6 +54,83 @@ ) +class DataProcCreateWorkflowBaseOperator(BaseOpera

[airflow] branch main updated (ca45bba -> 27144bd)

2021-09-13 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from ca45bba Chart: Allow running and waiting for DB Migrations using default image (#18218) add 27144bd Doc cha

[GitHub] [airflow] kaxil merged pull request #18212: Doc change: XCOM / Taskflow

2021-09-13 Thread GitBox
kaxil merged pull request #18212: URL: https://github.com/apache/airflow/pull/18212 -- 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-unsubs

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #18212: Doc change: XCOM / Taskflow

2021-09-13 Thread GitBox
boring-cyborg[bot] commented on pull request #18212: URL: https://github.com/apache/airflow/pull/18212#issuecomment-918707090 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [airflow] phemmmie commented on issue #18213: airflow bug

2021-09-13 Thread GitBox
phemmmie commented on issue #18213: URL: https://github.com/apache/airflow/issues/18213#issuecomment-918705933 starting airflow 2.13 fails to start shows ```Something bad has happened. Please consider letting us know by creating a bug report using GitHub

[GitHub] [airflow] o-nikolas commented on pull request #17951: Refresh credentials for long-running pods on EKS

2021-09-13 Thread GitBox
o-nikolas commented on pull request #17951: URL: https://github.com/apache/airflow/pull/17951#issuecomment-918684577 > @potiuk I am awaiting a review from AWS Team, but the person is on vacation. Yes, sorry folks, both Dennis and I have been on vacation for the past 3 weeks. One of

[GitHub] [airflow] o-nikolas commented on pull request #17145: Adds an s3 list prefixes operator

2021-09-13 Thread GitBox
o-nikolas commented on pull request #17145: URL: https://github.com/apache/airflow/pull/17145#issuecomment-918681350 > @o-nikolas I went ahead and reverted back to having two distinct operators - if for no other reason than to keep this PR / convo alive. If I had a better understanding of

[GitHub] [airflow] github-actions[bot] commented on pull request #18212: Doc change: XCOM / Taskflow

2021-09-13 Thread GitBox
github-actions[bot] commented on pull request #18212: URL: https://github.com/apache/airflow/pull/18212#issuecomment-918679211 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 tak

[GitHub] [airflow] github-actions[bot] commented on issue #9048: ERROR - Can't compile non template nodes when I am using Papermill Operator

2021-09-13 Thread GitBox
github-actions[bot] commented on issue #9048: URL: https://github.com/apache/airflow/issues/9048#issuecomment-918678141 This issue has been closed because it has not received response from the issue author. -- This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [airflow] github-actions[bot] commented on issue #9721: InvalidToken error when using PostgresHook with PythonOperator

2021-09-13 Thread GitBox
github-actions[bot] commented on issue #9721: URL: https://github.com/apache/airflow/issues/9721#issuecomment-918678121 This issue has been closed because it has not received response from the issue author. -- This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [airflow] github-actions[bot] closed issue #9048: ERROR - Can't compile non template nodes when I am using Papermill Operator

2021-09-13 Thread GitBox
github-actions[bot] closed issue #9048: URL: https://github.com/apache/airflow/issues/9048 -- 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

[GitHub] [airflow] github-actions[bot] closed issue #16025: AirflowException: Celery command failed on host: [...]

2021-09-13 Thread GitBox
github-actions[bot] closed issue #16025: URL: https://github.com/apache/airflow/issues/16025 -- 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: commi

[GitHub] [airflow] github-actions[bot] commented on issue #16025: AirflowException: Celery command failed on host: [...]

2021-09-13 Thread GitBox
github-actions[bot] commented on issue #16025: URL: https://github.com/apache/airflow/issues/16025#issuecomment-918678021 This issue has been closed because it has not received response from the issue author. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] github-actions[bot] closed issue #9721: InvalidToken error when using PostgresHook with PythonOperator

2021-09-13 Thread GitBox
github-actions[bot] closed issue #9721: URL: https://github.com/apache/airflow/issues/9721 -- 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

[GitHub] [airflow] mik-laj commented on pull request #18211: Support all Unix wildcards in S3KeySensor

2021-09-13 Thread GitBox
mik-laj commented on pull request #18211: URL: https://github.com/apache/airflow/pull/18211#issuecomment-918665410 Can you add tests to avoid regression? -- 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 t

[airflow] branch main updated (b7f366c -> ca45bba)

2021-09-13 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from b7f366c Fix failing main due to #18209 (#18215) add ca45bba Chart: Allow running and waiting for DB Migrati

[GitHub] [airflow] kaxil merged pull request #18218: Chart: Allow running and waiting for DB Migrations using default image

2021-09-13 Thread GitBox
kaxil merged pull request #18218: URL: https://github.com/apache/airflow/pull/18218 -- 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-unsubs

[GitHub] [airflow] mik-laj removed a comment on pull request #18214: Add some basic metrics to the Triggerer

2021-09-13 Thread GitBox
mik-laj removed a comment on pull request #18214: URL: https://github.com/apache/airflow/pull/18214#issuecomment-918663488 Can you update docs? https://github.com/apache/airflow/blob/main/docs/apache-airflow/logging-monitoring/metrics.rst -- This is an automated message from the Apache

[GitHub] [airflow] mik-laj commented on pull request #18214: Add some basic metrics to the Triggerer

2021-09-13 Thread GitBox
mik-laj commented on pull request #18214: URL: https://github.com/apache/airflow/pull/18214#issuecomment-918663488 Can you update docs? https://github.com/apache/airflow/blob/main/docs/apache-airflow/logging-monitoring/metrics.rst -- This is an automated message from the Apache Git Serv

[GitHub] [airflow] mik-laj commented on issue #18216: Helm chart ingress support multiple hostnames

2021-09-13 Thread GitBox
mik-laj commented on issue #18216: URL: https://github.com/apache/airflow/issues/18216#issuecomment-918663018 @fredthomsen Assigned. I know some corporations have complex DNS configurations. -- This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [airflow] kaxil commented on pull request #18212: Doc change: XCOM / Taskflow

2021-09-13 Thread GitBox
kaxil commented on pull request #18212: URL: https://github.com/apache/airflow/pull/18212#issuecomment-918661955 Static checks are failing: https://github.com/apache/airflow/pull/18212/checks?check_run_id=3592918893#step:10:276 -- This is an automated message from the Apache Git Service.

[GitHub] [airflow] mik-laj commented on issue #18213: airflow bug

2021-09-13 Thread GitBox
mik-laj commented on issue #18213: URL: https://github.com/apache/airflow/issues/18213#issuecomment-918661411 No content. Closing. If you update your ticket with more information, let me know to open it. -- This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [airflow] mik-laj closed issue #18213: airflow bug

2021-09-13 Thread GitBox
mik-laj closed issue #18213: URL: https://github.com/apache/airflow/issues/18213 -- 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.

[GitHub] [airflow] mik-laj closed issue #18223: Cannot import name 'FlowerCommand' from 'flower.command'

2021-09-13 Thread GitBox
mik-laj closed issue #18223: URL: https://github.com/apache/airflow/issues/18223 -- 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.

[GitHub] [airflow] mik-laj commented on issue #18223: Cannot import name 'FlowerCommand' from 'flower.command'

2021-09-13 Thread GitBox
mik-laj commented on issue #18223: URL: https://github.com/apache/airflow/issues/18223#issuecomment-918660406 Celery 5.0 and Flower 1.0 are not supported by Airflow 2.1. Please use the constraint files to install Apache Airflow. to avoid dependency problems. https://github.com/apache/air

[GitHub] [airflow] boring-cyborg[bot] commented on issue #18223: Cannot import name 'FlowerCommand' from 'flower.command'

2021-09-13 Thread GitBox
boring-cyborg[bot] commented on issue #18223: URL: https://github.com/apache/airflow/issues/18223#issuecomment-918658596 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 mess

[GitHub] [airflow] sbwilli3 opened a new issue #18223: Cannot import name 'FlowerCommand' from 'flower.command'

2021-09-13 Thread GitBox
sbwilli3 opened a new issue #18223: URL: https://github.com/apache/airflow/issues/18223 ### Apache Airflow Provider(s) celery ### Versions of Apache Airflow Providers apache-airflow==2.1.3 apache-airflow-providers-celery==2.1.0 celery==5.1.2 flower==1.0.0

[GitHub] [airflow] ryanyuan commented on pull request #18184: Migrate Google Cloud Build from Discovery API to Python SDK

2021-09-13 Thread GitBox
ryanyuan commented on pull request #18184: URL: https://github.com/apache/airflow/pull/18184#issuecomment-918655847 Thanks @turbaszek @eladkal @potiuk -- 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] jaketf opened a new pull request #18222: make current_user_has_permissions backwards compatible

2021-09-13 Thread GitBox
jaketf opened a new pull request #18222: URL: https://github.com/apache/airflow/pull/18222 make current_user_has_permissions backwards compatible --- **^ Add meaningful description above** IMHO Subclasses that already override `get_current_user_permissions` should n

[GitHub] [airflow] kaxil closed pull request #18218: Chart: Allow running and waiting for DB Migrations using default image

2021-09-13 Thread GitBox
kaxil closed pull request #18218: URL: https://github.com/apache/airflow/pull/18218 -- 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-unsubs

[GitHub] [airflow] kaxil closed pull request #18212: Doc change: XCOM / Taskflow

2021-09-13 Thread GitBox
kaxil closed pull request #18212: URL: https://github.com/apache/airflow/pull/18212 -- 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-unsubs

[airflow] branch main updated (dd313a5 -> b7f366c)

2021-09-13 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from dd313a5 Omit ``airflow._vendor`` package in coverage report (#18221) add b7f366c Fix failing main due to #18

[GitHub] [airflow] github-actions[bot] commented on pull request #18215: Solve CI issues

2021-09-13 Thread GitBox
github-actions[bot] commented on pull request #18215: URL: https://github.com/apache/airflow/pull/18215#issuecomment-918645971 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 tak

[GitHub] [airflow] kaxil merged pull request #18215: Solve CI issues

2021-09-13 Thread GitBox
kaxil merged pull request #18215: URL: https://github.com/apache/airflow/pull/18215 -- 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-unsubs

[GitHub] [airflow] kaxil commented on pull request #18215: Solve CI issues

2021-09-13 Thread GitBox
kaxil commented on pull request #18215: URL: https://github.com/apache/airflow/pull/18215#issuecomment-918643341 I am that someone πŸ˜„ -- my bad, should have checked the failing tests -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [airflow] kaxil closed pull request #18215: Solve CI issues

2021-09-13 Thread GitBox
kaxil closed pull request #18215: URL: https://github.com/apache/airflow/pull/18215 -- 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-unsubs

[airflow] branch main updated (3b2b7e7 -> dd313a5)

2021-09-13 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 3b2b7e7 Upgrade ``importlib-resources`` version (#18209) add dd313a5 Omit ``airflow._vendor`` package in co

[GitHub] [airflow] kaxil merged pull request #18221: Omit airflow._vendor package in coverage report

2021-09-13 Thread GitBox
kaxil merged pull request #18221: URL: https://github.com/apache/airflow/pull/18221 -- 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-unsubs

[GitHub] [airflow] github-actions[bot] commented on pull request #18214: Add some basic metrics to the Triggerer

2021-09-13 Thread GitBox
github-actions[bot] commented on pull request #18214: URL: https://github.com/apache/airflow/pull/18214#issuecomment-918642138 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 tak

[airflow] branch constraints-main updated: Bump importlib-resources

2021-09-13 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch constraints-main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-main by this push: new 0ee0232 Bump importlib-

[GitHub] [airflow] kaxil closed pull request #18214: Add some basic metrics to the Triggerer

2021-09-13 Thread GitBox
kaxil closed pull request #18214: URL: https://github.com/apache/airflow/pull/18214 -- 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-unsubs

[GitHub] [airflow] github-actions[bot] commented on pull request #18218: Chart: Allow running and waiting for DB Migrations using default image

2021-09-13 Thread GitBox
github-actions[bot] commented on pull request #18218: URL: https://github.com/apache/airflow/pull/18218#issuecomment-918640084 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 m

[GitHub] [airflow] kaxil commented on pull request #18212: Doc change: XCOM / Taskflow

2021-09-13 Thread GitBox
kaxil commented on pull request #18212: URL: https://github.com/apache/airflow/pull/18212#issuecomment-918638072 (retriggering CI) -- 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 co

[GitHub] [airflow] kaxil closed pull request #18212: Doc change: XCOM / Taskflow

2021-09-13 Thread GitBox
kaxil closed pull request #18212: URL: https://github.com/apache/airflow/pull/18212 -- 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-unsubs

[GitHub] [airflow] jedcunningham commented on a change in pull request #18218: Chart: Allow running and waiting for DB Migrations using default image

2021-09-13 Thread GitBox
jedcunningham commented on a change in pull request #18218: URL: https://github.com/apache/airflow/pull/18218#discussion_r707753920 ## File path: chart/values.schema.json ## @@ -613,6 +613,12 @@ } ] }, +"useDefaultImageForMigration

[GitHub] [airflow] mik-laj opened a new pull request #18221: Omit airflow._vendor package in coverage report

2021-09-13 Thread GitBox
mik-laj opened a new pull request #18221: URL: https://github.com/apache/airflow/pull/18221 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more inf

[GitHub] [airflow] mik-laj opened a new pull request #18220: Improve coverage for airflow.cli

2021-09-13 Thread GitBox
mik-laj opened a new pull request #18220: URL: https://github.com/apache/airflow/pull/18220 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more inf

[GitHub] [airflow] jedcunningham commented on issue #17191: Healthcheck endpoint for workers

2021-09-13 Thread GitBox
jedcunningham commented on issue #17191: URL: https://github.com/apache/airflow/issues/17191#issuecomment-918605664 I'm leaning toward just having a way to check both the health of the worker and log server at the same time (be it exec or http based). I feel the log server is an extra feat

[GitHub] [airflow] jameslamb commented on pull request #18219: add more type hints in CeleryKubernetesExecutor

2021-09-13 Thread GitBox
jameslamb commented on pull request #18219: URL: https://github.com/apache/airflow/pull/18219#issuecomment-918589885 I don't believe the failing CI build is related to this PR's changes. https://github.com/apache/airflow/pull/18219/checks?check_run_id=3592125119 ```text ER

[GitHub] [airflow] jameslamb opened a new pull request #18219: add more type hints in CeleryKubernetesExecutor

2021-09-13 Thread GitBox
jameslamb opened a new pull request #18219: URL: https://github.com/apache/airflow/pull/18219 This PR proposes adding a few more type hints to places in `CeleryKubernetesExecutor` that are currently missing them. I think this change may improve the ability of static analysis tools li

[GitHub] [airflow] kaxil opened a new pull request #18218: Chart: Allow running and waiting for DB Migrations using default image

2021-09-13 Thread GitBox
kaxil opened a new pull request #18218: URL: https://github.com/apache/airflow/pull/18218 This keeps the same behaviour but allows setting `useDefaultImageForMigration` to `true` which will then use the default image (set by `.Values.defaultAirflowRepository` and `.Values.defaultAirflowTag

[GitHub] [airflow] alex-astronomer opened a new issue #18217: Audit Logging for Variables, Connections, Pools

2021-09-13 Thread GitBox
alex-astronomer opened a new issue #18217: URL: https://github.com/apache/airflow/issues/18217 ### Description Audit Logs or some other type of logging that shows details about the [who, what, where, when, why] [changed, added, delted] Airflow Variables, Connections, Pools etc.

[GitHub] [airflow] fredthomsen opened a new issue #18216: Helm chart ingress support multiple hostnames

2021-09-13 Thread GitBox
fredthomsen opened a new issue #18216: URL: https://github.com/apache/airflow/issues/18216 ### Description When the official airflow helm chart is used to install airflow in k8s, I want to be able to access the airflow UI from multiple hostnames. Currently given how the ingress res

[GitHub] [airflow] github-actions[bot] commented on pull request #18205: Add 2.2.0b1 changelog

2021-09-13 Thread GitBox
github-actions[bot] commented on pull request #18205: URL: https://github.com/apache/airflow/pull/18205#issuecomment-918566599 The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide

[GitHub] [airflow] jedcunningham edited a comment on issue #18146: Deferred TI's `next_method` and `next_kwargs` not cleared on retries

2021-09-13 Thread GitBox
jedcunningham edited a comment on issue #18146: URL: https://github.com/apache/airflow/issues/18146#issuecomment-918519695 The following DAG can be used to both reproduce (dagruns fail) and verify the issue is fixed (dagruns succeed). Basically, the first run it will fail so it ends up in

[GitHub] [airflow] shlomi-viz commented on issue #18111: HeadObject error when trying to upload file to S3

2021-09-13 Thread GitBox
shlomi-viz commented on issue #18111: URL: https://github.com/apache/airflow/issues/18111#issuecomment-918562444 I created a PR https://github.com/apache/airflow/pull/18201, what is the process to get it reviewed? Thanks -- This is an automated message from the Apache Git Service. To re

[GitHub] [airflow] jedcunningham commented on a change in pull request #18205: Add 2.2.0b1 changelog

2021-09-13 Thread GitBox
jedcunningham commented on a change in pull request #18205: URL: https://github.com/apache/airflow/pull/18205#discussion_r707664922 ## File path: CHANGELOG.txt ## @@ -1,3 +1,278 @@ +Airflow 2.2.0b1, TBD +- + +New Features + + +- AIP-39: Handl

[GitHub] [airflow] bmckallagat-os opened a new pull request #18215: Solve CI issues

2021-09-13 Thread GitBox
bmckallagat-os opened a new pull request #18215: URL: https://github.com/apache/airflow/pull/18215 Hey there someone merged my previous PR #18155 before I could update it to pass, I believe this will fix it. Closes #18155 --- **^ Add meaningful description above**

[GitHub] [airflow] john-jac commented on issue #13824: Cloudwatch Integration: SIGTERM/SIGKILL Sent Following DAG Completion, Causing Errors in Worker Logs

2021-09-13 Thread GitBox
john-jac commented on issue #13824: URL: https://github.com/apache/airflow/issues/13824#issuecomment-918539921 Hi Folks...have there been any new insights into this issue? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [airflow] dimberman commented on a change in pull request #15330: Add a Docker Taskflow decorator

2021-09-13 Thread GitBox
dimberman commented on a change in pull request #15330: URL: https://github.com/apache/airflow/pull/15330#discussion_r707646679 ## File path: airflow/providers_manager.py ## @@ -278,6 +309,51 @@ def _get_attr(obj: Any, attr_name: str): return None return g

[GitHub] [airflow] jedcunningham commented on issue #18146: Deferred TI's `next_method` and `next_kwargs` not cleared on retries

2021-09-13 Thread GitBox
jedcunningham commented on issue #18146: URL: https://github.com/apache/airflow/issues/18146#issuecomment-918519695 The following DAG can be used to both reproduce (dagruns fail) and verify the issue is fixed (dagruns succeed). Basically, the first run it will fail so it ends up in UP_FOR_

[GitHub] [airflow] andrewgodwin opened a new pull request #18214: Add some basic metrics to the Triggerer

2021-09-13 Thread GitBox
andrewgodwin opened a new pull request #18214: URL: https://github.com/apache/airflow/pull/18214 This adds some basic metrics to the Triggerer: * `triggers.running`: A gauge of how many are currently running * `triggers.succeeded` and `triggers.failed`: Counters of how many triggers h

[GitHub] [airflow] Jorricks commented on pull request #17207: Fix external_executor_id not being set for manually run jobs.

2021-09-13 Thread GitBox
Jorricks commented on pull request #17207: URL: https://github.com/apache/airflow/pull/17207#issuecomment-918514365 Tests should be fixed now πŸ‘ -- 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 t

[GitHub] [airflow] fritz-astronomer edited a comment on pull request #18212: Doc change: XCOM / Taskflow

2021-09-13 Thread GitBox
fritz-astronomer edited a comment on pull request #18212: URL: https://github.com/apache/airflow/pull/18212#issuecomment-918503159 CICD and Breeze both seem to be failing for unrelated reasons for me 😞 -- This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [airflow] fritz-astronomer commented on pull request #18212: Doc change: XCOM / Taskflow

2021-09-13 Thread GitBox
fritz-astronomer commented on pull request #18212: URL: https://github.com/apache/airflow/pull/18212#issuecomment-918503159 PR and Breeze both seem to be failing for unrelated reasons for me 😞 -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow] ridwantahir commented on issue #17507: Task processes killed with WARNING - Recorded pid does not match the current pid

2021-09-13 Thread GitBox
ridwantahir commented on issue #17507: URL: https://github.com/apache/airflow/issues/17507#issuecomment-918499820 this proble is still there in airflow 2.13!! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [airflow] boring-cyborg[bot] commented on issue #18213: airflow bug

2021-09-13 Thread GitBox
boring-cyborg[bot] commented on issue #18213: URL: https://github.com/apache/airflow/issues/18213#issuecomment-918466133 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 mess

[GitHub] [airflow] phemmmie opened a new issue #18213: airflow bug

2021-09-13 Thread GitBox
phemmmie opened a new issue #18213: URL: https://github.com/apache/airflow/issues/18213 ### Apache Airflow version 2.1.3 (latest released) ### Operating System mac Os big sur ### Versions of Apache Airflow Providers _No response_ ### Deployment

[airflow] branch main updated (9d49772 -> 3b2b7e7)

2021-09-13 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 9d49772 BugFix: Wipe ``next_kwargs`` and ``next_method`` on task failure (#18210) add 3b2b7e7 Upgrade ``imp

[GitHub] [airflow] kaxil closed issue #18155: Upgrade `importlib-resources` version

2021-09-13 Thread GitBox
kaxil closed issue #18155: URL: https://github.com/apache/airflow/issues/18155 -- 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...

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #18209: Upgrade importlib-resources version

2021-09-13 Thread GitBox
boring-cyborg[bot] commented on pull request #18209: URL: https://github.com/apache/airflow/pull/18209#issuecomment-918454419 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [airflow] kaxil merged pull request #18209: Upgrade importlib-resources version

2021-09-13 Thread GitBox
kaxil merged pull request #18209: URL: https://github.com/apache/airflow/pull/18209 -- 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-unsubs

[GitHub] [airflow] github-actions[bot] commented on pull request #18209: Upgrade importlib-resources version

2021-09-13 Thread GitBox
github-actions[bot] commented on pull request #18209: URL: https://github.com/apache/airflow/pull/18209#issuecomment-918454067 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 tak

[airflow] branch main updated (9c8f7ac -> 9d49772)

2021-09-13 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 9c8f7ac Add heartbeat to TriggererJob (#18129) add 9d49772 BugFix: Wipe ``next_kwargs`` and ``next_method``

[GitHub] [airflow] kaxil merged pull request #18210: Wipe ``next_kwargs`` and ``next_method`` on task failure

2021-09-13 Thread GitBox
kaxil merged pull request #18210: URL: https://github.com/apache/airflow/pull/18210 -- 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-unsubs

[GitHub] [airflow] kaxil closed issue #18146: Deferred TI's `next_method` and `next_kwargs` not cleared on retries

2021-09-13 Thread GitBox
kaxil closed issue #18146: URL: https://github.com/apache/airflow/issues/18146 -- 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...

[GitHub] [airflow] github-actions[bot] commented on pull request #18210: Wipe ``next_kwargs`` and ``next_method`` on task failure

2021-09-13 Thread GitBox
github-actions[bot] commented on pull request #18210: URL: https://github.com/apache/airflow/pull/18210#issuecomment-918450562 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 tak

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #18212: Doc change: XCOM / Taskflow

2021-09-13 Thread GitBox
boring-cyborg[bot] commented on pull request #18212: URL: https://github.com/apache/airflow/pull/18212#issuecomment-918445126 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 Con

[GitHub] [airflow] fritz-astronomer opened a new pull request #18212: Doc change: XCOM / Taskflow

2021-09-13 Thread GitBox
fritz-astronomer opened a new pull request #18212: URL: https://github.com/apache/airflow/pull/18212 Added a description and code sample of going FROM normal Airflow Operators TO Taskflow Operators, passing in XCOMs -- This is an automated message from the Apache Git Service. To resp

[GitHub] [airflow] eladkal commented on issue #8568: Rewrite Google Cloud Build integration to use the native library.

2021-09-13 Thread GitBox
eladkal commented on issue #8568: URL: https://github.com/apache/airflow/issues/8568#issuecomment-918445061 Thank you @ryanyuan πŸ™ -- 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 com

[GitHub] [airflow] bmckallagat-os commented on pull request #18209: Upgrade importlib-resources version

2021-09-13 Thread GitBox
bmckallagat-os commented on pull request #18209: URL: https://github.com/apache/airflow/pull/18209#issuecomment-918441886 Checking the errors and will update accordingly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

  1   2   >