alexott commented on code in PR #28950:
URL: https://github.com/apache/airflow/pull/28950#discussion_r590565
##
airflow/providers/databricks/sensors/databricks_sql.py:
##
@@ -0,0 +1,136 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
pateash commented on issue #28465:
URL: https://github.com/apache/airflow/issues/28465#issuecomment-1436491373
@eladkal , I would like to pick this up.
--
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
harishkrao commented on code in PR #28950:
URL: https://github.com/apache/airflow/pull/28950#discussion_r577966
##
airflow/providers/databricks/sensors/databricks_sql.py:
##
@@ -0,0 +1,136 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
eladkal commented on issue #28465:
URL: https://github.com/apache/airflow/issues/28465#issuecomment-1436479493
**The needed fix(s):**
1. `connection.extra` accepts only json while Jenkins conn attempts to set
boolean value. This needs to be changed. Probably to: `{"use_https": true}`
Taragolis commented on code in PR #29623:
URL: https://github.com/apache/airflow/pull/29623#discussion_r561157
##
airflow/providers/amazon/aws/hooks/base_aws.py:
##
@@ -312,19 +312,35 @@ def _get_web_identity_credential_fetcher(
base_session =
takersk commented on issue #29561:
URL: https://github.com/apache/airflow/issues/29561#issuecomment-1436439675
@tirkarthi
thank you I solved 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
talnagar commented on PR #29413:
URL: https://github.com/apache/airflow/pull/29413#issuecomment-1436416320
this feature looks amazing!
--
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
talnagar commented on PR #29042:
URL: https://github.com/apache/airflow/pull/29042#issuecomment-1436414683
thanks @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 go to the specific comment.
takersk commented on issue #29561:
URL: https://github.com/apache/airflow/issues/29561#issuecomment-1436408171
@tirkarthi
There is no value returned by task, exec_trino_query_pod task will run the
actual
==>
return KubernetesPodOperator(
name=task_id.replace("_", "-"),
Bowrna commented on issue #29405:
URL: https://github.com/apache/airflow/issues/29405#issuecomment-1436393744
i will start working on this today.
--
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
tirkarthi commented on issue #29561:
URL: https://github.com/apache/airflow/issues/29561#issuecomment-1436385102
Values returned by tasks should be JSON serializable. Please see comments
https://github.com/apache/airflow/issues/28492#issuecomment-1359564322 and
takersk commented on issue #29561:
URL: https://github.com/apache/airflow/issues/29561#issuecomment-1436378898
@tirkarthi cc: @hussein-awala
Thank you for answer. I tested it after changing task_id, but another error
is occurring. Can I check it?
dags code modiftyed
```
uranusjr opened a new pull request, #29625:
URL: https://github.com/apache/airflow/pull/29625
`importlib.metadata.distributions()` reads information from the actual
installations, which is a lot of IO that we can avoid by caching.
The benefit of this depends on how many packages you
amoghrajesh opened a new pull request, #29624:
URL: https://github.com/apache/airflow/pull/29624
The "workers" section in the values.yaml is only used to configure the
celery executor and celerykubernetes executors. We should split the common
section of the workers section into celery
This is an automated email from the ASF dual-hosted git repository.
eladkal pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 0604033829 Add Amazon Redshift-data to S3<>RS
eladkal merged PR #27947:
URL: https://github.com/apache/airflow/pull/27947
--
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:
tirkarthi commented on issue #29561:
URL: https://github.com/apache/airflow/issues/29561#issuecomment-1436341424
`example_task_kubernetes` is the function name decorated by `@task` . You
also seem to be passing the same value as task_id to `exec_trino_query_pod`
where if you are trying to
pgagnon opened a new pull request, #29623:
URL: https://github.com/apache/airflow/pull/29623
This PR implements a new `web_identity_token_loader` for the AWS connection
`AssumeRoleWithWebIdentity` authentication mechanism that fetches the access
token from a location on the Airflow
This is an automated email from the ASF dual-hosted git repository.
github-bot 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 6fd769aa4d Updating
amoghrajesh opened a new pull request, #29622:
URL: https://github.com/apache/airflow/pull/29622
As per the review comments here:
https://github.com/apache/airflow/pull/29270#pullrequestreview-1304890651, due
to this design, the upgrades might suffer. Fix them to be helm upgrade
This is an automated email from the ASF dual-hosted git repository.
uranusjr pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 7dd19731f2 Avoid importing executor during conf
uranusjr merged PR #29569:
URL: https://github.com/apache/airflow/pull/29569
--
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:
uranusjr commented on code in PR #29569:
URL: https://github.com/apache/airflow/pull/29569#discussion_r432584
##
tests/www/views/test_views_tasks.py:
##
@@ -492,7 +496,7 @@ def test_code_from_db_all_example_dags(admin_client):
amoghrajesh opened a new issue, #29621:
URL: https://github.com/apache/airflow/issues/29621
### Official Helm Chart version
1.8.0 (latest released)
### Apache Airflow version
2.5.0
### Kubernetes Version
v1.25.4
### Helm Chart configuration
boring-cyborg[bot] commented on issue #29621:
URL: https://github.com/apache/airflow/issues/29621#issuecomment-1436259360
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
amoghrajesh commented on code in PR #29270:
URL: https://github.com/apache/airflow/pull/29270#discussion_r417094
##
chart/templates/logs-persistent-volume-claim.yaml:
##
@@ -29,6 +29,10 @@ metadata:
{{- with .Values.labels }}
{{- toYaml . | nindent 4 }}
{{-
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 2d13d04064 Fix bitnami/postgresql default username
potiuk merged PR #29478:
URL: https://github.com/apache/airflow/pull/29478
--
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:
potiuk commented on PR #29357:
URL: https://github.com/apache/airflow/pull/29357#issuecomment-1436171637
I think this one needs documentation explaining usage of this feature.
Otherwise it won't be discoverable.
--
This is an automated message from the Apache Git Service.
To respond to
potiuk closed issue #29458: Identify logging statements from Airflow versus top
level code
URL: https://github.com/apache/airflow/issues/29458
--
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
potiuk commented on issue #29458:
URL: https://github.com/apache/airflow/issues/29458#issuecomment-1436167051
I think it should be done via convention. In your DAGs you can add specific
logger "filtered.out" for example and configue it in the way you want in
LOGGING_CONFIG:
michaelmicheal commented on PR #29441:
URL: https://github.com/apache/airflow/pull/29441#issuecomment-1436166837
@potiuk is this good to merge?
--
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
potiuk commented on issue #25060:
URL: https://github.com/apache/airflow/issues/25060#issuecomment-1436163207
Detailed logs and circumstance would be needed (see
https://github.com/apache/airflow/issues/25200 for example how detailed
information helped in fixing similar issue).
--
This
wonhyeongseo commented on PR #27841:
URL: https://github.com/apache/airflow/pull/27841#issuecomment-1436162381
@potiuk Thank you so much for helping! Hope you have a great day
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
goodseog commented on issue #29393:
URL: https://github.com/apache/airflow/issues/29393#issuecomment-1436162264
Same here :)
--
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.
potiuk closed issue #27715: Add pre-commit rule to validate using `urlsplit`
rather than `urlparse`
URL: https://github.com/apache/airflow/issues/27715
--
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
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new a99254ffd3 fix: add pre-commit to ban urlparse
potiuk merged PR #27841:
URL: https://github.com/apache/airflow/pull/27841
--
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:
potiuk commented on issue #12771:
URL: https://github.com/apache/airflow/issues/12771#issuecomment-1436161279
FYI. Suppport and POC on how to do the OpenTelemetry integration is already
aproved
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new cd01650192 Add ttlSecondsAfterFinished to
potiuk merged PR #29314:
URL: https://github.com/apache/airflow/pull/29314
--
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:
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 1677d80e65 Add deferrable mode to
potiuk merged PR #28618:
URL: https://github.com/apache/airflow/pull/28618
--
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:
potiuk commented on issue #29258:
URL: https://github.com/apache/airflow/issues/29258#issuecomment-1436158237
> @Taragolis Can we have somebody to assist on the banner timeout errors?
Its coming inspite of maintaining banner timeout parameters in Compute Engine
SSH Operator
Also if
potiuk commented on PR #29409:
URL: https://github.com/apache/airflow/pull/29409#issuecomment-1436157344
@ephraimbuddy ?
--
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
takersk commented on issue #29561:
URL: https://github.com/apache/airflow/issues/29561#issuecomment-1436153289
@hussein-awala
yes!!
```
import os
from datetime import datetime, timedelta
from airflow import DAG
from common.enum.enum import DagEnum,
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new c5d548b2d9 Include Rancher in Helm chart install
boring-cyborg[bot] commented on PR #28416:
URL: https://github.com/apache/airflow/pull/28416#issuecomment-1436153155
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 to
potiuk merged PR #28416:
URL: https://github.com/apache/airflow/pull/28416
--
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:
potiuk commented on PR #28626:
URL: https://github.com/apache/airflow/pull/28626#issuecomment-1436151044
> Can we leave it outside the scope of this PR?
Likely yes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
potiuk commented on PR #29042:
URL: https://github.com/apache/airflow/pull/29042#issuecomment-1436150282
Sure. You can build your own airflow package and install it. Look at the
INSTALL and https://github.com/apache/airflow/blob/main/INSTALL#L114 . You can
also use breeze CLI
potiuk commented on PR #27264:
URL: https://github.com/apache/airflow/pull/27264#issuecomment-1436149095
> @potiuk any updated on this?
No. Not yet. It's not the highest priority - but if someone would like to
pick it up from here in the meantime, that would be awesome.
--
This is
potiuk closed issue #29365: Airflow Workers Trying to Create Pods in Default
Namespace
URL: https://github.com/apache/airflow/issues/29365
--
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
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new e9e3f2c8c7 Missing timezone in next_dagrun in
boring-cyborg[bot] commented on PR #29362:
URL: https://github.com/apache/airflow/pull/29362#issuecomment-1436147137
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 to
potiuk merged PR #29362:
URL: https://github.com/apache/airflow/pull/29362
--
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:
potiuk merged PR #29146:
URL: https://github.com/apache/airflow/pull/29146
--
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:
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 0a4184e34c Add note about workaround to potential error with argparse
to hdfs (#29614)
add 2d3cc504db Remove
boring-cyborg[bot] commented on PR #29146:
URL: https://github.com/apache/airflow/pull/29146#issuecomment-1436145128
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 to
potiuk closed issue #29137: Add ability to access context in functions
decorated by task.sensor
URL: https://github.com/apache/airflow/issues/29137
--
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
potiuk closed issue #29339: Failing to execute successive Dynamic Task Group
Mapping
URL: https://github.com/apache/airflow/issues/29339
--
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
potiuk closed issue #28836: Airflow Scheduler Hangs After Failed K8 API Call
URL: https://github.com/apache/airflow/issues/28836
--
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-actions[bot] commented on PR #28265:
URL: https://github.com/apache/airflow/pull/28265#issuecomment-1436142743
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 you for
potiuk commented on PR #26269:
URL: https://github.com/apache/airflow/pull/26269#issuecomment-1436141544
> Hi is there any update on this PR, thanks!
Not really - as you see it's closed. But if you want to pick it up where the
original author left it - feel free to continue in a new
hussein-awala commented on issue #29555:
URL: https://github.com/apache/airflow/issues/29555#issuecomment-1436137871
Can you provide some code for the mapped task group? It's not clear to me
why would you map different TIs based on the batches list instead of mapping
the same TIs with
potiuk commented on issue #29275:
URL: https://github.com/apache/airflow/issues/29275#issuecomment-1436133430
Yes it would be interesting feature to add.
--
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
potiuk commented on issue #25297:
URL: https://github.com/apache/airflow/issues/25297#issuecomment-1436130977
I think if If you can provide reproducible case with logs @seub from the
last Airflow version, that would be good new issue. I guess the case is
different than explained originally
hussein-awala commented on code in PR #29270:
URL: https://github.com/apache/airflow/pull/29270#discussion_r335700
##
chart/templates/logs-persistent-volume-claim.yaml:
##
@@ -29,6 +29,10 @@ metadata:
{{- with .Values.labels }}
{{- toYaml . | nindent 4 }}
{{-
hussein-awala commented on issue #29557:
URL: https://github.com/apache/airflow/issues/29557#issuecomment-1436106024
Airflow raises `AirflowTaskTimeout` exception when the task timed out, and
your code can catch this exception and handle it if needed.
Since you have `[2023-02-15,
hussein-awala commented on issue #29582:
URL: https://github.com/apache/airflow/issues/29582#issuecomment-1436093181
> Cannot trigger the same dag_id within the same second
You cannot trigger the dag multiple time with the same execution date
because there are two unique constraint
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 0a4184e34c Add note about workaround to potential
potiuk closed issue #22689: HDFS provider causes TypeError: __init__() got an
unexpected keyword argument 'encoding'
URL: https://github.com/apache/airflow/issues/22689
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
potiuk merged PR #29614:
URL: https://github.com/apache/airflow/pull/29614
--
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:
hussein-awala commented on issue #29561:
URL: https://github.com/apache/airflow/issues/29561#issuecomment-1436086556
Can you add your dag code?
--
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
Taragolis closed pull request #29616: Refactor docker-compose quick start test
URL: https://github.com/apache/airflow/pull/29616
--
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.
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 79c07e3fc5 convert moment with timezone to UTC
Taragolis closed issue #29576: DateTimeSensorAsync breaks if target_time is
timezone-aware
URL: https://github.com/apache/airflow/issues/29576
--
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
Taragolis merged PR #29606:
URL: https://github.com/apache/airflow/pull/29606
--
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:
Taragolis opened a new pull request, #29616:
URL: https://github.com/apache/airflow/pull/29616
There is quite a few tests failed with the same error during the PR CI
pipeline
```console
> assert dag_state == "success"
E AssertionError:
dimberman commented on code in PR #29580:
URL: https://github.com/apache/airflow/pull/29580#discussion_r314542
##
airflow/providers/amazon/aws/secrets/secrets_manager.py:
##
@@ -84,12 +88,24 @@ class SecretsManagerBackend(BaseSecretsBackend,
LoggingMixin):
:param
dimberman commented on code in PR #29606:
URL: https://github.com/apache/airflow/pull/29606#discussion_r314236
##
airflow/triggers/temporal.py:
##
@@ -41,10 +41,8 @@ def __init__(self, moment: datetime.datetime):
# Make sure it's in UTC
elif moment.tzinfo
dimberman commented on PR #29609:
URL: https://github.com/apache/airflow/pull/29609#issuecomment-1436081501
@o-nikolas looks like there are a few failing tests. Once these are fixed
would be glad to merge.
--
This is an automated message from the Apache Git Service.
To respond to the
potiuk commented on PR #29614:
URL: https://github.com/apache/airflow/pull/29614#issuecomment-1436067561
It's a niche case but yes. We could detect 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
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from a21c17bc07 Specific use-case: adding packages via requirements.txt in
compose (#29598)
add fd000684d0 SSH
Taragolis closed issue #29282: Ssh connection extra parameter conn_timeout
doesn't work with ssh operator
URL: https://github.com/apache/airflow/issues/29282
--
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
Taragolis merged PR #29347:
URL: https://github.com/apache/airflow/pull/29347
--
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:
potiuk closed issue #29613: Importing CreateCustomContainerTrainingJobOperator
takes too much time
URL: https://github.com/apache/airflow/issues/29613
--
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
potiuk commented on PR #28968:
URL: https://github.com/apache/airflow/pull/28968#issuecomment-1436057533
> @potiuk @aamster following up on my last comment
I do not know by heart - I would have to go to details, and I think it's a
good exercise to do if you want to make sure the fix
Taragolis commented on issue #29613:
URL: https://github.com/apache/airflow/issues/29613#issuecomment-1436057506
Check it locally, most of the time spend during this import:
`from google.cloud.aiplatform.models import Model`
potiuk commented on PR #27841:
URL: https://github.com/apache/airflow/pull/27841#issuecomment-1436056893
You need rebase and resolve conflicts @wonhyeongseo -> this was likely
temporary issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, please
potiuk commented on PR #29347:
URL: https://github.com/apache/airflow/pull/29347#issuecomment-1436056370
@Taragolis - are you ok ? This one LGTM.
--
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
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 83cdd5031b Handling error on cluster policy itself (#29056)
add a21c17bc07 Specific use-case: adding packages via
potiuk merged PR #29598:
URL: https://github.com/apache/airflow/pull/29598
--
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:
potiuk commented on issue #22689:
URL: https://github.com/apache/airflow/issues/22689#issuecomment-1436055249
This is what #29614 does.
--
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
potiuk opened a new pull request, #29614:
URL: https://github.com/apache/airflow/pull/29614
Instead of removing or reimplementing hdfs provider - in the event of a
possibility to break argparse, we simply add a note in the provider
documentation to remove argprase if such problem happens.
Taragolis commented on PR #29511:
URL: https://github.com/apache/airflow/pull/29511#issuecomment-1436048654
@john-jac could you fix Static Checks, the only way in your case it is use
`pre-commit`, see [Static code
Aakcht commented on PR #29347:
URL: https://github.com/apache/airflow/pull/29347#issuecomment-1436048563
Looks like the unsuccessful check is not related to 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
Aakcht commented on code in PR #29347:
URL: https://github.com/apache/airflow/pull/29347#discussion_r290081
##
airflow/providers/ssh/hooks/ssh.py:
##
@@ -173,6 +182,14 @@ def __init__(
if "conn_timeout" in extra_options and self.conn_timeout is
None:
RachitSharma2001 commented on PR #29406:
URL: https://github.com/apache/airflow/pull/29406#issuecomment-1436047140
Hi everybody, it looks like all the tests pass now. I was wondering if I
could get some feedback on the approach I used for adding this feature. Does
the approach look good,
potiuk commented on PR #29012:
URL: https://github.com/apache/airflow/pull/29012#issuecomment-1436046117
So @snjypl are you going to work on it as explained by @jedcunningham ? Or
maybe you would like to create theissue to fix the manual behaviour if you are
not sure how to fix it?
--
1 - 100 of 152 matches
Mail list logo