uranusjr commented on PR #30083:
URL: https://github.com/apache/airflow/pull/30083#issuecomment-1471387016
Many things to fix. Actually the test failure is DST related, which strongly
indicates the logic is wrong (in the sense of not being backward compatible).
--
This is an automated mes
uranusjr commented on code in PR #27106:
URL: https://github.com/apache/airflow/pull/27106#discussion_r1138174686
##
airflow/cli/commands/task_command.py:
##
@@ -126,10 +126,12 @@ def _get_dag_run(
dag_run = DagRun(dag.dag_id, run_id=exec_date_or_run_id,
execution_date
josh-fell opened a new pull request, #30138:
URL: https://github.com/apache/airflow/pull/30138
#30057 added a couple field validators in a new connection form for
Hashicorp Vault. Behind the scenes, all connection form data is persisted upon
submission. When creating a new connection that i
uranusjr commented on code in PR #29094:
URL: https://github.com/apache/airflow/pull/29094#discussion_r1138123017
##
airflow/jobs/scheduler_job.py:
##
@@ -221,26 +221,31 @@ def is_alive(self, grace_multiplier: float | None = None)
-> bool:
def __get_concurrency_maps(
uranusjr commented on code in PR #29094:
URL: https://github.com/apache/airflow/pull/29094#discussion_r1138123017
##
airflow/jobs/scheduler_job.py:
##
@@ -221,26 +221,31 @@ def is_alive(self, grace_multiplier: float | None = None)
-> bool:
def __get_concurrency_maps(
uranusjr commented on code in PR #29094:
URL: https://github.com/apache/airflow/pull/29094#discussion_r1138121909
##
airflow/jobs/scheduler_job.py:
##
@@ -221,26 +221,31 @@ def is_alive(self, grace_multiplier: float | None = None)
-> bool:
def __get_concurrency_maps(
uranusjr commented on PR #30105:
URL: https://github.com/apache/airflow/pull/30105#issuecomment-1471311853
Hmm, this works for those, but may not for other exception classes that
format the error message otherwise. Perhaps it is better to explicitly add the
exception class to the error mess
SamWheating commented on code in PR #29909:
URL: https://github.com/apache/airflow/pull/29909#discussion_r1138094579
##
airflow/models/dag.py:
##
@@ -546,6 +553,9 @@ def __init__(
self.last_loaded = timezone.utcnow()
self.safe_dag_id = dag_id.replace(".", "__do
uranusjr commented on code in PR #30113:
URL: https://github.com/apache/airflow/pull/30113#discussion_r1138093946
##
tests/api_connexion/endpoints/test_dag_run_endpoint.py:
##
Review Comment:
Should we have a test case for an invalid state value?
--
This is an automated
gbonazzoli opened a new issue, #30137:
URL: https://github.com/apache/airflow/issues/30137
### Apache Airflow version
2.5.2
### What happened
Everything works fine, but there are the following warning:
```
airflow@airflow-crm-worker-0:/opt/airflow$ airflow versi
yuqian90 commented on PR #29971:
URL: https://github.com/apache/airflow/pull/29971#issuecomment-1471208053
> @yuqian90 wdyt of expand/collapse like this?
>
> 
This is an automated email from the ASF dual-hosted git repository.
joshfell pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 051cd2938e Add removal of old RCs to RC release process (#30133)
add 2842a0f30b Add custom_operator_name to @tas
josh-fell merged PR #30131:
URL: https://github.com/apache/airflow/pull/30131
--
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...@airflo
uranusjr commented on PR #29842:
URL: https://github.com/apache/airflow/pull/29842#issuecomment-1471184980
There’s precedence of using `inspect` to check whether a function has a
specific argument, and only pass extra values if it is present. We can
implement that here so `template_fields`
bbovenzi opened a new pull request, #30136:
URL: https://github.com/apache/airflow/pull/30136
We were listing the Dag Runs Summary twice in the Grid -> Dag Details. Looks
to have been an issue when rebasing the new graph PR.
---
**^ Add meaningful description above**
Read th
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 14d9e90b3a Improve creating Vault connection from the UI and add
documentation for this conn (#30057)
add
jedcunningham merged PR #30133:
URL: https://github.com/apache/airflow/pull/30133
--
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
pingzh commented on code in PR #30104:
URL: https://github.com/apache/airflow/pull/30104#discussion_r1137952986
##
docs/apache-airflow/core-concepts/dags.rst:
##
@@ -165,6 +165,14 @@ While both DAG constructors get called when the file is
accessed, only ``dag_1``
You can als
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 6c4761aa37 add a describe step on failure in eks tests (#30134)
add 14d9e90b3a Improve creating Vault connection f
potiuk merged PR #30057:
URL: https://github.com/apache/airflow/pull/30057
--
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 code in PR #30104:
URL: https://github.com/apache/airflow/pull/30104#discussion_r1137913254
##
docs/apache-airflow/core-concepts/dags.rst:
##
@@ -165,6 +165,14 @@ While both DAG constructors get called when the file is
accessed, only ``dag_1``
You can als
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 6c4761aa37 add a describe step on failure in eks te
potiuk merged PR #30134:
URL: https://github.com/apache/airflow/pull/30134
--
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
github-actions[bot] commented on issue #19627:
URL: https://github.com/apache/airflow/issues/19627#issuecomment-1471015001
This issue has been automatically marked as stale because it has been open
for 30 days with no response from the author. It will be closed in next 7 days
if no further
github-actions[bot] commented on PR #28291:
URL: https://github.com/apache/airflow/pull/28291#issuecomment-1471014934
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 #28304: expand_kwargs does not handle update
of dataset correctly when "outlets" parameter is used as kwargs
URL: https://github.com/apache/airflow/issues/28304
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
github-actions[bot] commented on PR #28292:
URL: https://github.com/apache/airflow/pull/28292#issuecomment-1471014912
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 #28304:
URL: https://github.com/apache/airflow/issues/28304#issuecomment-1471014896
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 #29223:
URL: https://github.com/apache/airflow/pull/29223#issuecomment-1471014853
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
TJaniF commented on PR #29842:
URL: https://github.com/apache/airflow/pull/29842#issuecomment-1471013878
> > Users who are using the GitHubSensor directly and pass a
`result_processor` function directly would either need to turn
`allow_templates_in_result_processor` to False or add a `templ
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 8d22828e25 Fix `TriggerRuleDep` when the mapped tasks count is 0
(#30084)
add f1e40cf799 Adding more information
potiuk closed issue #30023: Variable with template is ambiguous, especially for
new users
URL: https://github.com/apache/airflow/issues/30023
--
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 spec
potiuk merged PR #30040:
URL: https://github.com/apache/airflow/pull/30040
--
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 #29266:
URL: https://github.com/apache/airflow/pull/29266#issuecomment-1470999113
I have not followed too much the deferrable operators - I know it was
somewhere in between Astronomer and Goole work :) and you worked, so I guess It
would take more time for me to get f
potiuk commented on code in PR #30076:
URL: https://github.com/apache/airflow/pull/30076#discussion_r1137871955
##
airflow/dag_processing/manager.py:
##
@@ -727,7 +740,8 @@ def _add_callback_to_queue(self, request: CallbackRequest):
self._file_path_queue = colle
potiuk commented on code in PR #30076:
URL: https://github.com/apache/airflow/pull/30076#discussion_r1137864223
##
airflow/dag_processing/manager.py:
##
@@ -1202,6 +1226,15 @@ def _kill_timed_out_processors(self):
for proc in processors_to_remove:
self._pro
potiuk commented on PR #29935:
URL: https://github.com/apache/airflow/pull/29935#issuecomment-1470955227
> I have read through the article but I was not able to find the extras that
do not require to install anything for the setup to run properly. Could you
give me some directions on what t
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 66b5f90f45 Update workday example (#30026)
add 8d22828e25 Fix `TriggerRuleDep` when the mapped tasks count is 0
(
potiuk closed issue #30073: Task group expand fails on empty list at
get_relevant_upstream_map_indexes
URL: https://github.com/apache/airflow/issues/30073
--
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
potiuk merged PR #30084:
URL: https://github.com/apache/airflow/pull/30084
--
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 #30026:
URL: https://github.com/apache/airflow/pull/30026#issuecomment-1470950100
> Sorry Jarek...I didn't realize the sample code was being used in tests as
well. I've updated the corresponding test code and all checks have passed.
No wories. This is normal :)
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 f18cbe Explicit skipped states list for ExternalTaskSensor (#29933)
add 66b5f90f45 Update workday example (#30
potiuk commented on PR #29910:
URL: https://github.com/apache/airflow/pull/29910#issuecomment-1470949360
> @hussein-awala I installed pre-commit to fix static checks and committed
changes.
There are few more suggestions from @hussein-awala
--
This is an automated message from the
potiuk merged PR #30026:
URL: https://github.com/apache/airflow/pull/30026
--
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 issue #29963:
URL: https://github.com/apache/airflow/issues/29963#issuecomment-1470947763
> I am aware that the pod no longer exists, but the logs are in the file
system.
> What I am trying to say is that airflow could recover the logs even if the
pod has been deleted
potiuk commented on PR #27264:
URL: https://github.com/apache/airflow/pull/27264#issuecomment-1470932110
> Snowflake 3.11 wheels are out
https://github.com/snowflakedb/snowflake-connector-python/pull/1349/files
referenced pull request is outdated
Marked it as such
--
This is an au
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 feab463757 Add a new Airflow conf to specify a SSL ca cert for
Kubernetes client (#30048)
add f18cbe Explicit
potiuk merged PR #29933:
URL: https://github.com/apache/airflow/pull/29933
--
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
hussein-awala commented on PR #30057:
URL: https://github.com/apache/airflow/pull/30057#issuecomment-1470929507
It should be fine 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 specific
potiuk commented on PR #28943:
URL: https://github.com/apache/airflow/pull/28943#issuecomment-1470928814
There some tests to fix though
--
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 specifi
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 feab463757 Add a new Airflow conf to specify a SSL
potiuk merged PR #30048:
URL: https://github.com/apache/airflow/pull/30048
--
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 closed issue #8019: Custom SSL CA CERTIFICATE by Configuration for
Kubernetes Executor
URL: https://github.com/apache/airflow/issues/8019
--
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 s
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 5a3be7256b Handling project location param on async BigQuery dts
trigger (#29786)
add 39cfc67cad Update scripts t
potiuk merged PR #30054:
URL: https://github.com/apache/airflow/pull/30054
--
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 closed issue #29967: Build our supporting images for chart in
multi-platform versions
URL: https://github.com/apache/airflow/issues/29967
--
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 s
potiuk commented on PR #30057:
URL: https://github.com/apache/airflow/pull/30057#issuecomment-1470921301
There is a conflict to reseolve though.
--
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 th
potiuk closed issue #19681: Can't launch Redis container on a cluster enforcing
MustRunAsNonRoot policy
URL: https://github.com/apache/airflow/issues/19681
--
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
potiuk commented on issue #19681:
URL: https://github.com/apache/airflow/issues/19681#issuecomment-1470918073
closing then. @tomasgatial please check if it fixes it, and let us know- we
can always re-open.
--
This is an automated message from the Apache Git Service.
To respond to the mess
VinceLegendre commented on PR #28525:
URL: https://github.com/apache/airflow/pull/28525#issuecomment-1470915296
I'll try to fix tests in the coming days.
--
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
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 5a3be7256b Handling project location param on async
potiuk merged PR #29786:
URL: https://github.com/apache/airflow/pull/29786
--
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 code in PR #30054:
URL: https://github.com/apache/airflow/pull/30054#discussion_r1137798001
##
chart/newsfragments/30054.improvement.rst:
##
@@ -0,0 +1 @@
+The pgbouncer and pgbouncer-images are now based on newer software and os
versions. Thye also have nat
potiuk commented on PR #30133:
URL: https://github.com/apache/airflow/pull/30133#issuecomment-1470826417
> Does it cover alao providers/helm RCs?
Providers are deleted already as part of the process.
--
This is an automated message from the Apache Git Service.
To respond to the mes
potiuk merged PR #30002:
URL: https://github.com/apache/airflow/pull/30002
--
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
eladkal commented on PR #30003:
URL: https://github.com/apache/airflow/pull/30003#issuecomment-1470818675
> Yes it's a valid worry. But maybe we could make it clearer by slight
modification of the pytest code so that it will raise Skip exception for those
tests unless we setup some variable
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 f5ed6ae67d Update 2.5.2 release date (#30135)
add b87cbc388b Ignore error when changing log folder permissions (#3
potiuk closed issue #29112: "Operation not permitted" error when chmod on log
folder
URL: https://github.com/apache/airflow/issues/29112
--
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 merged PR #30123:
URL: https://github.com/apache/airflow/pull/30123
--
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 #30003:
URL: https://github.com/apache/airflow/pull/30003#issuecomment-1470810141
> I'm with Elad on this. Having just a couple of examples left in examples
folder is rather confusing for user's pov. Providing all examples / system
tests in one place makes it easier t
potiuk commented on code in PR #29926:
URL: https://github.com/apache/airflow/pull/29926#discussion_r1137742117
##
airflow/configuration.py:
##
Review Comment:
I am ok with whatever you guys agree to :)
--
This is an automated message from the Apache Git Service.
To re
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch gh-pages
in repository https://gitbox.apache.org/repos/asf/airflow-site.git
The following commit(s) were added to refs/heads/gh-pages by this push:
new 1ebf109cb0 Deploying to gh-pages f
potiuk commented on issue #27292:
URL: https://github.com/apache/airflow/issues/27292#issuecomment-1470793624
There is a WIP from Google team to upgrade the SDKs
https://github.com/apache/airflow/pull/30067
--
This is an automated message from the Apache Git Service.
To respond to the mes
potiuk commented on issue #29948:
URL: https://github.com/apache/airflow/issues/29948#issuecomment-1470788257
> @eladkal Did this pull request got forgotten? #29694 Please let me know if
I should do something about it to make it visible.
Only Hashicorp ad-hoc release was released then
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-site.git
The following commit(s) were added to refs/heads/main by this push:
new bc075d5a20 Update release date for 2.5
pierrejeambrun merged PR #755:
URL: https://github.com/apache/airflow-site/pull/755
--
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...@
amoghrajesh commented on PR #29809:
URL: https://github.com/apache/airflow/pull/29809#issuecomment-1470782793
@hussein-awala i think we should go back to throwing an exception when the
type is invalid for the retry parameters.
I need some help in figuring out a way to fix the other f
hussein-awala commented on code in PR #29929:
URL: https://github.com/apache/airflow/pull/29929#discussion_r1137720316
##
airflow/kubernetes/kubernetes_helper_functions.py:
##
@@ -119,3 +120,7 @@ def annotations_to_key(annotations: dict[str, str]) ->
TaskInstanceKey:
t
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a commit to branch v2-5-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/v2-5-test by this push:
new cfaa9eac8a Update 2.5.2 release d
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 76a884c552 Ensure that `dag.partial_subset` doesn't mutate task group
properties (#30129)
add f5ed6ae67d
pierrejeambrun merged PR #30135:
URL: https://github.com/apache/airflow/pull/30135
--
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
ashb closed issue #26059: [Graph view] After clearing the task (and its
downstream tasks) in a task group the task group becomes disconnected from the
dag
URL: https://github.com/apache/airflow/issues/26059
--
This is an automated message from the Apache Git Service.
To respond to the messag
ashb merged PR #30129:
URL: https://github.com/apache/airflow/pull/30129
--
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.apa
This is an automated email from the ASF dual-hosted git repository.
ash pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 4dde8ececf Update Airflow version to 2.5.2 (#30111)
add 76a884c552 Ensure that `dag.partial_subset` doesn't mutate ta
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from c7488334cd Improve install instruction for test package (#30030)
add 4dde8ececf Update Airflow version to
pierrejeambrun merged PR #30111:
URL: https://github.com/apache/airflow/pull/30111
--
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
jedcunningham commented on code in PR #29926:
URL: https://github.com/apache/airflow/pull/29926#discussion_r1137705279
##
airflow/configuration.py:
##
Review Comment:
I'm not in favor of moving it. I'd rather see it actually be a proper config
entry `e.g. [webserver] confi
amoghrajesh commented on code in PR #29929:
URL: https://github.com/apache/airflow/pull/29929#discussion_r1137703660
##
airflow/kubernetes/kubernetes_helper_functions.py:
##
@@ -119,3 +120,7 @@ def annotations_to_key(annotations: dict[str, str]) ->
TaskInstanceKey:
try
amoghrajesh commented on code in PR #29926:
URL: https://github.com/apache/airflow/pull/29926#discussion_r1137699588
##
airflow/configuration.py:
##
Review Comment:
I have a proposal here. How about we do this:
1. We have a constant called AIRFLOW_WEBSERVER_CONFIG ar
jedcunningham commented on code in PR #29926:
URL: https://github.com/apache/airflow/pull/29926#discussion_r1137698124
##
airflow/configuration.py:
##
Review Comment:
So to summarize, I'd rather we make it actually configurable than to just
move it to a dir that happens to
jedcunningham commented on code in PR #29926:
URL: https://github.com/apache/airflow/pull/29926#discussion_r1137694479
##
airflow/configuration.py:
##
Review Comment:
Ah, TIL about AIRFLOW_HOME/config being added to sys.path automatically 🤦♂️
Makes a little more sense now
amoghrajesh commented on code in PR #29926:
URL: https://github.com/apache/airflow/pull/29926#discussion_r1137688177
##
airflow/configuration.py:
##
Review Comment:
Thanks for the review.
I am trying to make the webserver config file rather sit in the config
directo
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from b8ab594130 Remove "boilerplate" from all taskflow decorators (#30118)
add c7488334cd Improve install instr
pierrejeambrun merged PR #30030:
URL: https://github.com/apache/airflow/pull/30030
--
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
jedcunningham commented on code in PR #29926:
URL: https://github.com/apache/airflow/pull/29926#discussion_r1137660553
##
airflow/configuration.py:
##
Review Comment:
This feels a little half-baked.
- This doesn't actually make the path configurable, just moves it un
vandonr-amz opened a new pull request, #30134:
URL: https://github.com/apache/airflow/pull/30134
we've seen that every once in a while, an eks system test will fail with a
timeout on the pod starting.
Since we delete all resources in the teardown, we're pretty clueless on
what's happeni
jedcunningham commented on code in PR #30129:
URL: https://github.com/apache/airflow/pull/30129#discussion_r1137644676
##
airflow/models/dag.py:
##
@@ -2215,29 +2215,40 @@ def _deepcopy_task(t) -> Operator:
def filter_task_group(group, parent_group):
"""E
shubham22 commented on PR #30003:
URL: https://github.com/apache/airflow/pull/30003#issuecomment-1470643869
I'm with Elad on this. Having just a couple of examples left in examples
folder is rather confusing for user's pov. Providing all examples / system
tests in one place makes it easier
jedcunningham commented on code in PR #30111:
URL: https://github.com/apache/airflow/pull/30111#discussion_r1137613768
##
RELEASE_NOTES.rst:
##
@@ -21,6 +21,114 @@
.. towncrier release notes start
+Airflow 2.5.2 (2023-03-13)
Review Comment:
Oh, yes, good call.
--
Th
eladkal commented on PR #30003:
URL: https://github.com/apache/airflow/pull/30003#issuecomment-1470603601
The current state where we have examples speared throughout the project is
not intuitive. Which is why I raised this as an issue
https://github.com/apache/airflow/issues/22438#issuecomm
1 - 100 of 215 matches
Mail list logo
