potiuk commented on PR #29210:
URL: https://github.com/apache/airflow/pull/29210#issuecomment-1407588395
Needs tests fix as well.
--
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 commented on issue #22790:
URL: https://github.com/apache/airflow/issues/22790#issuecomment-1407587460
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
potiuk commented on code in PR #29203:
URL: https://github.com/apache/airflow/pull/29203#discussion_r1089881439
##
docs/apache-airflow/howto/docker-compose/index.rst:
##
@@ -68,6 +68,7 @@ This file contains several service definitions:
task instances once their dependencies
potiuk commented on PR #29116:
URL: https://github.com/apache/airflow/pull/29116#issuecomment-1407581892
> * Wait until we approve the tests (as you are first time contributor) and
somoene reviews it/merges it.
>
> I am sorry I wasn't clear. I meant what should I do about the test
Taragolis opened a new pull request, #29213:
URL: https://github.com/apache/airflow/pull/29213
Use MySQL backend instead of Postgres during `Tests / Integration Tests
MySQL` run
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
amoghrajesh commented on issue #22790:
URL: https://github.com/apache/airflow/issues/22790#issuecomment-1407560650
@potiuk @evanrosebrook saw that this issue has been idle for sometime. Can I
pick it up?
--
This is an automated message from the Apache Git Service.
To respond to the
yehoshuadimarsky commented on PR #27947:
URL: https://github.com/apache/airflow/pull/27947#issuecomment-1407547426
Also update the reverse transfer RS -> S3 with option to use the RS data api
--
This is an automated message from the Apache Git Service.
To respond to the message, please
yehoshuadimarsky commented on PR #27947:
URL: https://github.com/apache/airflow/pull/27947#issuecomment-1407542880
Ok, per comments above added the RS data api to the S3 -> RS transfer
operator
--
This is an automated message from the Apache Git Service.
To respond to the message, please
hussein-awala opened a new pull request, #29212:
URL: https://github.com/apache/airflow/pull/29212
closes: #29197
---
This PR fixes the origin url for the Trigger DAG w/config web page, where
currently, it redirects to the origin endpoint with the param dag_id, but for
some
hussein-awala commented on issue #29197:
URL: https://github.com/apache/airflow/issues/29197#issuecomment-1407520436
Thank you for opening this issue, I can reproduce the problem with 2.5.1 and
master.
Can someone assign it to me?
--
This is an automated message from the Apache Git
hussein-awala opened a new pull request, #29211:
URL: https://github.com/apache/airflow/pull/29211
closes: #29198
---
In the operators `SnowflakeCheckOperator`, `SnowflakeValueCheckOperator` and
`SnowflakeIntervalCheckOperator` we are extending some operators from the
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a commit to branch update-release-process
in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git
commit 7bb66b36f2fe549a758c242bf031c3e8af4f3c66
Author: pierrejeambrun
AuthorDate: Sun
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a change to branch update-release-process
in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git
at 7bb66b3 Use airflow_version in subject
This branch includes the following
pierrejeambrun commented on code in PR #30:
URL: https://github.com/apache/airflow-client-go/pull/30#discussion_r1089829149
##
.asf.yaml:
##
@@ -37,4 +37,9 @@ github:
enabled_merge_buttons:
squash: true
merge: false
-rebase: false
+rebase: true
Review
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a change to branch update-release-process
in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git
discard 31deca4 Update release process doc
add 9f6490d Update release process doc
pierrejeambrun commented on code in PR #30:
URL: https://github.com/apache/airflow-client-go/pull/30#discussion_r1089829149
##
.asf.yaml:
##
@@ -37,4 +37,9 @@ github:
enabled_merge_buttons:
squash: true
merge: false
-rebase: false
+rebase: true
Review
pierrejeambrun opened a new pull request, #30:
URL: https://github.com/apache/airflow-client-go/pull/30
Following: https://github.com/apache/airflow-client-python/pull/59 update in
a similar way the release process to include for go client:
- vote against sources
- uploading artifact
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a change to branch update-release-process
in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git
at 31deca4 Update release process doc
This branch includes the following new
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a commit to branch update-release-process
in repository https://gitbox.apache.org/repos/asf/airflow-client-go.git
commit 31deca497a6c7b6ce4282ce0b1fbdd92d0390d98
Author: pierrejeambrun
AuthorDate: Sun Jan
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 264ace5f86 SQLA20: Replace Query(pk).get by `Session.get` (#29205)
add a72e28d6e1 Remove misleading
jedcunningham merged PR #29169:
URL: https://github.com/apache/airflow/pull/29169
--
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:
maciejmochol opened a new issue, #61:
URL: https://github.com/apache/airflow-client-python/issues/61
Hello,
Version 2.5.0 of the client library stopped generating Authorization header
when sending HTTP requests to Airflow.
I did some debug of the code and I can see that eg. for
pierrejeambrun commented on PR #29098:
URL: https://github.com/apache/airflow/pull/29098#issuecomment-1407494178
Updated the PR and solved merge conflicts.
--
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 commented on issue #22689:
URL: https://github.com/apache/airflow/issues/22689#issuecomment-1407490415
I tried to build Docker image with HDFS provider on 2.5.1, it is installed
and Airflow started without any problem.
I believe that in some circumstances it is possible that
dnskr opened a new pull request, #29210:
URL: https://github.com/apache/airflow/pull/29210
The PR changes default `pgbouncer.auth_type` value to `scram-sha-256`.
It fixes the case when installing the chart with enabled `pgbouncer`, i.e.
the following deployment:
```
helm install
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 de7b8a2a50 Updating
Taragolis commented on code in PR #29203:
URL: https://github.com/apache/airflow/pull/29203#discussion_r1089812051
##
docs/apache-airflow/howto/docker-compose/index.rst:
##
@@ -68,6 +68,7 @@ This file contains several service definitions:
task instances once their
yxiao1996 commented on issue #28847:
URL: https://github.com/apache/airflow/issues/28847#issuecomment-1407485475
Took a look at the code, seems to me this should be a rather small change as
we have the capacity to query downstreams of a task in a dag already defined in
the dag data model
amarlearning commented on code in PR #29203:
URL: https://github.com/apache/airflow/pull/29203#discussion_r1089809614
##
docs/apache-airflow/howto/docker-compose/index.rst:
##
@@ -68,6 +68,7 @@ This file contains several service definitions:
task instances once their
Taragolis commented on code in PR #29203:
URL: https://github.com/apache/airflow/pull/29203#discussion_r1089806027
##
docs/apache-airflow/howto/docker-compose/index.rst:
##
@@ -68,6 +68,7 @@ This file contains several service definitions:
task instances once their
jvorenkamp opened a new issue, #29209:
URL: https://github.com/apache/airflow/issues/29209
### Apache Airflow Provider(s)
google
### Versions of Apache Airflow Providers
[apache-airflow-providers-google
boring-cyborg[bot] commented on issue #29209:
URL: https://github.com/apache/airflow/issues/29209#issuecomment-1407474399
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
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from eff677c418 Snowflake Provider - hide host from UI (#29208)
add 264ace5f86 SQLA20: Replace Query(pk).get by
kaxil merged PR #29205:
URL: https://github.com/apache/airflow/pull/29205
--
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:
kaxil commented on issue #28723:
URL: https://github.com/apache/airflow/issues/28723#issuecomment-1407473316
> I think we should support both 1.4 and 2.0 for at least one minor release,
preferrably much longer. Dependencies don’t tend to catch up very fast for this
kind of migrations, and
loulsb commented on PR #29116:
URL: https://github.com/apache/airflow/pull/29116#issuecomment-1407462128
* Wait until we approve the tests (as you are first time contributor) and
somoene reviews it/merges it.
I am sorry I wasn't clear. I meant what should I do about the test that
potiuk commented on PR #29116:
URL: https://github.com/apache/airflow/pull/29116#issuecomment-1407459585
> @potiuk what should I do next?
Wait until we approve the tests (as you are first time contributor) and
somoene reviews it/merges it. I just run the tests. Let's see. If you see
potiuk commented on issue #29031:
URL: https://github.com/apache/airflow/issues/29031#issuecomment-1407456368
I think there is no way someone will help if there are no new evidences that
we can see. The errors you have, indicate that when creating the POD in K8S
remote end closed the
loulsb commented on PR #29116:
URL: https://github.com/apache/airflow/pull/29116#issuecomment-1407445003
@potiuk what should I do next?
--
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 commented on code in PR #28338:
URL: https://github.com/apache/airflow/pull/28338#discussion_r1089778787
##
airflow/providers/amazon/aws/sensors/dynamodb.py:
##
@@ -0,0 +1,83 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
Taragolis commented on code in PR #28338:
URL: https://github.com/apache/airflow/pull/28338#discussion_r1089778787
##
airflow/providers/amazon/aws/sensors/dynamodb.py:
##
@@ -0,0 +1,83 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
Taragolis commented on PR #29205:
URL: https://github.com/apache/airflow/pull/29205#issuecomment-1407443549
Ok, lets do a full test. Just in case.
--
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.
taragolis pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 6c479437b1 enhance production guide with a few Argo specific
guidelines (#29078)
add eff677c418 Snowflake
Taragolis merged PR #29208:
URL: https://github.com/apache/airflow/pull/29208
--
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 #29171:
URL: https://github.com/apache/airflow/pull/29171#issuecomment-1407441679
Also usually when you see a failure (like building docs or static checks) -
the error message explain what you should do to replicate it locally (so just
follow the advises)
--
This
potiuk commented on code in PR #29171:
URL: https://github.com/apache/airflow/pull/29171#discussion_r1088001308
##
docs/docker-stack/changelog.rst:
##
@@ -46,6 +46,9 @@ here so that users affected can find the reason for the
changes.
potiuk commented on PR #29171:
URL: https://github.com/apache/airflow/pull/29171#issuecomment-1407441027
For any substantial changes, making a UI change is not nearly good enough.
You need a development environment.
However if you want to use GitHub UI you can use codespaces (they
dwreeves commented on PR #29208:
URL: https://github.com/apache/airflow/pull/29208#issuecomment-1407439560
Ah, woops, I meant to link to #24572.
--
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 PR #29207:
URL: https://github.com/apache/airflow/pull/29207#issuecomment-1407428169
> I propose the following solution:
Would you like to add a PR for that @dnskr ? I think what @jedcunningham is
trying to do is to prepare a new relese of the Helm Chart soon. I
Taragolis commented on PR #27841:
URL: https://github.com/apache/airflow/pull/27841#issuecomment-1407418388
I restart failed pipeline (on wait/build images). Let's have a look maybe it
pass now.
You could also join slack, if you not joined yet:
https://s.apache.org/airflow-slack.
dnskr commented on PR #29207:
URL: https://github.com/apache/airflow/pull/29207#issuecomment-1407403522
I recently upgraded Postgres instance, used by Airflow, from 11 to 15
version, so I would like to share my thoughts.
1. `pgbouncer.auth_type` shoud be `scram-sha-256` by default
magges commented on PR #29204:
URL: https://github.com/apache/airflow/pull/29204#issuecomment-1407394968
@ferruzzi sure, go for it!
I chose sql_default just because I didn't know better
--
This is an automated message from the Apache Git Service.
To respond to the message, please
Taragolis commented on PR #29195:
URL: https://github.com/apache/airflow/pull/29195#issuecomment-1407383640
> to my knowledge the function lacked tests to begin with and i was
wondering how to test this as it requires a backend db with DAG runs
Yeah it required db backend, if you
mlnsharma commented on code in PR #28981:
URL: https://github.com/apache/airflow/pull/28981#discussion_r1089715650
##
airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py:
##
@@ -168,6 +168,9 @@ class KubernetesPodOperator(BaseOperator):
:param labels: labels to
mlnsharma commented on code in PR #28981:
URL: https://github.com/apache/airflow/pull/28981#discussion_r1089713913
##
airflow/providers/cncf/kubernetes/utils/pod_manager.py:
##
@@ -264,15 +279,69 @@ def consume_logs(*, since_time: DateTime | None = None,
follow: bool = True)
ashb commented on PR #29205:
URL: https://github.com/apache/airflow/pull/29205#issuecomment-1407369282
Yeah, mostly because we have so many composite PKs, and using `get()` you
have to match the column order directly, which is error prone/hard to read.
(Plus for whatever reason
duke8585 commented on PR #29195:
URL: https://github.com/apache/airflow/pull/29195#issuecomment-1407346904
> @duke8585, thanks for contribution, however we also need tests for this
changes, for more details see:
>
> * [TESTING](https://github.com/apache/airflow/blob/main/TESTING.rst)
57 matches
Mail list logo
