uranusjr commented on PR #34914:
URL: https://github.com/apache/airflow/pull/34914#issuecomment-1763836444
Why? `isinstance` should work just fine.
--
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
uranusjr commented on code in PR #34905:
URL: https://github.com/apache/airflow/pull/34905#discussion_r1360187183
##
airflow/api_connexion/openapi/v1.yaml:
##
@@ -368,6 +368,8 @@ paths:
summary: Test a connection
description: |
Test a connection.
+
aritra24 commented on issue #34930:
URL: https://github.com/apache/airflow/issues/34930#issuecomment-1763821873
Tested out #32926, works as expected
--
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
abdurafeyf commented on issue #34953:
URL: https://github.com/apache/airflow/issues/34953#issuecomment-1763814494
Assign it to me. maybe i can help
--
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
tadeha commented on PR #876:
URL: https://github.com/apache/airflow-site/pull/876#issuecomment-176377
> You need to fix the static checks (pre-commit might fix it for you
automatically) seems there are spaces at the end of lines.
I made changes to the code by using the examples an
Aakcht opened a new issue, #34959:
URL: https://github.com/apache/airflow/issues/34959
### Description
Hi!
I think it would be useful to have a helm chart parameter that (if set) will
allow to overwrite DAGs mount path in airflow containers. Mount path is already
defined in `
boring-cyborg[bot] commented on issue #34958:
URL: https://github.com/apache/airflow/issues/34958#issuecomment-1763748353
Thanks for opening your first issue here! Be sure to follow the issue
template! If you are willing to raise PR to address this issue please do so, no
need to wait for ap
gky249 opened a new issue, #34958:
URL: https://github.com/apache/airflow/issues/34958
### Apache Airflow version
2.7.2
### What happened
Hi Guys,
All our dags which use the PythonVirtualEnvOperator and call a python
function using python_callable param, they fail
KsM0 commented on issue #34929:
URL: https://github.com/apache/airflow/issues/34929#issuecomment-1763735848
Hi. I had similar problem in the version 2.5.1. I set
`max_active_runs_per_dag = 1` in configs, but dags will be launched several
times simultaneously according to the schedule, w
zeotuan commented on PR #34936:
URL: https://github.com/apache/airflow/pull/34936#issuecomment-1763735782
> Is it possible to have a unit test for this function?
I added some tests
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
gbonazzoli commented on issue #34937:
URL: https://github.com/apache/airflow/issues/34937#issuecomment-1763709845
@hussein-awala
looking at the events collected by the cluster during helm install
```
kube@kube-console:~$ kubectl get events
--sort-by='.metadata.creationTimes
pankajkoti commented on code in PR #34905:
URL: https://github.com/apache/airflow/pull/34905#discussion_r1360086360
##
airflow/api_connexion/openapi/v1.yaml:
##
@@ -368,6 +368,8 @@ paths:
summary: Test a connection
description: |
Test a connection.
+
boring-cyborg[bot] commented on PR #34957:
URL: https://github.com/apache/airflow/pull/34957#issuecomment-1763695242
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 Contribution
michalsosn opened a new pull request, #34957:
URL: https://github.com/apache/airflow/pull/34957
Closes: #10385
This change adds the on_kill method to the
`DataprocInstantiateWorkflowTemplateOperator` and
`DataprocInstantiateInlineWorkflowTemplateOperator`. This metho
dirrao commented on issue #34877:
URL: https://github.com/apache/airflow/issues/34877#issuecomment-1763648593
@eladkal They are related. Its more close to #7542
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
HumanIearning commented on issue #34956:
URL: https://github.com/apache/airflow/issues/34956#issuecomment-1763595489
https://github.com/apache/airflow/assets/81360154/a5f8e665-418f-4d16-bba1-0a671cdc9932";>
I found where function x is used, but still don't know about BASE_DIR
--
Thi
boring-cyborg[bot] commented on issue #34956:
URL: https://github.com/apache/airflow/issues/34956#issuecomment-1763592732
Thanks for opening your first issue here! Be sure to follow the issue
template! If you are willing to raise PR to address this issue please do so, no
need to wait for ap
HumanIearning opened a new issue, #34956:
URL: https://github.com/apache/airflow/issues/34956
### What do you see as an issue?
https://github.com/apache/airflow/blob/main/airflow/example_dags/example_python_operator.py#L40C1-L44C9
https://github.com/apache/airflow/assets/813601
vchiapaikeo commented on PR #34862:
URL: https://github.com/apache/airflow/pull/34862#issuecomment-1763562897
@uranusjr , what are your thoughts on the previous comment? Sorry - forgot
to tag ya.
--
This is an automated message from the Apache Git Service.
To respond to the message, pleas
Adaverse commented on issue #34930:
URL: https://github.com/apache/airflow/issues/34930#issuecomment-1763557956
Tested #32779, it works as expected.
--
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 t
github-actions[bot] commented on PR #33915:
URL: https://github.com/apache/airflow/pull/33915#issuecomment-1763548158
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 you
github-actions[bot] commented on issue #34429:
URL: https://github.com/apache/airflow/issues/34429#issuecomment-1763548110
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 issue #34429: UI/UX Graph Display
URL: https://github.com/apache/airflow/issues/34429
--
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,
github-actions[bot] commented on issue #34558:
URL: https://github.com/apache/airflow/issues/34558#issuecomment-1763548102
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] commented on PR #33930:
URL: https://github.com/apache/airflow/pull/33930#issuecomment-1763548135
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 you
github-actions[bot] closed issue #34558: Circular dependency issue when trying
to locally test DAG
URL: https://github.com/apache/airflow/issues/34558
--
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
cjames23 commented on code in PR #34693:
URL: https://github.com/apache/airflow/pull/34693#discussion_r1359985505
##
airflow/providers/amazon/aws/hooks/opensearch.py:
##
@@ -0,0 +1,130 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor lic
syedahsn commented on PR #34908:
URL: https://github.com/apache/airflow/pull/34908#issuecomment-1763534714
@dstandish do you mind taking a look at this issue? I'm wondering if this is
the best way to solve the problem.
--
This is an automated message from the Apache Git Service.
To respon
hankehly commented on issue #34930:
URL: https://github.com/apache/airflow/issues/34930#issuecomment-1763530704
#33031 is working on amazon-8.8.0rc1 😄
Here are the commands I used to setup my environment / run my checks if it
helps anyone.
```shell
breeze
pip install apache-airf
utkarsharma2 commented on issue #34930:
URL: https://github.com/apache/airflow/issues/34930#issuecomment-1763519160
Checked my changes, and they seem to be working fine.
Thanks,
Utkarsh Sharma
On Sun, Oct 15, 2023 at 1:01 PM Owen Leung ***@***.***> wrote:
> checked
potiuk merged PR #34949:
URL: https://github.com/apache/airflow/pull/34949
--
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...@airflow.a
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 91986b0987 Consolidate the permissions name in SlaM
potiuk commented on PR #34905:
URL: https://github.com/apache/airflow/pull/34905#issuecomment-1763508250
Can you please also add the disclaimers from the release notes. Something
alongside this:
```
For security reasons, the test connection functionality is disabled by
default acr
potiuk commented on PR #32221:
URL: https://github.com/apache/airflow/pull/32221#issuecomment-1763507022
Should be rebased now. The issue should be fixed with 3 PRs I did during the
last few days.
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
potiuk commented on PR #33490:
URL: https://github.com/apache/airflow/pull/33490#issuecomment-1763504320
Not sure. It was really 2 minutes case/. This is UX issue really and I'd
defer to others who are more into the UI/UX part.
--
This is an automated message from the Apache Git Service.
potiuk commented on PR #34769:
URL: https://github.com/apache/airflow/pull/34769#issuecomment-1763503221
Can you please rebase.? It's indeed hard to test 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 abov
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 9f55fd6ede Update docs errors.rst - Mention sentry
potiuk merged PR #34912:
URL: https://github.com/apache/airflow/pull/34912
--
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...@airflow.a
akshaychitneni commented on code in PR #34840:
URL: https://github.com/apache/airflow/pull/34840#discussion_r1359945595
##
airflow/providers/papermill/operators/__init__.py:
##
@@ -15,3 +15,8 @@
# KIND, either express or implied. See the License for the
# specific language go
akshaychitneni commented on PR #34840:
URL: https://github.com/apache/airflow/pull/34840#issuecomment-1763496404
> The situation around Papermill is not good, this provider might be
suspended in the future (or even soon), see:
>
> * Comment [Remove Airflow 2.3 compat from `PapermillOp
akshaychitneni commented on code in PR #34840:
URL: https://github.com/apache/airflow/pull/34840#discussion_r1359945002
##
airflow/providers/papermill/operators/papermill.py:
##
@@ -20,14 +20,97 @@
from typing import TYPE_CHECKING, ClassVar, Collection, Sequence
import attr
pierrejeambrun merged PR #34954:
URL: https://github.com/apache/airflow/pull/34954
--
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...@a
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun 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 dd59e3e63e Remove sensitive information fro
eladkal commented on code in PR #34693:
URL: https://github.com/apache/airflow/pull/34693#discussion_r1359933438
##
airflow/providers/amazon/aws/hooks/opensearch.py:
##
@@ -0,0 +1,130 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor lice
JustMaris opened a new issue, #34955:
URL: https://github.com/apache/airflow/issues/34955
### Apache Airflow version
2.7.2
### What happened
After updating Airflow to 2.7.2, on the home page in the UI, the count in
"All/ Active/ Paused" shows a number of dag runs not the
Bisk1 commented on PR #34945:
URL: https://github.com/apache/airflow/pull/34945#issuecomment-1763474577
converting to draft becuase I still need to fix some unit tests
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
eladkal commented on PR #34705:
URL: https://github.com/apache/airflow/pull/34705#issuecomment-1763467597
🎉 🎉 🎉 🎉 🎉 🎉
--
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 un
eladkal merged PR #34705:
URL: https://github.com/apache/airflow/pull/34705
--
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...@airflow.
potiuk commented on issue #34748:
URL: https://github.com/apache/airflow/issues/34748#issuecomment-1763465728
> Although version 2.7.8 is for GCP, but it is still very old. The
requirements in the repo mention unsupported versions (2.4.1 / 1.0.0) . The
versions need an update either way to
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 1e9c807ed7 Clean local tags in tag_providers for ne
potiuk merged PR #34951:
URL: https://github.com/apache/airflow/pull/34951
--
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...@airflow.a
potiuk commented on PR #34714:
URL: https://github.com/apache/airflow/pull/34714#issuecomment-1763453451
sure
--
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
potiuk commented on PR #876:
URL: https://github.com/apache/airflow-site/pull/876#issuecomment-1763452852
You need to fix the static checks (pre-commit might fix it for you
automatically) seems there are spaces at the end of lines.
--
This is an automated message from the Apache Git Servi
hussein-awala opened a new pull request, #34954:
URL: https://github.com/apache/airflow/pull/34954
(no comment)
--
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 unsubscri
eladkal commented on code in PR #34951:
URL: https://github.com/apache/airflow/pull/34951#discussion_r1359910389
##
dev/provider_packages/tag_providers.sh:
##
@@ -35,6 +35,18 @@ do
{ git tag "${tag}" -m "Release $(date '+%Y-%m-%d') of providers" &&
tags+=("$tag") ; } || tr
amoghrajesh commented on code in PR #34839:
URL: https://github.com/apache/airflow/pull/34839#discussion_r1359909746
##
helm_tests/airflow_core/test_env.py:
##
@@ -0,0 +1,32 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreement
amoghrajesh commented on PR #34879:
URL: https://github.com/apache/airflow/pull/34879#issuecomment-1763442965
cc @jedcunningham for the question regarding the next chart release dates
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
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 bb81c45aeb Fix typo in error message when `render_content` method is
not implemented (#34952)
add 4dbec15bce Enha
potiuk commented on PR #34868:
URL: https://github.com/apache/airflow/pull/34868#issuecomment-1763442650
Cool!
--
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 unsubscrib
potiuk merged PR #34868:
URL: https://github.com/apache/airflow/pull/34868
--
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...@airflow.a
amoghrajesh commented on code in PR #34951:
URL: https://github.com/apache/airflow/pull/34951#discussion_r1359908651
##
dev/provider_packages/tag_providers.sh:
##
@@ -35,6 +35,18 @@ do
{ git tag "${tag}" -m "Release $(date '+%Y-%m-%d') of providers" &&
tags+=("$tag") ; } |
This is an automated email from the ASF dual-hosted git repository.
husseinawala 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 bb81c45aeb Fix typo in error message when `re
hussein-awala merged PR #34952:
URL: https://github.com/apache/airflow/pull/34952
--
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...@ai
potiuk commented on code in PR #34842:
URL: https://github.com/apache/airflow/pull/34842#discussion_r1359893878
##
CONTRIBUTORS_QUICK_START.rst:
##
@@ -302,18 +302,27 @@ Setting up Breeze
5. When you enter Breeze environment you should see prompt similar to
``root@e4756f6ac886
potiuk opened a new issue, #34953:
URL: https://github.com/apache/airflow/issues/34953
### Discussed in https://github.com/apache/airflow/discussions/24548
Originally posted by **don1uppa** June 16, 2022
### Apache Airflow version
2.2.5
### What happened
potiuk commented on code in PR #34349:
URL: https://github.com/apache/airflow/pull/34349#discussion_r1359888351
##
airflow/www/extensions/init_views.py:
##
@@ -308,5 +318,16 @@ def init_api_experimental(app):
"The authenticated user has full access.",
RemovedIn
potiuk commented on code in PR #34951:
URL: https://github.com/apache/airflow/pull/34951#discussion_r1359881765
##
dev/provider_packages/tag_providers.sh:
##
@@ -35,6 +35,18 @@ do
{ git tag "${tag}" -m "Release $(date '+%Y-%m-%d') of providers" &&
tags+=("$tag") ; } || tru
utkarsharma2 opened a new pull request, #34952:
URL: https://github.com/apache/airflow/pull/34952
(no comment)
--
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 unsubscrib
amoghrajesh commented on code in PR #34951:
URL: https://github.com/apache/airflow/pull/34951#discussion_r1359866734
##
dev/provider_packages/tag_providers.sh:
##
@@ -35,6 +35,18 @@ do
{ git tag "${tag}" -m "Release $(date '+%Y-%m-%d') of providers" &&
tags+=("$tag") ; } |
eladkal commented on code in PR #34951:
URL: https://github.com/apache/airflow/pull/34951#discussion_r1359865489
##
dev/provider_packages/tag_providers.sh:
##
@@ -35,6 +35,18 @@ do
{ git tag "${tag}" -m "Release $(date '+%Y-%m-%d') of providers" &&
tags+=("$tag") ; } || tr
potiuk commented on PR #34951:
URL: https://github.com/apache/airflow/pull/34951#issuecomment-1763366538
LGTM. @eladkal ?
--
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
amoghrajesh commented on PR #34951:
URL: https://github.com/apache/airflow/pull/34951#issuecomment-1763362313
@eladkal @ephraimbuddy handling this like this as per offline discussions.
Do let me know what you feel
cc @potiuk
--
This is an automated message from the Apache Git Serv
amoghrajesh opened a new pull request, #34951:
URL: https://github.com/apache/airflow/pull/34951
As reported on slack, sometimes due to network connectivity issues with
Github, our tag_providers.sh script creates tags locally with errors like:
```
➜ airflow git:(main)
zhenghanyang opened a new issue, #34950:
URL: https://github.com/apache/airflow/issues/34950
### Apache Airflow version
2.7.2
### What happened
When I set sql_alchemy_pool_enabled to false
https://airflow.apache.org/docs/apache-airflow/stable/configurations-ref.html#c
boring-cyborg[bot] commented on issue #34950:
URL: https://github.com/apache/airflow/issues/34950#issuecomment-1763344182
Thanks for opening your first issue here! Be sure to follow the issue
template! If you are willing to raise PR to address this issue please do so, no
need to wait for ap
eladkal commented on issue #34877:
URL: https://github.com/apache/airflow/issues/34877#issuecomment-1763328395
> So, this leads the scheduler to spend most of its time in this function
rather than scheduling the tasks
This is interesting.
I believe this is the K8s equivalent for
h
hussein-awala commented on issue #34929:
URL: https://github.com/apache/airflow/issues/34929#issuecomment-1763327067
We had a similar report in the past
(https://github.com/apache/airflow/issues/15588#issuecomment-1529520511) from a
user who had the problem until 2.5.1 too.
It is very li
jens-scheffler-bosch commented on issue #27657:
URL: https://github.com/apache/airflow/issues/27657#issuecomment-1763323254
This error report is stale since a while, I tried to follow the discussion
to clean up the issue backlog. I assume under the current release 2.7.2 there
had been many
hussein-awala commented on issue #34937:
URL: https://github.com/apache/airflow/issues/34937#issuecomment-1763323437
> pod/airflow-scheduler-77b484b6d5-hfjlz 0/2 Init:0/1 3 (50s ago)
Did you check why the scheduler (and the other services) pods restart? Do
you have any error
Owen-CH-Leung commented on issue #34930:
URL: https://github.com/apache/airflow/issues/34930#issuecomment-1763305955
checked `apache-airflow-providers-cncf-kubernetes==7.7.0rc1`. The fix is
working as expected.
--
This is an automated message from the Apache Git Service.
To respond to the
80 matches
Mail list logo
