potiuk commented on PR #28976:
URL: https://github.com/apache/airflow/pull/28976#issuecomment-1398925508
conflicts to resolve.
--
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
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 e926bb9bb0 Add deferrable mode to
potiuk merged PR #28690:
URL: https://github.com/apache/airflow/pull/28690
--
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:
snjypl opened a new pull request, #29072:
URL: https://github.com/apache/airflow/pull/29072
Fix:
Invalid value for --integration. K should be small for kerberos.
![image](https://user-images.githubusercontent.com/4460062/213803546-35b7aa5c-a0d8-4629-b182-77c6e37da5c5.png)
potiuk merged PR #27833:
URL: https://github.com/apache/airflow/pull/27833
--
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.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 4b3a9ca5a3 Minor improvements to serde helpers (#28978)
add 5fcdd321c5 Add deferrable mode for Big Query Transfer
pierrejeambrun commented on issue #29026:
URL: https://github.com/apache/airflow/issues/29026#issuecomment-1398921159
Thanks to everyone that helped testing `2.5.1rc1` and `2.5.1rc2`. :pray:
Closing as 2.5.1 has been released based on `2.5.1rc2` :tada:
--
This is an automated
potiuk commented on PR #28979:
URL: https://github.com/apache/airflow/pull/28979#issuecomment-1398907022
> I have a look and also found that we have environment variable
USE_AIRFLOW_VERSION but we explicit set it to "wheel" (maybe somewhere in the
past it make sense). We could use it for
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 024f32f528 Deploying to gh-pages
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 826f3bd380 Properly test that audit log secrets are masked for
connection objects (#29011)
add 4b3a9ca5a3 Minor
potiuk merged PR #28978:
URL: https://github.com/apache/airflow/pull/28978
--
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:
Taragolis commented on PR #28979:
URL: https://github.com/apache/airflow/pull/28979#issuecomment-1398895970
> Not updated when we bumped it :)
I have a look and also found that we have environment variable
`USE_AIRFLOW_VERSION` but we explicit set it to "wheel" (maybe somewhere in
potiuk commented on PR #29035:
URL: https://github.com/apache/airflow/pull/29035#issuecomment-1398891137
Ah - isn't that the "airflow.jobs.backfill_job.BackfillJob" generated log ?
I thought simply backfill job sends the log entry when reconnecting? Because if
not then maybe for some
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 0c902bd046 Announce Apache Airflow
pierrejeambrun merged PR #725:
URL: https://github.com/apache/airflow-site/pull/725
--
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 #29071:
URL: https://github.com/apache/airflow/pull/29071#issuecomment-1398887503
https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/creating-a-commit-with-multiple-authors
--
This is an automated message from
Taragolis commented on PR #29035:
URL: https://github.com/apache/airflow/pull/29035#issuecomment-1398885650
> No -not skipped - simply a bad timing /flaky test:
>
> ```
> Failed to reconnect to scheduler after 30.00 seconds, closing client
> ```
Technically we should not
snjypl commented on PR #27357:
URL: https://github.com/apache/airflow/pull/27357#issuecomment-1398884543
@potiuk i have opened a PR with fixes to the failing unittest and also
upgraded the chart to lastest version 12.1.9
https://github.com/apache/airflow/pull/29071
i dont
potiuk commented on PR #28979:
URL: https://github.com/apache/airflow/pull/28979#issuecomment-1398876650
> Well seems like broken compatibility with 2.3.x, I will check later on as
well as will change wrong message for error in the CI (point to 2.2.0 but
actually we test on 2.3.0)
snjypl opened a new pull request, #29071:
URL: https://github.com/apache/airflow/pull/29071
Fixes: https://github.com/apache/airflow/issues/27353
this is a continuation of https://github.com/apache/airflow/pull/27357. i
fixed the failing unittests
---
**^ Add
Taragolis commented on PR #28979:
URL: https://github.com/apache/airflow/pull/28979#issuecomment-1398870960
Well seems like broken compatibility with 2.3.x, I will check later on as
well as change wrong message for error in the CI (point to 2.2.0 but actually
we test on 2.3.0)
--
This
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a commit to branch announce_251
in repository https://gitbox.apache.org/repos/asf/airflow-site.git
commit ca678a746d64480d04d519dd41e0d7747a5b580d
Author: pierrejeambrun
AuthorDate: Fri Jan 20 21:01:54
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a change to branch announce_251
in repository https://gitbox.apache.org/repos/asf/airflow-site.git
at ca678a746d Announce Apache Airflow 2.5.1 release
This branch includes the following new commits:
potiuk commented on code in PR #28926:
URL: https://github.com/apache/airflow/pull/28926#discussion_r1082995085
##
tests/www/views/test_views_connection.py:
##
@@ -40,11 +41,10 @@
"password": "admin",
}
-CONNECTION_WITH_EXTRA = CONNECTION.update(
Review Comment:
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 481f271706 fixing import error for dataset (#29007)
add 826f3bd380 Properly test that audit log secrets are
potiuk merged PR #29011:
URL: https://github.com/apache/airflow/pull/29011
--
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:
schwartzpub commented on issue #29037:
URL: https://github.com/apache/airflow/issues/29037#issuecomment-1398867257
![image](https://user-images.githubusercontent.com/5823949/213794037-a0d5a95b-ffa2-40b2-85a3-f32301ec414c.png)
I still don't fully understand the Logical Date, and I
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 bac7b3027d Fix SFTP operator's template fields processing (#29068)
add 481f271706 fixing import error for dataset
potiuk merged PR #29007:
URL: https://github.com/apache/airflow/pull/29007
--
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:
boring-cyborg[bot] commented on PR #29007:
URL: https://github.com/apache/airflow/pull/29007#issuecomment-1398865184
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.
jedcunningham pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git
The following commit(s) were added to refs/heads/master by this push:
new da5d34c Exclude openapi
jedcunningham merged PR #57:
URL: https://github.com/apache/airflow-client-python/pull/57
--
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.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 10f0f8bc4b Capitalize dag to DAG (#29064)
add bac7b3027d Fix SFTP operator's template fields processing (#29068)
potiuk merged PR #29068:
URL: https://github.com/apache/airflow/pull/29068
--
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 closed issue #27328: SFTPOperator throws object of type 'PlainXComArg'
has no len() when using with Taskflow API
URL: https://github.com/apache/airflow/issues/27328
--
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 PR #29068:
URL: https://github.com/apache/airflow/pull/29068#issuecomment-1398840425
good first issue created https://github.com/apache/airflow/issues/29070
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
potiuk opened a new issue, #29070:
URL: https://github.com/apache/airflow/issues/29070
### Body
Similarly to SFTP (fixed in https://github.com/apache/airflow/pull/29068)
the logic from __init__ should be moved to execute.
The #29068 provides a blueprint for that.
###
potiuk commented on PR #29068:
URL: https://github.com/apache/airflow/pull/29068#issuecomment-1398838008
> FTP provider has the same issue if you want to get two birds with one
stone.
I prefer to provide an example for others to folllow :)
--
This is an automated message from the
potiuk commented on issue #27328:
URL: https://github.com/apache/airflow/issues/27328#issuecomment-1398832969
> I like this. Though it will take quite some tmie I am afraid - unless we
can do most of it semi-automatically (which I think might be quite possible).
Created
potiuk opened a new issue, #29069:
URL: https://github.com/apache/airflow/issues/29069
### Body
It happens all-too-often that in our operators we have some initialization
logic that checks the content of templated fields or generally parameters
passed to the operators. This however
pierrejeambrun merged PR #724:
URL: https://github.com/apache/airflow-site/pull/724
--
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.
basph 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 10f0f8bc4b Capitalize dag to DAG (#29064)
BasPH merged PR #29064:
URL: https://github.com/apache/airflow/pull/29064
--
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.
pierrejeambrun pushed a change to branch 2.5.1-docs
in repository https://gitbox.apache.org/repos/asf/airflow-site.git
at 85d7b6cf4b Add documentation for Apache Airflow 2.5.1
This branch includes the following new
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 ad73819854 AIP-44 Migrate DagModel.get_paused_dag_ids to Internal API
(#28693)
add bab57f1a61 Renaming nose
potiuk merged PR #29035:
URL: https://github.com/apache/airflow/pull/29035
--
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 #29035:
URL: https://github.com/apache/airflow/pull/29035#issuecomment-1398821920
> Just single non-relevant to this PR failure. Somehow `distributed` could
avoid caplog filter:
https://github.com/apache/airflow/actions/runs/3970014684/jobs/6805354398#step:6:6520
This is an automated email from the ASF dual-hosted git repository.
jedcunningham pushed a change to branch extra_rat_exclude
in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git
at acdf0c2 Exclude openapi generator ignore file from Rat
This branch includes the
This is an automated email from the ASF dual-hosted git repository.
jedcunningham pushed a commit to branch extra_rat_exclude
in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git
commit acdf0c2e47ef2def21faeb63e609449e1f4522de
Author: Jed Cunningham
potiuk commented on PR #28502:
URL: https://github.com/apache/airflow/pull/28502#issuecomment-1398819090
(Pending tests passing :) )
--
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
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 f114c67c03 Fix pre-commit warning for exclude in inclusive-language
check (#29057)
add ad73819854 AIP-44 Migrate
potiuk closed issue #28614: AIP-44 Migrate DagModel.get_paused_dag_ids to
Internal API
URL: https://github.com/apache/airflow/issues/28614
--
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 merged PR #28693:
URL: https://github.com/apache/airflow/pull/28693
--
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 opened a new pull request, #29068:
URL: https://github.com/apache/airflow/pull/29068
The SFTP operator had logic in `__init__` that was running checks on passed
arguments, but that means that the operator woudl fail when using TaskFlow.
Fixes: #27328
---
**^
Taragolis commented on PR #29035:
URL: https://github.com/apache/airflow/pull/29035#issuecomment-1398808949
Just single non-relevant to this PR failure. Somehow `distributed` could
avoid caplog filter:
https://github.com/apache/airflow/actions/runs/3970014684/jobs/6805354398#step:6:6520
mrn-aglic opened a new issue, #29067:
URL: https://github.com/apache/airflow/issues/29067
### Apache Airflow version
2.5.0
### What happened
I was trying to learn about the feature of using dynamic task group mapping.
However, whenever I try to run the DAG, I get the
vincbeck commented on PR #28693:
URL: https://github.com/apache/airflow/pull/28693#issuecomment-1398800466
Done
--
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
vincbeck commented on code in PR #28502:
URL: https://github.com/apache/airflow/pull/28502#discussion_r1082938326
##
airflow/dag_processing/processor.py:
##
@@ -365,8 +365,10 @@ def __init__(self, dag_ids: list[str] | None,
dag_directory: str, log: logging.L
o-nikolas commented on code in PR #28934:
URL: https://github.com/apache/airflow/pull/28934#discussion_r1082935302
##
airflow/cli/commands/standalone_command.py:
##
@@ -159,14 +159,28 @@ def calculate_env(self):
# Make sure we're using a local executor flavour
dstandish commented on code in PR #28981:
URL: https://github.com/apache/airflow/pull/28981#discussion_r1082925535
##
airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py:
##
@@ -168,6 +168,9 @@ class KubernetesPodOperator(BaseOperator):
:param labels: labels to
Taragolis closed issue #16386: SageMaker operators return non-serializable value
URL: https://github.com/apache/airflow/issues/16386
--
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
Taragolis commented on issue #16386:
URL: https://github.com/apache/airflow/issues/16386#issuecomment-1398781623
Fixed by: https://github.com/apache/airflow/pull/23628
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
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 367e8f135c AIP-44: Add CLI command for running standalone Internal API
(#28425)
add f114c67c03 Fix pre-commit
potiuk merged PR #29057:
URL: https://github.com/apache/airflow/pull/29057
--
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 code in PR #28981:
URL: https://github.com/apache/airflow/pull/28981#discussion_r1082919473
##
airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py:
##
@@ -168,6 +168,9 @@ class KubernetesPodOperator(BaseOperator):
:param labels: labels to
potiuk commented on PR #27763:
URL: https://github.com/apache/airflow/pull/27763#issuecomment-1398770894
@kazanzhy ?
--
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 PR #28783:
URL: https://github.com/apache/airflow/pull/28783#issuecomment-1398769506
I guess now ew can rebase those :)
--
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
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 760c52949a logging poke info when external dag is not none and task_id
and task_ids are none (#28097)
add
potiuk merged PR #28425:
URL: https://github.com/apache/airflow/pull/28425
--
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 closed issue #28266: AIP-44 Implement standalone internal-api component
URL: https://github.com/apache/airflow/issues/28266
--
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
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 863ec46e25 Migrate Helm tests to `pytest` (#29063)
add 760c52949a logging poke info when external dag is not none
boring-cyborg[bot] commented on PR #28097:
URL: https://github.com/apache/airflow/pull/28097#issuecomment-1398765025
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
potiuk merged PR #28097:
URL: https://github.com/apache/airflow/pull/28097
--
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:
dstandish commented on code in PR #28981:
URL: https://github.com/apache/airflow/pull/28981#discussion_r1082906748
##
airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py:
##
@@ -168,6 +168,9 @@ class KubernetesPodOperator(BaseOperator):
:param labels: labels to
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a change to annotated tag 2.5.1
in repository https://gitbox.apache.org/repos/asf/airflow.git
*** WARNING: tag 2.5.1 was modified! ***
from 49867b660b (commit)
to 95f528cb64 (tag)
tagging
dstandish commented on code in PR #28981:
URL: https://github.com/apache/airflow/pull/28981#discussion_r1082903938
##
airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py:
##
@@ -168,6 +168,9 @@ class KubernetesPodOperator(BaseOperator):
:param labels: labels to
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a change to annotated tag constraints-latest
in repository https://gitbox.apache.org/repos/asf/airflow.git
*** WARNING: tag constraints-latest was modified! ***
from 63df097f00 (tag)
to
Taragolis commented on PR #28592:
URL: https://github.com/apache/airflow/pull/28592#issuecomment-1398757976
> Hi all, I'm seeing the same issue on `rc2` when passing an XComArg object
to the mapped task. Lists appear to work as expected.
Hi @mattppal would you like to create an
This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a change to annotated tag constraints-2.5.1
in repository https://gitbox.apache.org/repos/asf/airflow.git
*** WARNING: tag constraints-2.5.1 was modified! ***
from cacae5b639 (commit)
to
potiuk commented on code in PR #28502:
URL: https://github.com/apache/airflow/pull/28502#discussion_r1082897117
##
airflow/dag_processing/processor.py:
##
@@ -365,8 +365,10 @@ def __init__(self, dag_ids: list[str] | None,
dag_directory: str, log: logging.L
mattppal commented on PR #28592:
URL: https://github.com/apache/airflow/pull/28592#issuecomment-1398744685
Hi all, I'm seeing the same issue on `rc2` when passing an XComArg object to
the mapped task. Lists appear to work as expected.
--
This is an automated message from the Apache Git
vincbeck commented on code in PR #28502:
URL: https://github.com/apache/airflow/pull/28502#discussion_r1082893036
##
airflow/dag_processing/processor.py:
##
@@ -365,8 +365,10 @@ def __init__(self, dag_ids: list[str] | None,
dag_directory: str, log: logging.L
potiuk commented on PR #29064:
URL: https://github.com/apache/airflow/pull/29064#issuecomment-1398738922
I cannot figure out why 'dag' is not flagged by our spellchecker. It looks
like it should, and I tried to experiment a bit but I really cannot find why it
has been allowed in the first
Author: ephraimanierobi
Date: Fri Jan 20 17:46:34 2023
New Revision: 59463
Log:
Remove old release: 2.5.0
Removed:
release/airflow/2.5.0/
Author: ephraimanierobi
Date: Fri Jan 20 17:45:42 2023
New Revision: 59462
Log:
Release Airflow 2.5.1 from 2.5.1rc2
Added:
release/airflow/2.5.1/
release/airflow/2.5.1/apache-airflow-2.5.1-source.tar.gz
- copied unchanged from r59461,
This is an automated email from the ASF dual-hosted git repository.
taragolis 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 863ec46e25 Migrate Helm tests to `pytest`
Taragolis merged PR #29063:
URL: https://github.com/apache/airflow/pull/29063
--
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:
Taragolis commented on PR #29035:
URL: https://github.com/apache/airflow/pull/29035#issuecomment-1398705272
> > @potiuk, is it the right place?
>
> Yep. But you can also add "full tests needed" label on a PR to run a
complete set of tests.
And very easy to forget add this
jedcunningham commented on PR #28808:
URL: https://github.com/apache/airflow/pull/28808#issuecomment-1398700496
Looking at this with fresh eyes, this works?
```
kpo_task = KubernetesPodOperator(...)
kpo_task.BASE_CONTAINER_NAME = "whatever"
```
Is it worth adding this
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 fe1af7b2e1 Updating
potiuk commented on PR #29063:
URL: https://github.com/apache/airflow/pull/29063#issuecomment-1398695049
(pending tess succeed ofc)
--
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
bbovenzi opened a new pull request, #29066:
URL: https://github.com/apache/airflow/pull/29066
Some parts of the app linked over to the graph and gantt views with a
`run_id=` param, but we weren't actually using that param to decide which run
to show for a user.
Now, we will first check
potiuk commented on code in PR #28846:
URL: https://github.com/apache/airflow/pull/28846#discussion_r1082853296
##
airflow/models/serialized_dag.py:
##
@@ -102,7 +102,7 @@ def __init__(self, dag: DAG, processor_subdir: str | None =
None):
dag_data =
boring-cyborg[bot] commented on PR #29065:
URL: https://github.com/apache/airflow/pull/29065#issuecomment-1398692028
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
sean-rose opened a new pull request, #29065:
URL: https://github.com/apache/airflow/pull/29065
This fixes problems when recursively clearing task instances across multiple
DAGs:
* Task instances in downstream DAGs weren't having their `max_tries`
property incremented, which could
Taragolis opened a new pull request, #29063:
URL: https://github.com/apache/airflow/pull/29063
Migrate `kubernetes_tests` to pytest.
Also resolve issue that tests use combination of `unittests`/`nose`/`pytest`
which found during this PR:
-
BasPH opened a new pull request, #29064:
URL: https://github.com/apache/airflow/pull/29064
DAG is an abbreviation (Directed Acyclic Graph) and should therefore be
written capitalized. This PR changes "dags" to "DAGs" wherever it's written for
humans (i.e. not code, hyperlinks, etc.).
shyft-mike commented on code in PR #28846:
URL: https://github.com/apache/airflow/pull/28846#discussion_r1082843374
##
airflow/models/serialized_dag.py:
##
@@ -102,7 +102,7 @@ def __init__(self, dag: DAG, processor_subdir: str | None =
None):
dag_data =
potiuk commented on PR #28476:
URL: https://github.com/apache/airflow/pull/28476#issuecomment-1398678859
> > I guess that one should be updated after discussion in #28502 with
`@classmethod` ?
>
> Correct! I rather wait for #28502 to be merged and then I'll update this PR
+1
potiuk commented on code in PR #28502:
URL: https://github.com/apache/airflow/pull/28502#discussion_r1082833504
##
airflow/dag_processing/processor.py:
##
@@ -365,8 +365,10 @@ def __init__(self, dag_ids: list[str] | None,
dag_directory: str, log: logging.L
101 - 200 of 253 matches
Mail list logo