mik-laj opened a new pull request #17248:
URL: https://github.com/apache/airflow/pull/17248
These commands no longer work in Airflow 2.0, so we can drop support for
this.
---
**^ Add meaningful description above**
Read the **[Pull Request
uranusjr commented on issue #17240:
URL: https://github.com/apache/airflow/issues/17240#issuecomment-887229655
I think this is working as expected (that how `env` in the stdlib subprocess
works). Does the documentation says otherwise? We should fix the documentation
if that's the case.
github-actions[bot] commented on pull request #17245:
URL: https://github.com/apache/airflow/pull/17245#issuecomment-887223676
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-actions[bot] commented on pull request #17244:
URL: https://github.com/apache/airflow/pull/17244#issuecomment-887223609
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-actions[bot] commented on pull request #17242:
URL: https://github.com/apache/airflow/pull/17242#issuecomment-887223516
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
This is an automated email from the ASF dual-hosted git repository.
kamilbregula pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 87f408b Prepares docs for Rc2 release of July providers (#17116)
add 997f7d0 Update spark_kubernetes.py
boring-cyborg[bot] commented on pull request #17237:
URL: https://github.com/apache/airflow/pull/17237#issuecomment-887222595
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
mik-laj merged pull request #17237:
URL: https://github.com/apache/airflow/pull/17237
--
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:
github-actions[bot] commented on pull request #17211:
URL: https://github.com/apache/airflow/pull/17211#issuecomment-887221436
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
mik-laj commented on a change in pull request #17207:
URL: https://github.com/apache/airflow/pull/17207#discussion_r677127098
##
File path: airflow/cli/commands/task_command.py
##
@@ -174,6 +175,14 @@ def _run_raw_task(args, ti: TaskInstance) -> None:
)
+def
duynguyenhoang commented on issue #15572:
URL: https://github.com/apache/airflow/issues/15572#issuecomment-887214103
Our team had same issue with import. Finally we decided to move the `utils`
stuff to `plugins` out of `dags` folder and it works like a charm
You will see the
mik-laj commented on pull request #17201:
URL: https://github.com/apache/airflow/pull/17201#issuecomment-887213468
What do you think about adding system tests to these operators?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
mik-laj commented on a change in pull request #17201:
URL: https://github.com/apache/airflow/pull/17201#discussion_r677121790
##
File path: docs/apache-airflow-providers-alibabacloud/operators/oss.rst
##
@@ -0,0 +1,64 @@
+ .. Licensed to the Apache Software Foundation (ASF)
mik-laj commented on a change in pull request #17201:
URL: https://github.com/apache/airflow/pull/17201#discussion_r677121404
##
File path: docs/apache-airflow-providers-alibabacloud/operators/oss.rst
##
@@ -0,0 +1,64 @@
+ .. Licensed to the Apache Software Foundation (ASF)
mik-laj commented on a change in pull request #17201:
URL: https://github.com/apache/airflow/pull/17201#discussion_r677120851
##
File path: airflow/providers/alibabacloud/sensors/oss_key.py
##
@@ -0,0 +1,106 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
mik-laj commented on a change in pull request #17201:
URL: https://github.com/apache/airflow/pull/17201#discussion_r677120581
##
File path: airflow/providers/alibabacloud/provider.yaml
##
@@ -0,0 +1,52 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
mik-laj commented on a change in pull request #17201:
URL: https://github.com/apache/airflow/pull/17201#discussion_r677120045
##
File path: airflow/providers/alibabacloud/operators/oss_bucket.py
##
@@ -0,0 +1,122 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under
mik-laj commented on a change in pull request #17201:
URL: https://github.com/apache/airflow/pull/17201#discussion_r677118461
##
File path: airflow/providers/alibabacloud/hooks/oss.py
##
@@ -0,0 +1,441 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or
github-actions[bot] commented on pull request #17214:
URL: https://github.com/apache/airflow/pull/17214#issuecomment-887204978
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
subkanthi opened a new pull request #17247:
URL: https://github.com/apache/airflow/pull/17247
Fixes slackApiFileOperator so that both content and file can be uploaded.
closes: #ISSUE
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
baaj2109 edited a comment on issue #16903:
URL: https://github.com/apache/airflow/issues/16903#issuecomment-887132165
thank you for your help, actually I followed instructions from
https://medium.com/@ryanroline/installing-apache-airflow-on-windows-10-5247aa1249ef,
I will reinstall
baaj2109 commented on issue #16903:
URL: https://github.com/apache/airflow/issues/16903#issuecomment-887132165
thank you for your help, actually I followed instructions from
https://medium.com/@ryanroline/installing-apache-airflow-on-windows-10-5247aa1249ef,
I will reinstall airflow again
orossini commented on a change in pull request #17182:
URL: https://github.com/apache/airflow/pull/17182#discussion_r677026322
##
File path: airflow/providers/google/cloud/example_dags/example_dataflow.py
##
@@ -166,7 +166,7 @@
# [START howto_sensor_wait_for_job_status]
mcreenan commented on pull request #17210:
URL: https://github.com/apache/airflow/pull/17210#issuecomment-887112931
>
>
> Please rebase to latest `main` - we had some problems with `main` broken.
Those problem should be fixed now.
Done.
--
This is an automated message
naomiaro commented on issue #17243:
URL: https://github.com/apache/airflow/issues/17243#issuecomment-887112554
https://airflow.apache.org/index.yaml Index now includes 1.1.0 at least
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
github-actions[bot] commented on pull request #15469:
URL: https://github.com/apache/airflow/pull/15469#issuecomment-887109406
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-actions[bot] commented on pull request #15919:
URL: https://github.com/apache/airflow/pull/15919#issuecomment-887109387
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-actions[bot] closed pull request #15610:
URL: https://github.com/apache/airflow/pull/15610
--
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:
github-actions[bot] commented on pull request #15448:
URL: https://github.com/apache/airflow/pull/15448#issuecomment-887109418
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-actions[bot] commented on pull request #15935:
URL: https://github.com/apache/airflow/pull/15935#issuecomment-887109370
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
kaxil opened a new pull request #17245:
URL: https://github.com/apache/airflow/pull/17245
Since we have released 1.1.0 of the Chart, we will start working towards
1.2.0-rc1
---
**^ Add meaningful description above**
Read the **[Pull Request
kaxil opened a new pull request #17244:
URL: https://github.com/apache/airflow/pull/17244
We released it on 26th July 2021 instead of 25th
---
**^ Add meaningful description above**
Read the **[Pull Request
fatmumuhomer closed pull request #17236:
URL: https://github.com/apache/airflow/pull/17236
--
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:
boring-cyborg[bot] commented on issue #17243:
URL: https://github.com/apache/airflow/issues/17243#issuecomment-887097453
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
naomiaro opened a new issue #17243:
URL: https://github.com/apache/airflow/issues/17243
https://downloads.apache.org/airflow/helm-chart/
There is no more
https://downloads.apache.org/airflow/helm-chart/1.0.0/airflow-1.0.0.tgz, only
1.1.0.
However, there's no entry here for
kaxil merged pull request #457:
URL: https://github.com/apache/airflow-site/pull/457
--
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.
kaxilnaik pushed a change to branch chart-docs-1.1.0
in repository https://gitbox.apache.org/repos/asf/airflow-site.git.
discard 1092424 Add documentation for Apache Airflow Helm Chart 1.1.0
add c81b334 Add
kaxil opened a new pull request #457:
URL: https://github.com/apache/airflow-site/pull/457
Docs for Helm Chart 1.1.0
--
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
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch chart-docs-1.1.0
in repository https://gitbox.apache.org/repos/asf/airflow-site.git.
at 1092424 Add documentation for Apache Airflow Helm Chart 1.1.0
This branch includes the
kaxil opened a new pull request #17242:
URL: https://github.com/apache/airflow/pull/17242
Before:
```
❯ for f in ../../../airflow-dev/helm-chart/$RC/*; do svn cp $f ${$(basename
$f)/rc?/}; done
A airflow-1.1.0.tgz
A airflow-1.1.0.tgz.asc
A
Author: kaxilnaik
Date: Mon Jul 26 22:53:52 2021
New Revision: 49011
Log:
Remove old Helm Chart release: 1.0.0
Removed:
release/airflow/helm-chart/1.0.0/
Author: kaxilnaik
Date: Mon Jul 26 22:53:34 2021
New Revision: 49010
Log:
Release Airflow Helm Chart Check 1.1.0 from 1.1.0rc1
Added:
release/airflow/helm-chart/1.1.0/
release/airflow/helm-chart/1.1.0/airflow-1.1.0.tgz
- copied unchanged from r48955,
warrenstephens commented on issue #17241:
URL: https://github.com/apache/airflow/issues/17241#issuecomment-887040232
May be interesting and related:
```
$ kubectl logs annotateduglogit.e7d5501e50d644d8894934b6880613b7
[2021-07-26 21:24:26,515] {cli_action_loggers.py:105} WARNING -
mik-laj commented on a change in pull request #16571:
URL: https://github.com/apache/airflow/pull/16571#discussion_r676955747
##
File path: airflow/providers/amazon/aws/operators/eks.py
##
@@ -0,0 +1,797 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or
warrenstephens commented on issue #17241:
URL: https://github.com/apache/airflow/issues/17241#issuecomment-887036464
Hitting the "back" button on the UI, then shows the "log_it" task (rather
than "print_it"), and then running "log_it" will crash the pod, like this:
```
$ kubectl get
warrenstephens opened a new issue #17241:
URL: https://github.com/apache/airflow/issues/17241
**Apache Airflow version**:
Airflow version: 2.1.2
**Kubernetes version (if you are using kubernetes)** (use `kubectl
version`):
Client Version:
boring-cyborg[bot] commented on issue #17241:
URL: https://github.com/apache/airflow/issues/17241#issuecomment-887033146
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
omriAl opened a new issue #17240:
URL: https://github.com/apache/airflow/issues/17240
**Apache Airflow version**: 1.10.15
**Kubernetes version (if you are using kubernetes)** (use `kubectl version`):
`Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.6",
boring-cyborg[bot] commented on issue #17240:
URL: https://github.com/apache/airflow/issues/17240#issuecomment-887028975
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
ptrhck edited a comment on issue #17190:
URL: https://github.com/apache/airflow/issues/17190#issuecomment-887002384
Both, the task and execution role of the services have the following
statement:
```
AssumeRolePolicyDocument:
Statement:
- Effect:
ptrhck edited a comment on issue #17190:
URL: https://github.com/apache/airflow/issues/17190#issuecomment-887002384
Both, the task and execution role of the services have the following
statement:
```
AssumeRolePolicyDocument:
Statement:
- Effect:
mehmax commented on issue #17032:
URL: https://github.com/apache/airflow/issues/17032#issuecomment-887011820
Hi, I added a PR for this. Lets discuss:
https://github.com/apache/airflow/pull/17239
#17239
--
This is an automated message from the Apache Git Service.
To respond to the
ptrhck edited a comment on issue #17190:
URL: https://github.com/apache/airflow/issues/17190#issuecomment-887002384
Both, the task and execution role of the services have the following
statement:
```
AssumeRolePolicyDocument:
Statement:
- Effect:
mehmax opened a new pull request #17239:
URL: https://github.com/apache/airflow/pull/17239
UI displayed rendered template dicts and lists as a json.dump.
This will not add any line breaks.
Especially when handling templated files (e.g. sql) line breaks will make
the display more
boring-cyborg[bot] commented on pull request #17239:
URL: https://github.com/apache/airflow/pull/17239#issuecomment-887010648
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
ptrhck commented on issue #11057:
URL: https://github.com/apache/airflow/issues/11057#issuecomment-887003513
@ivica-k
I am wondering whehter my issue
https://github.com/apache/airflow/issues/17190 is related?
--
This is an automated message from the Apache Git Service.
To
ptrhck commented on issue #17190:
URL: https://github.com/apache/airflow/issues/17190#issuecomment-887002384
Both, the task and execution role of the services have the following
statement:
```
AssumeRolePolicyDocument:
Statement:
- Effect: Allow
Author: potiuk
Date: Mon Jul 26 20:16:56 2021
New Revision: 49007
Log:
Remove old artifacts for Airflow Providers 2021-07-26
Removed:
dev/airflow/providers/apache-airflow-providers-google-4.1.0.tar.gz
dev/airflow/providers/apache-airflow-providers-google-4.1.0.tar.gz.asc
Author: potiuk
Date: Mon Jul 26 20:14:59 2021
New Revision: 49006
Log:
Add artifacts for Airflow Providers 2021-07-26
Added:
dev/airflow/providers/apache-airflow-providers-airbyte-2.1.0.tar.gz (with
props)
dev/airflow/providers/apache-airflow-providers-airbyte-2.1.0.tar.gz.asc
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 9504cb5 Be verbose about failure to import airflow_local_settings
(#17195)
add 87f408b Prepares docs for Rc2
potiuk merged pull request #17116:
URL: https://github.com/apache/airflow/pull/17116
--
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 pull request #17211:
URL: https://github.com/apache/airflow/pull/17211#issuecomment-886984991
CC: @jedcunningham @kaxil WDYT ?
--
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
mik-laj commented on pull request #16248:
URL: https://github.com/apache/airflow/pull/16248#issuecomment-886980394
Updated
--
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.
kaxil commented on pull request #16248:
URL: https://github.com/apache/airflow/pull/16248#issuecomment-886977127
ping @mik-laj
--
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
mgorsk1 opened a new issue #17238:
URL: https://github.com/apache/airflow/issues/17238
**Apache Airflow version**:
**2.1.2**
**Kubernetes version (if you are using kubernetes)** (use `kubectl version`):
**v1.19.4**
**Environment**:
- **Cloud provider or
potiuk commented on issue #10860:
URL: https://github.com/apache/airflow/issues/10860#issuecomment-886974397
Great comment ! Thanks for summarising it @arkadiuszbach !
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
jedcunningham closed pull request #16741:
URL: https://github.com/apache/airflow/pull/16741
--
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:
jedcunningham opened a new pull request #16741:
URL: https://github.com/apache/airflow/pull/16741
Instead of immediately sending callbacks to be processed, wait until
after we commit so the dagrun.end_date is guaranteed to be there when
the callback runs.
--
This is an automated
potiuk commented on pull request #17231:
URL: https://github.com/apache/airflow/pull/17231#issuecomment-886967267
Useful for testing CI failures :)
--
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 change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 569659d Updating Amazon-AWS example DAGs to use XComArgs (#16868)
add 9504cb5 Be verbose about failure 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 0dbd0f4 Remove/refactor default_args pattern for miscellaneous
providers (#16872)
add 569659d Updating
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 b10ed95 Updating Docker example DAGs to use XComArgs (#16871)
add 0dbd0f4 Remove/refactor default_args pattern
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 a777537 Updating Jenkins example DAGs to use XComArgs (#16874)
add b10ed95 Updating Docker example DAGs to use
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 448e50b Updating Google Cloud example DAGs to use XComArgs (#16875)
add a777537 Updating Jenkins example DAGs
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 aaf44cc Updating miscellaneous Google example DAGs to use XComArgs
(#16876)
add 448e50b Updating Google Cloud
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 cd3307f fix string encoding when using xcom / json (#13536)
add aaf44cc Updating miscellaneous Google example
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 2de7793 Adds mssql version parameter to Breeze (#17234)
add cd3307f fix string encoding when using xcom / json
baryluk commented on pull request #17195:
URL: https://github.com/apache/airflow/pull/17195#issuecomment-886937047
@potiuk Thanks for your patience and quick review!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
potiuk merged pull request #17195:
URL: https://github.com/apache/airflow/pull/17195
--
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:
boring-cyborg[bot] commented on pull request #17195:
URL: https://github.com/apache/airflow/pull/17195#issuecomment-886936380
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
potiuk commented on pull request #17195:
URL: https://github.com/apache/airflow/pull/17195#issuecomment-886935895
seems like transient errors only. Merging.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
john-jac commented on pull request #16956:
URL: https://github.com/apache/airflow/pull/16956#issuecomment-886935332
Opened new PR https://github.com/apache/airflow/pull/17237 on correct 'main'
branch.
--
This is an automated message from the Apache Git Service.
To respond to the
john-jac opened a new pull request #17237:
URL: https://github.com/apache/airflow/pull/17237
** new PR in correct branch as response to
https://github.com/apache/airflow/pull/16956 **
Per
john-jac closed pull request #16956:
URL: https://github.com/apache/airflow/pull/16956
--
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:
github-actions[bot] commented on pull request #16868:
URL: https://github.com/apache/airflow/pull/16868#issuecomment-886921639
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
potiuk merged pull request #16868:
URL: https://github.com/apache/airflow/pull/16868
--
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 pull request #16872:
URL: https://github.com/apache/airflow/pull/16872
--
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:
ephraimbuddy commented on issue #13853:
URL: https://github.com/apache/airflow/issues/13853#issuecomment-886919517
I was able to reproduce this in 2.0.0 but not in 2.1.2. @oz-r can you
confirm?
--
This is an automated message from the Apache Git Service.
To respond to the message,
potiuk merged pull request #16871:
URL: https://github.com/apache/airflow/pull/16871
--
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 pull request #16874:
URL: https://github.com/apache/airflow/pull/16874
--
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 pull request #16875:
URL: https://github.com/apache/airflow/pull/16875
--
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:
github-actions[bot] commented on pull request #16875:
URL: https://github.com/apache/airflow/pull/16875#issuecomment-886916188
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
potiuk merged pull request #16876:
URL: https://github.com/apache/airflow/pull/16876
--
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:
boring-cyborg[bot] commented on pull request #13536:
URL: https://github.com/apache/airflow/pull/13536#issuecomment-886907121
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
potiuk merged pull request #13536:
URL: https://github.com/apache/airflow/pull/13536
--
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:
fatmumuhomer commented on issue #16364:
URL: https://github.com/apache/airflow/issues/16364#issuecomment-886904138
@uranusjr, @potiuk I created a pull request for this change -
https://github.com/apache/airflow/pull/17236
Can someone take a look when you have time? Also, it says I
boring-cyborg[bot] commented on pull request #17236:
URL: https://github.com/apache/airflow/pull/17236#issuecomment-886902332
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
fatmumuhomer opened a new pull request #17236:
URL: https://github.com/apache/airflow/pull/17236
closes: 16364 (Timeout is ambiguous in SSHHook and SSHOperator)
Add conn_timeout and cmd_timeout params to SSHOperator. Add conn_timeout
param to SSHHook. The legacy timeout param
potiuk commented on issue #17235:
URL: https://github.com/apache/airflow/issues/17235#issuecomment-886893971
Yeah. Smart merging of the fields is best approach. There are already quite
a few connections where the "extra" field is hidden (when available extra are
exhaustive) but I think it
josh-fell opened a new issue #17235:
URL: https://github.com/apache/airflow/issues/17235
**Apache Airflow version**:
2.1.0
**What happened**:
There are several hooks that still use optional parameters from the classic
`Extra` field. However, when creating the connection the
1 - 100 of 225 matches
Mail list logo
