potiuk merged PR #40802:
URL: https://github.com/apache/airflow/pull/40802
--
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 0f4884c3ca [AIP-49] OpenTelemetry Traces for Apache
shahar1 commented on issue #40889:
URL: https://github.com/apache/airflow/issues/40889#issuecomment-2240985980
> Looks like this PR #38512 would solve the issue.
I assigned you this issue as you indicated it in the checkbox.
Feel free to
> Looks like this PR #38512 would so
shahar1 closed issue #40890: Breeze svg not working in the developer tasks doc
URL: https://github.com/apache/airflow/issues/40890
--
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
jscheffl commented on code in PR #40897:
URL: https://github.com/apache/airflow/pull/40897#discussion_r1685297405
##
airflow/api_internal/internal_api_call.py:
##
@@ -68,21 +73,31 @@ def get_internal_api_endpoint():
InternalApiConfig._init_values()
return I
potiuk commented on code in PR #40897:
URL: https://github.com/apache/airflow/pull/40897#discussion_r1685302555
##
airflow/api_internal/internal_api_call.py:
##
@@ -68,21 +73,31 @@ def get_internal_api_endpoint():
InternalApiConfig._init_values()
return Int
potiuk commented on code in PR #40897:
URL: https://github.com/apache/airflow/pull/40897#discussion_r1685304307
##
airflow/api_internal/internal_api_call.py:
##
@@ -68,21 +73,31 @@ def get_internal_api_endpoint():
InternalApiConfig._init_values()
return Int
jscheffl commented on code in PR #40897:
URL: https://github.com/apache/airflow/pull/40897#discussion_r1685306150
##
airflow/api_internal/internal_api_call.py:
##
@@ -68,21 +73,31 @@ def get_internal_api_endpoint():
InternalApiConfig._init_values()
return I
eladkal commented on PR #40270:
URL: https://github.com/apache/airflow/pull/40270#issuecomment-2241038732
@jx2lee can you add the unit test?
--
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 sp
jscheffl commented on code in PR #40897:
URL: https://github.com/apache/airflow/pull/40897#discussion_r1685320255
##
airflow/api_internal/internal_api_call.py:
##
@@ -68,21 +73,31 @@ def get_internal_api_endpoint():
InternalApiConfig._init_values()
return I
jx2lee commented on PR #40270:
URL: https://github.com/apache/airflow/pull/40270#issuecomment-2241049385
@amoghrajesh @eladkal
yes, i've written unit test, but having a little problem in code. I'll push
soon.
--
This is an automated message from the Apache Git Service.
To respond to t
eladkal commented on PR #39031:
URL: https://github.com/apache/airflow/pull/39031#issuecomment-2241067587
cc @jedcunningham as you marked request changes
--
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 t
jscheffl opened a new pull request, #40899:
URL: https://github.com/apache/airflow/pull/40899
During the implementation of authentication protection for AIP-69 I realized
that Internal API does not carry support for authentication access.
Due to review of #40897 - alternative PR using
jscheffl commented on code in PR #40897:
URL: https://github.com/apache/airflow/pull/40897#discussion_r1685412727
##
airflow/api_internal/internal_api_call.py:
##
@@ -68,21 +73,31 @@ def get_internal_api_endpoint():
InternalApiConfig._init_values()
return I
Seoji commented on issue #39002:
URL: https://github.com/apache/airflow/issues/39002#issuecomment-2241107330
I think you are using
[DatabricksRunNowOperator](https://github.com/apache/airflow/blob/0f4884c3ca26c39bc6bd21967c950e718589fcd6/airflow/providers/databricks/operators/databricks.py#L
jscheffl opened a new pull request, #40900:
URL: https://github.com/apache/airflow/pull/40900
This PR is a DRAFT to further get to a MVP release of AIP-69. DO NOT MERGE
(yet) - but please feedback on it if you like.
It is a continuation of PR #40224 which was a kind of PoC and step-by
jedcunningham merged PR #40898:
URL: https://github.com/apache/airflow/pull/40898
--
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
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 0f4884c3ca [AIP-49] OpenTelemetry Traces for Apache Airflow Part 2
(#40802)
add 1819c858ce Chart: Helm cha
jscheffl opened a new pull request, #40901:
URL: https://github.com/apache/airflow/pull/40901
In order to leverage the AIP-44 interface for AIP-69 / Remote Executor I
need some small adjustments to "hook into" the RPC API. This PR extracts from
PoC implementation of AIP-69 (#40900) the need
boring-cyborg[bot] commented on PR #39031:
URL: https://github.com/apache/airflow/pull/39031#issuecomment-2241147826
Awesome work, congrats on your first merged pull request! You are invited to
check our [Issue Tracker](https://github.com/apache/airflow/issues) for
additional contributions.
jedcunningham commented on PR #39031:
URL: https://github.com/apache/airflow/pull/39031#issuecomment-2241148031
Thanks @rasulkarimov! Sorry for the slow merge. Congrats on your first
commit 🎉
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
jedcunningham merged PR #39031:
URL: https://github.com/apache/airflow/pull/39031
--
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
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 1819c858ce Chart: Helm chart 1.15.0 release notes (#40898)
add c2a54efefb Chart: Add extraEnvFrom to git-sy
potiuk commented on PR #40899:
URL: https://github.com/apache/airflow/pull/40899#issuecomment-2241149045
I looked in detail, and (sorry for that I was just quickly looking it up -
but the concept is similar).
The actual code I think we should model it from is similar but adds a bit
potiuk commented on PR #40899:
URL: https://github.com/apache/airflow/pull/40899#issuecomment-2241149264
(and I am quickly fixing the selective check error)
--
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 closed pull request #37893: make default helm annotations
optional on extra cm and secrets
URL: https://github.com/apache/airflow/pull/37893
--
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 g
jedcunningham commented on PR #37893:
URL: https://github.com/apache/airflow/pull/37893#issuecomment-2241149516
Thanks for the PR @sudeepgupta90! This has been added in #40294.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
boring-cyborg[bot] commented on issue #40902:
URL: https://github.com/apache/airflow/issues/40902#issuecomment-2241150236
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
gsingh935 opened a new issue, #40902:
URL: https://github.com/apache/airflow/issues/40902
### Apache Airflow version
Other Airflow 2 version (please specify below)
### If "Other Airflow 2 version" selected, which one?
2.9.2
### What happened?
CVE 2023-44487
jedcunningham opened a new pull request, #40903:
URL: https://github.com/apache/airflow/pull/40903
One last minute addition!
--
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 opened a new pull request, #40904:
URL: https://github.com/apache/airflow/pull/40904
We were removing "Providers" from the list of candidates and handled the key
non-existing but there were a path when splitting to individual provider tests
for "compatibility checks" was failing.
potiuk commented on PR #40881:
URL: https://github.com/apache/airflow/pull/40881#issuecomment-2241159980
I separated out the fix in Breeze @omkar-foss (and did it slightly
differently and added tests covering more cases).
https://github.com/apache/airflow/pull/40904
--
This is an
potiuk commented on PR #40881:
URL: https://github.com/apache/airflow/pull/40881#issuecomment-2241160017
( I made you co-author)
--
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 comm
potiuk commented on PR #40899:
URL: https://github.com/apache/airflow/pull/40899#issuecomment-2241160179
Fix to selective checks failing here:
https://github.com/apache/airflow/pull/40904
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
Seoji commented on issue #40787:
URL: https://github.com/apache/airflow/issues/40787#issuecomment-2241169890
I think it looks fine
other Sensor, something like databricks, mongo, redis work same as S3.
they made their own hooks in poke method.
if it is wrong, we should change whold
jx2lee commented on code in PR #40270:
URL: https://github.com/apache/airflow/pull/40270#discussion_r1685457830
##
helm_tests/other/test_redis.py:
##
@@ -185,6 +185,32 @@ def test_redis_by_chart_password_secret_name(self,
executor):
self.assert_broker_url_env(k8s_obj
jx2lee commented on code in PR #40270:
URL: https://github.com/apache/airflow/pull/40270#discussion_r1685458025
##
helm_tests/other/test_redis.py:
##
@@ -143,7 +143,9 @@ def test_redis_by_chart_password(self, executor):
self.assert_broker_url_env(k8s_obj_by_key)
potiuk merged PR #40904:
URL: https://github.com/apache/airflow/pull/40904
--
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 f099bea9fa Fix failure of selective checks when onl
potiuk commented on PR #40881:
URL: https://github.com/apache/airflow/pull/40881#issuecomment-2241173928
You can rebase on top of main and remove your changes - it should work no2
@omkar-foss
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
potiuk commented on PR #40901:
URL: https://github.com/apache/airflow/pull/40901#issuecomment-2241174825
I rebased it after I fixed selective checks in #40904
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL ab
potiuk commented on code in PR #40901:
URL: https://github.com/apache/airflow/pull/40901#discussion_r1685462090
##
airflow/api_internal/internal_api_call.py:
##
@@ -75,10 +88,13 @@ def _init_values():
raise RuntimeError("The AIP_44 is not enabled so you cannot use
raphaelauv commented on issue #40787:
URL: https://github.com/apache/airflow/issues/40787#issuecomment-2241184821
It's S3keySensor not S3CredsSensor.
It's about checking if data is present or not in the bucket . soft_fail
means and only means in case of timeout and no data then the op
jscheffl commented on code in PR #40901:
URL: https://github.com/apache/airflow/pull/40901#discussion_r1685475837
##
airflow/api_internal/internal_api_call.py:
##
@@ -75,10 +88,13 @@ def _init_values():
raise RuntimeError("The AIP_44 is not enabled so you cannot use
jscheffl commented on PR #40897:
URL: https://github.com/apache/airflow/pull/40897#issuecomment-2241191209
Closing in favor of #40899
--
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
jscheffl closed pull request #40897: Add support for basic authentication on
internal API client
URL: https://github.com/apache/airflow/pull/40897
--
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
Seoji commented on issue #40787:
URL: https://github.com/apache/airflow/issues/40787#issuecomment-2241196758
I understood, thankyou for your explanation.
I miss understood purpose of soft_fail.
https://github.com/apache/airflow/pull/33401
I think this PR make this side effect
tha
gyli opened a new pull request, #40905:
URL: https://github.com/apache/airflow/pull/40905
- RST formatting fixes
- Flag name fixes:
- `--exclude-parallel-test-types` -> `--excluded-parallel-test-types`
- `--paralleltest-types` -> `--parallel-test-types`
potiuk commented on code in PR #40901:
URL: https://github.com/apache/airflow/pull/40901#discussion_r1685505660
##
airflow/api_internal/internal_api_call.py:
##
@@ -75,10 +88,13 @@ def _init_values():
raise RuntimeError("The AIP_44 is not enabled so you cannot use
mr1holmes commented on PR #40705:
URL: https://github.com/apache/airflow/pull/40705#issuecomment-2241235042
Please review
--
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 #40894:
URL: https://github.com/apache/airflow/pull/40894#discussion_r1685506885
##
airflow/serialization/pydantic/taskinstance.py:
##
@@ -458,9 +458,9 @@ def schedule_downstream_tasks(self, session: Session | None
= None, max_tis_per_
potiuk merged PR #40894:
URL: https://github.com/apache/airflow/pull/40894
--
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 change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from f099bea9fa Fix failure of selective checks when only API has been
modified (#40904)
add 6684481c67 AIP-44 make da
potiuk closed issue #40902: CVE 2023-50387 in airflow image 2.9.2
URL: https://github.com/apache/airflow/issues/40902
--
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 unsubsc
potiuk merged PR #40905:
URL: https://github.com/apache/airflow/pull/40905
--
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 0fddfbc4b2 docs: fix wrong flag name and wrong RST
potiuk closed issue #40571: DockerSwarmOperator does not work well with
Python's tqdm when displaying logs
URL: https://github.com/apache/airflow/issues/40571
--
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
boring-cyborg[bot] commented on PR #40705:
URL: https://github.com/apache/airflow/pull/40705#issuecomment-2241269208
Awesome work, congrats on your first merged pull request! You are invited to
check our [Issue Tracker](https://github.com/apache/airflow/issues) for
additional contributions.
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 0fddfbc4b2 docs: fix wrong flag name and wrong RST formatting in
contributing unit_tests (#40905)
add d44016ce65
potiuk merged PR #40705:
URL: https://github.com/apache/airflow/pull/40705
--
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
jscheffl commented on code in PR #40901:
URL: https://github.com/apache/airflow/pull/40901#discussion_r1685531763
##
airflow/api_internal/internal_api_call.py:
##
@@ -75,10 +88,13 @@ def _init_values():
raise RuntimeError("The AIP_44 is not enabled so you cannot use
jscheffl commented on PR #40899:
URL: https://github.com/apache/airflow/pull/40899#issuecomment-2241293552
@potiuk Thanks for the fix in selective checks. Have re-worked now with JWT
approach - this is how you wanted to have it?
Note I am off on a cycling tour through switzerland tomo
alizademhdi commented on issue #40676:
URL: https://github.com/apache/airflow/issues/40676#issuecomment-2241301833
Hello. I am interested in working on this issue. Can you assign this to me?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
fbertos commented on issue #40676:
URL: https://github.com/apache/airflow/issues/40676#issuecomment-2241337829
No problem on my side but I am not sure I have enough privileges to assign
this to you.
--
This is an automated message from the Apache Git Service.
To respond to the message, pl
github-actions[bot] closed pull request #39652: update: remove RFC3896
`conn_type` warning
URL: https://github.com/apache/airflow/pull/39652
--
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 speci
github-actions[bot] closed pull request #39647: WIP: AIP-58: Add
ObjectStoragePath dag loader
URL: https://github.com/apache/airflow/pull/39647
--
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 sp
github-actions[bot] commented on PR #39879:
URL: https://github.com/apache/airflow/pull/39879#issuecomment-2241346568
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 pull request #39727: removing deprecations gke
URL: https://github.com/apache/airflow/pull/39727
--
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 u
jedcunningham merged PR #40903:
URL: https://github.com/apache/airflow/pull/40903
--
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
This is an automated email from the ASF dual-hosted git repository.
jedcunningham 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 ff7463b162 Chart: Update release notes for 1
Author: jedcunningham
Date: Sun Jul 21 03:06:47 2024
New Revision: 70444
Log:
Add artifacts for Helm Chart 1.15.0rc1
Added:
dev/airflow/helm-chart/1.15.0rc1/
dev/airflow/helm-chart/1.15.0rc1/airflow-1.15.0.tgz (with props)
dev/airflow/helm-chart/1.15.0rc1/airflow-1.15.0.tgz.asc
This is an automated email from the ASF dual-hosted git repository.
jedcunningham pushed a change to annotated tag helm-chart/1.15.0rc1
in repository https://gitbox.apache.org/repos/asf/airflow.git
*** WARNING: tag helm-chart/1.15.0rc1 was modified! ***
from ff7463b162 (commit)
to 06b
Added: dev/airflow/helm-chart/1.15.0rc1/index.yaml
==
--- dev/airflow/helm-chart/1.15.0rc1/index.yaml (added)
+++ dev/airflow/helm-chart/1.15.0rc1/index.yaml Sun Jul 21 03:06:47 2024
@@ -0,0 +1,2246 @@
+apiVersion: v1
+entr
Author: jedcunningham
Date: Sun Jul 21 03:07:26 2024
New Revision: 70445
Log:
Remove old Helm Chart release: 1.14.0rc1
Removed:
dev/airflow/helm-chart/1.14.0rc1/
jedcunningham opened a new issue, #40907:
URL: https://github.com/apache/airflow/issues/40907
### Body
We are kindly requesting that contributors to [Apache Airflow Helm Chart
1.15.0rc1](https://dist.apache.org/repos/dist/dev/airflow/helm-chart/1.15.0rc1)
help test the RC.
Ple
veinkr commented on issue #28723:
URL: https://github.com/apache/airflow/issues/28723#issuecomment-2241469969
Hi, @Taragolis , just curious if we can disable the usage of
[FAB](https://airflow.apache.org/docs/apache-airflow-providers-fab/stable/index.html),
and then update the sqlalchemy-2.
eladkal commented on PR #40527:
URL: https://github.com/apache/airflow/pull/40527#issuecomment-2241473310
Can you resolve conflicts and rebase?
--
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
jscheffl commented on code in PR #40901:
URL: https://github.com/apache/airflow/pull/40901#discussion_r1685644969
##
airflow/api_internal/internal_api_call.py:
##
@@ -75,10 +88,13 @@ def _init_values():
raise RuntimeError("The AIP_44 is not enabled so you cannot use
kunaljubce commented on issue #40232:
URL: https://github.com/apache/airflow/issues/40232#issuecomment-2241498779
@githubwua FYI in case this fell under your radar :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
79 matches
Mail list logo
