potiuk commented on PR #29888:
URL: https://github.com/apache/airflow/pull/29888#issuecomment-1453119981
inlined all the constants 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 the
potiuk commented on PR #29888:
URL: https://github.com/apache/airflow/pull/29888#issuecomment-1453118275
> I wonder if we should just move all the underscore variables.
Yeah. I was on the fence with this one, but I realized this is far more than
our test's problem. Right now if
potiuk closed pull request #29888: Move help message to the google auth code
URL: https://github.com/apache/airflow/pull/29888
--
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.
shahar1 commented on issue #10596:
URL: https://github.com/apache/airflow/issues/10596#issuecomment-1453096326
I've been trying to resolve this issue, and it seems that:
- `GKEStartPodOperator` currently supports an impersonation chain. (#19518)
- `DataflowCreateJavaJobOperator` and
josh-fell opened a new pull request, #29892:
URL: https://github.com/apache/airflow/pull/29892
Closes: #29875
This PR introduces a new Airflow CLI command -- `airflow connections test
` -- to test a single, predefined Airflow connection.
_Successful test_
ephraimbuddy commented on code in PR #29832:
URL: https://github.com/apache/airflow/pull/29832#discussion_r1124058690
##
airflow/utils/task_group.py:
##
@@ -245,6 +245,8 @@ def add(self, task: DAGNode) -> None:
elif SetupTeardownContext.is_teardown:
if
ephraimbuddy opened a new pull request, #29891:
URL: https://github.com/apache/airflow/pull/29891
This implements the classic TaskGroup setup/teardown. Ensures that nested
TaskGroups are taken into account
--
This is an automated message from the Apache Git Service.
To respond to
uranusjr commented on code in PR #29850:
URL: https://github.com/apache/airflow/pull/29850#discussion_r1124054180
##
airflow/models/taskinstance.py:
##
@@ -139,6 +139,10 @@
from airflow.models.operator import Operator
from airflow.utils.task_group import
uranusjr commented on PR #29815:
URL: https://github.com/apache/airflow/pull/29815#issuecomment-1453013130
There are multiple db commits in the block and I wonder if the internal
state would be messed up if the db fails midway through and the entire block
got rerun.
--
This is an
uranusjr commented on code in PR #29689:
URL: https://github.com/apache/airflow/pull/29689#discussion_r1124049541
##
airflow/providers/google/cloud/hooks/cloud_build.py:
##
@@ -158,13 +159,17 @@ def create_build_without_waiting_for_result(
:param timeout: Optional, the
uranusjr commented on PR #29842:
URL: https://github.com/apache/airflow/pull/29842#issuecomment-1453008723
THe breakage sounds reasonable if we bump the GitHub provider to 3.0.0. So
the question would be, is having the kwarg default to true a good idea? Let’s
bump if it is.
--
This is
uranusjr commented on code in PR #29840:
URL: https://github.com/apache/airflow/pull/29840#discussion_r1124046647
##
airflow/providers/databricks/operators/databricks.py:
##
@@ -285,12 +285,12 @@ def __init__(
*,
json: Any | None = None,
tasks:
boring-cyborg[bot] commented on issue #29890:
URL: https://github.com/apache/airflow/issues/29890#issuecomment-1453001049
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
Narsimha1997 opened a new issue, #29890:
URL: https://github.com/apache/airflow/issues/29890
### Apache Airflow version
2.5.1
### What happened
Iam running airflow version 2.3.0 through docker and python version is
3.10.6, i installed dag-factory python module using pip
abin-tiger commented on issue #28751:
URL: https://github.com/apache/airflow/issues/28751#issuecomment-1452996944
We're facing the same issue when using KubernetesExecutor
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
ftaib commented on issue #27545:
URL: https://github.com/apache/airflow/issues/27545#issuecomment-1452938150
When will the new Helm Chart be available with this changes?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
ftaib commented on issue #27476:
URL: https://github.com/apache/airflow/issues/27476#issuecomment-1452930037
Hi, may I know when will the new Helm Chart will be available with the
changes above?
--
This is an automated message from the Apache Git Service.
To respond to the message,
uranusjr commented on PR #29888:
URL: https://github.com/apache/airflow/pull/29888#issuecomment-1452928059
I wonder if we should just move all the underscore variables.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
uranusjr commented on PR #29609:
URL: https://github.com/apache/airflow/pull/29609#issuecomment-1452912263
Scraping it sounds reasonable to me
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the
josh-fell commented on code in PR #29694:
URL: https://github.com/apache/airflow/pull/29694#discussion_r1123963291
##
airflow/providers/google/suite/hooks/drive.py:
##
@@ -243,8 +300,21 @@ def upload_file(
.create(body=file_metadata, media_body=media, fields="id",
agentdanger commented on issue #29444:
URL: https://github.com/apache/airflow/issues/29444#issuecomment-1452826643
I was able to solve this issue by installing 1.8 with the configured yaml
file.
My namespace for this implementation was colosseum.
I had a custom yaml filed
github-actions[bot] closed issue #17977: DagRun marked as success without
executing task
URL: https://github.com/apache/airflow/issues/17977
--
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
github-actions[bot] commented on issue #17977:
URL: https://github.com/apache/airflow/issues/17977#issuecomment-1452821207
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-actions[bot] closed pull request #24652: Add ``@task.snowpark`` decorator
URL: https://github.com/apache/airflow/pull/24652
--
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
hussein-awala commented on issue #29833:
URL: https://github.com/apache/airflow/issues/29833#issuecomment-1452785595
Can you extract the ti state from the DB for a task stuck in queued state?
```sql
SELECT *
FROM task_instance
WHERE dag_id='' AND task_id='' AND
execution_date=''
o-nikolas commented on PR #29609:
URL: https://github.com/apache/airflow/pull/29609#issuecomment-1452761063
Thoughts on the above anyone?
--
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
o-nikolas commented on PR #29469:
URL: https://github.com/apache/airflow/pull/29469#issuecomment-1452742994
@potiuk I think this one is now ready for another round of review
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
hussein-awala commented on code in PR #29809:
URL: https://github.com/apache/airflow/pull/29809#discussion_r1123866188
##
airflow/kubernetes/kube_client.py:
##
@@ -105,16 +107,24 @@ def get_kube_client(
if conf.getboolean("kubernetes_executor", "enable_tcp_keepalive"):
potiuk closed pull request #29888: Move help message to the google auth code
URL: https://github.com/apache/airflow/pull/29888
--
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 commented on PR #29888:
URL: https://github.com/apache/airflow/pull/29888#issuecomment-1452721246
Should fix failing "canary" main build.
--
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 opened a new pull request, #29888:
URL: https://github.com/apache/airflow/pull/29888
The "google-auth" 2.16.2 just released, removed the _HELP_MESSAGE that was
imported by google provider auth util thus failing the imports in canary builds
of ours attempting to upgrade to newver
raphaelauv commented on issue #27292:
URL: https://github.com/apache/airflow/issues/27292#issuecomment-1452712966
I din't found a working constraint for google-cloud-aiplatform for
apache-airflow-providers-google==8.10.0 with python 3.11
--
This is an automated message from the Apache
potiuk commented on issue #27292:
URL: https://github.com/apache/airflow/issues/27292#issuecomment-1452700803
Cool. Time to try 3.11 build back then.
--
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 closed issue #29828: JSON Schema Validation errors for Params prevent
DAG from importing and errors are not shown
URL: https://github.com/apache/airflow/issues/29828
--
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 issue #29828:
URL: https://github.com/apache/airflow/issues/29828#issuecomment-1452699881
Fixed in #29056 then. Closing as essentially duplicate of it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
raphaelauv commented on issue #27292:
URL: https://github.com/apache/airflow/issues/27292#issuecomment-1452686038
https://github.com/googleapis/python-translate/releases/tag/v3.10.0
oogle-cloud-translat :heavy_check_mark:
vandonr-amz commented on code in PR #28869:
URL: https://github.com/apache/airflow/pull/28869#discussion_r1123823559
##
airflow/providers/amazon/aws/hooks/appflow.py:
##
@@ -54,49 +50,35 @@ def conn(self) -> AppflowClient:
"""Get the underlying boto3 Appflow client
This is an automated email from the ASF dual-hosted git repository.
onikolas 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 76d8aaa836 Fix Amazon ECS Enums (#29871)
o-nikolas merged PR #29871:
URL: https://github.com/apache/airflow/pull/29871
--
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:
SamWheating commented on issue #29828:
URL: https://github.com/apache/airflow/issues/29828#issuecomment-1452638928
This was inadvertently fixed in
https://github.com/apache/airflow/pull/29056, where the `Except` in
`_process_modules` was modified to catch any exceptions.
Can
agentdanger commented on issue #29444:
URL: https://github.com/apache/airflow/issues/29444#issuecomment-1452631098
I'm caveating my "no issue" suggestion above. I'm currently working through
bugs on the git-sync but those could be my own doing. My work around was able
to solve the
agentdanger commented on issue #29444:
URL: https://github.com/apache/airflow/issues/29444#issuecomment-1452620409
I was able to resolve this issue.
Upgrading also refers to upgrading the existing 1.8 instance. You have to
follow @potiuk suggestion within the release notes.
SamWheating commented on issue #29844:
URL: https://github.com/apache/airflow/issues/29844#issuecomment-1452592062
Could you run `pip list` and share your results? I suspect that this is due
to an older version of the `google-cloud-compute` library.
--
This is an automated message from
agentdanger commented on issue #29444:
URL: https://github.com/apache/airflow/issues/29444#issuecomment-1452576191
I get the issue when I do a helm upgrade to install the custom values.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
agentdanger commented on issue #29444:
URL: https://github.com/apache/airflow/issues/29444#issuecomment-1452573578
I can recreate this issue by doing a fresh install using the 1.8 helm image
"my-airflow", in a fresh namespace on kubernetes, and customizing the
values.yaml to attempt to
ferruzzi commented on issue #29640:
URL: https://github.com/apache/airflow/issues/29640#issuecomment-1452530133
@antonio-antuan I am trying to reopen [the
ticket](https://github.com/boto/botocore/issues/2608) that Taragolis found on
the botocore side. It looks like it has been raised a
This is an automated email from the ASF dual-hosted git repository.
jedcunningham pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 372ea492e5 Update sftp_to_s3.rst (#29869)
add ab066b5823 Add note about k8s version support (#29884)
No
jedcunningham merged PR #29884:
URL: https://github.com/apache/airflow/pull/29884
--
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:
o-nikolas commented on PR #29772:
URL: https://github.com/apache/airflow/pull/29772#issuecomment-1452510296
Looks like there are still some failing tests @hussein-awala
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
o-nikolas merged PR #29869:
URL: https://github.com/apache/airflow/pull/29869
--
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.
onikolas 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 372ea492e5 Update sftp_to_s3.rst (#29869)
boring-cyborg[bot] commented on PR #29869:
URL: https://github.com/apache/airflow/pull/29869#issuecomment-1452505556
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
This is an automated email from the ASF dual-hosted git repository.
onikolas pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 9745032fa7 including airflow/example_dags/sql/sample.sql in
MANIFEST.in (#29883)
add faf792f634 Implement
o-nikolas merged PR #29822:
URL: https://github.com/apache/airflow/pull/29822
--
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:
SamWheating opened a new pull request, #29886:
URL: https://github.com/apache/airflow/pull/29886
While investigating https://github.com/apache/airflow/issues/29112 I found
another issue in the way that the file task handler checks file access:
potiuk commented on issue #29444:
URL: https://github.com/apache/airflow/issues/29444#issuecomment-1452466045
Yes. This is a known "expectation" for chart 1.8.0 and the way how to deal
with it is described in [release
PierreC1024 commented on code in PR #29835:
URL: https://github.com/apache/airflow/pull/29835#discussion_r1123625999
##
airflow/providers/google/cloud/hooks/bigquery_reservation.py:
##
@@ -0,0 +1,125 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
PierreC1024 commented on code in PR #29835:
URL: https://github.com/apache/airflow/pull/29835#discussion_r1123625999
##
airflow/providers/google/cloud/hooks/bigquery_reservation.py:
##
@@ -0,0 +1,125 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
potiuk commented on issue #18329:
URL: https://github.com/apache/airflow/issues/18329#issuecomment-1452458530
Feel free
--
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.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from b33b11bf9f Update ref anchor for env var link in Connection how-to doc
(#29816)
add 9745032fa7 including
potiuk merged PR #29883:
URL: https://github.com/apache/airflow/pull/29883
--
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:
SamWheating commented on issue #29112:
URL: https://github.com/apache/airflow/issues/29112#issuecomment-1452445056
I have been trying to replicate this issue this morning, here's what I did:
- set up a Kind cluster with a ReadWriteMany PVC (running on NFS)
- Deployed Airflow
bbovenzi opened a new pull request, #29885:
URL: https://github.com/apache/airflow/pull/29885
Add new filtering options in #29226 to the grid view filters and apply it to
both grid_data and graph_data endpoints.
Also, allow filtering tasks and resetting the root without a full reload
vincbeck commented on code in PR #29871:
URL: https://github.com/apache/airflow/pull/29871#discussion_r1123591998
##
airflow/providers/amazon/aws/hooks/ecs.py:
##
@@ -55,7 +55,24 @@ def should_retry_eni(exception: Exception):
return False
-class EcsClusterStates(str,
empire-sports commented on issue #29444:
URL: https://github.com/apache/airflow/issues/29444#issuecomment-1452416237
Commenting on this. I am upgrading from 1.7 to 1.8. I used a custom
values.yaml file with my implementation to enable the git-sync. I'm receiving
a similar error.
Taragolis commented on code in PR #29871:
URL: https://github.com/apache/airflow/pull/29871#discussion_r1123587735
##
airflow/providers/amazon/aws/hooks/ecs.py:
##
@@ -55,7 +55,24 @@ def should_retry_eni(exception: Exception):
return False
-class EcsClusterStates(str,
vincbeck commented on code in PR #29871:
URL: https://github.com/apache/airflow/pull/29871#discussion_r1123582113
##
airflow/providers/amazon/aws/hooks/ecs.py:
##
@@ -55,7 +55,24 @@ def should_retry_eni(exception: Exception):
return False
-class EcsClusterStates(str,
jedcunningham commented on code in PR #29878:
URL: https://github.com/apache/airflow/pull/29878#discussion_r1123583646
##
airflow/providers/mysql/CHANGELOG.rst:
##
@@ -27,6 +27,14 @@ used with MySQL server 5.6.4 through 5.7.
Changelog
-
+4.0.2
+.
+
+Misc
+
+
o-nikolas commented on code in PR #29871:
URL: https://github.com/apache/airflow/pull/29871#discussion_r1123582402
##
airflow/providers/amazon/aws/hooks/ecs.py:
##
@@ -55,7 +55,24 @@ def should_retry_eni(exception: Exception):
return False
-class EcsClusterStates(str,
jedcunningham closed pull request #29882: Drop support for k8s 1.23
URL: https://github.com/apache/airflow/pull/29882
--
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
ferruzzi commented on code in PR #28869:
URL: https://github.com/apache/airflow/pull/28869#discussion_r1123575711
##
airflow/providers/amazon/aws/hooks/appflow.py:
##
@@ -54,49 +50,35 @@ def conn(self) -> AppflowClient:
"""Get the underlying boto3 Appflow client
ferruzzi commented on code in PR #28869:
URL: https://github.com/apache/airflow/pull/28869#discussion_r1123575711
##
airflow/providers/amazon/aws/hooks/appflow.py:
##
@@ -54,49 +50,35 @@ def conn(self) -> AppflowClient:
"""Get the underlying boto3 Appflow client
jedcunningham commented on PR #29882:
URL: https://github.com/apache/airflow/pull/29882#issuecomment-1452395447
Okay, opened #29884 so it's more visible!
--
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
jedcunningham opened a new pull request, #29884:
URL: https://github.com/apache/airflow/pull/29884
This will help prevent us accidentally dropping support for k8s versions
earlier than we've agreed to it.
--
This is an automated message from the Apache Git Service.
To respond to the
eladkal commented on code in PR #29878:
URL: https://github.com/apache/airflow/pull/29878#discussion_r1123572510
##
airflow/providers/mysql/CHANGELOG.rst:
##
@@ -27,6 +27,14 @@ used with MySQL server 5.6.4 through 5.7.
Changelog
-
+4.0.2
+.
+
+Misc
+
+
+*
o-nikolas commented on PR #29869:
URL: https://github.com/apache/airflow/pull/29869#issuecomment-1452382584
> docs link in the PR description is old (1.10.3). The new docs is
potiuk commented on PR #29882:
URL: https://github.com/apache/airflow/pull/29882#issuecomment-1452381931
> Hmm, I wasn't aware of the k8s exception. Let me have a think on that.
Fwiw, as of today 2 support 1.22 still.
Yep. I noticed :)
--
This is an automated message from the
bbovenzi closed issue #23169: Task Duration shows as "empty" when it was really
less than 1 second
URL: https://github.com/apache/airflow/issues/23169
--
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
bbovenzi commented on issue #23169:
URL: https://github.com/apache/airflow/issues/23169#issuecomment-1452381187
https://user-images.githubusercontent.com/4600967/222523140-4728de05-80fa-40ca-b61b-aa2e9f54e459.png;>
We got rid of the tree view a long time ago and we know calculate
jedcunningham commented on code in PR #29878:
URL: https://github.com/apache/airflow/pull/29878#discussion_r1123558759
##
airflow/providers/mysql/CHANGELOG.rst:
##
@@ -27,6 +27,14 @@ used with MySQL server 5.6.4 through 5.7.
Changelog
-
+4.0.2
+.
+
+Misc
+
+
amoghrajesh commented on issue #23169:
URL: https://github.com/apache/airflow/issues/23169#issuecomment-1452354467
@eladkal @bbovenzi I can take this issue if nobody is working on it. Seems
like an interesting one.
--
This is an automated message from the Apache Git Service.
To respond
pankajastro commented on code in PR #29835:
URL: https://github.com/apache/airflow/pull/29835#discussion_r1123551136
##
airflow/providers/google/cloud/hooks/bigquery_reservation.py:
##
@@ -0,0 +1,125 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
amoghrajesh commented on issue #18329:
URL: https://github.com/apache/airflow/issues/18329#issuecomment-1452345107
@potiuk seems like the solution was not completed here. I would like to try
my hand at it if it is available.
--
This is an automated message from the Apache Git Service.
To
PierreC1024 commented on code in PR #29835:
URL: https://github.com/apache/airflow/pull/29835#discussion_r1123543321
##
airflow/providers/google/cloud/hooks/bigquery_reservation.py:
##
@@ -0,0 +1,125 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
PierreC1024 commented on code in PR #29835:
URL: https://github.com/apache/airflow/pull/29835#discussion_r1123543321
##
airflow/providers/google/cloud/hooks/bigquery_reservation.py:
##
@@ -0,0 +1,125 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
SamWheating opened a new pull request, #29883:
URL: https://github.com/apache/airflow/pull/29883
I was debugging another issue and running the official helm chart in Kind,
and when I tried to run the `example_python_operator` DAG it failed on the
`log_sql_query` task with the following
pankajastro commented on code in PR #29835:
URL: https://github.com/apache/airflow/pull/29835#discussion_r1123530878
##
airflow/providers/google/cloud/hooks/bigquery_reservation.py:
##
@@ -0,0 +1,125 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
ferruzzi commented on code in PR #29871:
URL: https://github.com/apache/airflow/pull/29871#discussion_r1123531685
##
airflow/providers/amazon/aws/hooks/ecs.py:
##
@@ -55,7 +55,24 @@ def should_retry_eni(exception: Exception):
return False
-class EcsClusterStates(str,
Taragolis commented on code in PR #29871:
URL: https://github.com/apache/airflow/pull/29871#discussion_r1123531262
##
airflow/providers/amazon/aws/hooks/ecs.py:
##
@@ -55,7 +55,24 @@ def should_retry_eni(exception: Exception):
return False
-class EcsClusterStates(str,
ferruzzi commented on PR #29871:
URL: https://github.com/apache/airflow/pull/29871#issuecomment-1452321027
Thanks for getting this so fast, I was planning to look at it this morning.
:+1:
LOL. I did a review and it wouldn't let me submit because Niko left the
same two comments
vincbeck commented on code in PR #28869:
URL: https://github.com/apache/airflow/pull/28869#discussion_r1123525633
##
airflow/providers/amazon/aws/hooks/appflow.py:
##
@@ -54,49 +50,35 @@ def conn(self) -> AppflowClient:
"""Get the underlying boto3 Appflow client
vincbeck commented on code in PR #28869:
URL: https://github.com/apache/airflow/pull/28869#discussion_r1123525633
##
airflow/providers/amazon/aws/hooks/appflow.py:
##
@@ -54,49 +50,35 @@ def conn(self) -> AppflowClient:
"""Get the underlying boto3 Appflow client
o-nikolas commented on code in PR #29871:
URL: https://github.com/apache/airflow/pull/29871#discussion_r1123521462
##
airflow/providers/amazon/aws/hooks/ecs.py:
##
@@ -55,7 +55,24 @@ def should_retry_eni(exception: Exception):
return False
-class EcsClusterStates(str,
josh-fell merged PR #29816:
URL: https://github.com/apache/airflow/pull/29816
--
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.
joshfell 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 b33b11bf9f Update ref anchor for env var link in
josh-fell commented on PR #29816:
URL: https://github.com/apache/airflow/pull/29816#issuecomment-1452306725
> We can merge but I guess since releases are not synced there will be some
time that the reference won't work
I think that's fine. It's the only provider that uses this link
vandonr-amz commented on code in PR #28869:
URL: https://github.com/apache/airflow/pull/28869#discussion_r1123512646
##
airflow/providers/amazon/aws/hooks/appflow.py:
##
@@ -54,49 +50,35 @@ def conn(self) -> AppflowClient:
"""Get the underlying boto3 Appflow client
shyft-mike commented on code in PR #28846:
URL: https://github.com/apache/airflow/pull/28846#discussion_r1123510383
##
airflow/compat/hashlib_wrapper.py:
##
@@ -0,0 +1,45 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license
jedcunningham commented on PR #29882:
URL: https://github.com/apache/airflow/pull/29882#issuecomment-1452270108
Hmm, I wasn't aware of the k8s exception. Let me have a think on that. Fwiw,
as of today 2 support 1.22 still.
--
This is an automated message from the Apache Git Service.
To
potiuk commented on PR #29882:
URL: https://github.com/apache/airflow/pull/29882#issuecomment-1452269709
(And yes, I would be happy to drop support immediately after they reach EOL,
but it's not what we agreed to, unfortunately).
--
This is an automated message from the Apache Git
1 - 100 of 195 matches
Mail list logo