github-actions[bot] commented on PR #23188:
URL: https://github.com/apache/airflow/pull/23188#issuecomment-1108107054
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and take the
uranusjr commented on code in PR #23178:
URL: https://github.com/apache/airflow/pull/23178#discussion_r857273834
##
airflow/migrations/versions/__add_task_try_table.py:
##
@@ -0,0 +1,53 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or
github-actions[bot] commented on PR #23190:
URL: https://github.com/apache/airflow/pull/23190#issuecomment-1108101418
The PR is likely OK to be merged with just subset of tests for default
Python and Database versions without running the full matrix of tests, because
it does not modify the
uranusjr commented on code in PR #23181:
URL: https://github.com/apache/airflow/pull/23181#discussion_r857270983
##
airflow/models/dag.py:
##
@@ -1984,15 +1987,18 @@ def partial_subset(
also_include.extend(t.get_flat_relatives(upstream=False))
if
uranusjr commented on code in PR #23181:
URL: https://github.com/apache/airflow/pull/23181#discussion_r857270875
##
airflow/models/dag.py:
##
@@ -1984,15 +1987,18 @@ def partial_subset(
also_include.extend(t.get_flat_relatives(upstream=False))
if
github-actions[bot] commented on PR #23205:
URL: https://github.com/apache/airflow/pull/23205#issuecomment-1108095672
The PR is likely ready to be merged. No tests are needed as no important
environment files, nor python files were modified by it. However, committers
might decide that full
pragyak412 commented on PR #23116:
URL: https://github.com/apache/airflow/pull/23116#issuecomment-1108081332
@mik-laj Please review it whenever you have time. Let me know if there is
any suggestion.
Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the
uranusjr commented on PR #23173:
URL: https://github.com/apache/airflow/pull/23173#issuecomment-1108068854
Would the decimal places always show, even when the duration is long?
Ideally I’d only want them when the duration is <1 and not if it’s e.g. 105.378
seconds.
--
This is an
uranusjr commented on code in PR #23160:
URL: https://github.com/apache/airflow/pull/23160#discussion_r857250955
##
airflow/providers/docker/operators/docker.py:
##
@@ -333,7 +333,7 @@ def _run_image_with_mounts(
return None
finally:
if
uranusjr commented on issue #23206:
URL: https://github.com/apache/airflow/issues/23206#issuecomment-1108064654
See also #22967.
--
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
boring-cyborg[bot] commented on issue #23207:
URL: https://github.com/apache/airflow/issues/23207#issuecomment-1108043282
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
Sha-of-Lazy opened a new issue, #23207:
URL: https://github.com/apache/airflow/issues/23207
### Apache Airflow version
2.2.5 (latest released)
### What happened
The DAG run randomly(about 5%) failed when scheduled. In the web UI it was
like 'failed' and 'Not yet
MatrixManAtYrService opened a new issue, #23206:
URL: https://github.com/apache/airflow/issues/23206
### Apache Airflow version
2.3.0b1 (pre-release)
### What happened
I tried to delete a dag from the grid view, and I saw this instead
```
Ooops!
Something
zengbotang commented on code in PR #23160:
URL: https://github.com/apache/airflow/pull/23160#discussion_r857238847
##
airflow/providers/docker/operators/docker.py:
##
@@ -333,7 +333,7 @@ def _run_image_with_mounts(
return None
finally:
if
dbarrundiag commented on code in PR #22692:
URL: https://github.com/apache/airflow/pull/22692#discussion_r857235665
##
airflow/providers/delta/sharing/hooks/delta_sharing.py:
##
@@ -0,0 +1,286 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
dbarrundiag commented on code in PR #22692:
URL: https://github.com/apache/airflow/pull/22692#discussion_r857235665
##
airflow/providers/delta/sharing/hooks/delta_sharing.py:
##
@@ -0,0 +1,286 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
dbarrundiag commented on code in PR #22692:
URL: https://github.com/apache/airflow/pull/22692#discussion_r857231218
##
airflow/providers/delta/sharing/operators/delta_sharing.py:
##
@@ -0,0 +1,227 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
ashutoshcipher commented on issue #21897:
URL: https://github.com/apache/airflow/issues/21897#issuecomment-1107953372
Hi @potiuk - If you @yeachan153 is not working on it, Can I pick this up?
--
This is an automated message from the Apache Git Service.
To respond to the message, please
github-actions[bot] commented on PR #21618:
URL: https://github.com/apache/airflow/pull/21618#issuecomment-1107947108
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
This is an automated email from the ASF dual-hosted git repository.
kamilbregula 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 f02b0b6b40 Further improvement of Databricks
mik-laj merged PR #23199:
URL: https://github.com/apache/airflow/pull/23199
--
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:
github-actions[bot] commented on PR #23199:
URL: https://github.com/apache/airflow/pull/23199#issuecomment-1107944676
The PR is likely OK to be merged with just subset of tests for default
Python and Database versions without running the full matrix of tests, because
it does not modify the
potiuk commented on PR #23205:
URL: https://github.com/apache/airflow/pull/23205#issuecomment-1107940966
cc: @Bowrna @eladkal @edithturn - we came to the point when we should
split the commands into separate packages
(This change is based on #23195 so it should be merged first).
potiuk opened a new pull request, #23205:
URL: https://github.com/apache/airflow/pull/23205
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)**
for more
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 8880e6c46a Updating
potiuk opened a new issue, #23204:
URL: https://github.com/apache/airflow/issues/23204
The breeze.py file is already huge. It should be split in mulitple files:
* common options
* separate file for each group of functions (commands/[GROUP].py)
--
This is an automated message
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 7d9f8bcb27 Fix static-checks actually work after
potiuk merged PR #23202:
URL: https://github.com/apache/airflow/pull/23202
--
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 #23202:
URL: https://github.com/apache/airflow/pull/23202#issuecomment-1107896592
Glad I tested it over the weekend when we had very little number of PRs :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
potiuk commented on PR #23202:
URL: https://github.com/apache/airflow/pull/23202#issuecomment-1107896445
Hey @eladkal -> I looked closely and I found that static chceks were not
really "running" (but succeeded) - so I changed this PR as I used it as a
"testing ground"
--
This is an
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 47cdfac76b Remove fix-ownership after upload coverage and extra cache
(#23203)
add f4deec87f9 Add cleanup of
potiuk merged PR #23201:
URL: https://github.com/apache/airflow/pull/23201
--
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 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 47cdfac76b Remove fix-ownership after upload
potiuk merged PR #23203:
URL: https://github.com/apache/airflow/pull/23203
--
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:
github-actions[bot] commented on PR #23203:
URL: https://github.com/apache/airflow/pull/23203#issuecomment-1107882292
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and take the
potiuk opened a new pull request, #23203:
URL: https://github.com/apache/airflow/pull/23203
There is no Breeze installed in upload coverage so fix-ownership
is not needed (and harmful because breeze is missing :))
---
**^ Add meaningful description above**
Read
github-actions[bot] commented on PR #23202:
URL: https://github.com/apache/airflow/pull/23202#issuecomment-1107878960
The PR is likely ready to be merged. No tests are needed as no important
environment files, nor python files were modified by it. However, committers
might decide that full
potiuk opened a new pull request, #23202:
URL: https://github.com/apache/airflow/pull/23202
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)**
for more
potiuk commented on code in PR #23193:
URL: https://github.com/apache/airflow/pull/23193#discussion_r857153525
##
BREEZE.rst:
##
@@ -445,64 +448,81 @@ Airflow Breeze is a bash script serving as a
"swiss-army-knife" of Airflow testi
hood it uses other scripts that you can also
rbiegacz commented on issue #22465:
URL: https://github.com/apache/airflow/issues/22465#issuecomment-1107876313
Please, assign this item to me. I started working on it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
bbovenzi commented on issue #19943:
URL: https://github.com/apache/airflow/issues/19943#issuecomment-1107872310
It's all yours Pierre! Feel free to reach out if you have any questions
whatsoever.
Quick feedback on the mocks:
- Let's not completely hide non-matching task instances,
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 6c3ec31ce8 Updating
pierrejeambrun commented on issue #19943:
URL: https://github.com/apache/airflow/issues/19943#issuecomment-1107871621
Hello guys,
This issue looks really interesting and I would love to work on it.
I was thinking of something like this for the UI:
![image
ephraimbuddy commented on issue #20442:
URL: https://github.com/apache/airflow/issues/20442#issuecomment-1107867739
> any ideas?
Fixed in 2.3.0. We have a beta release of 2.3.0, can you test it? See
https://lists.apache.org/thread/9tsks6hg2974154w5jt1jq0j9x8wlj7k
--
This is an
github-actions[bot] commented on PR #23201:
URL: https://github.com/apache/airflow/pull/23201#issuecomment-1107865307
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and take the
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 87e3733bbc Unify all "breeze" tools under breeze
potiuk merged PR #23193:
URL: https://github.com/apache/airflow/pull/23193
--
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 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 f3f07574a6 Fix main failure after moto upgrade
potiuk merged PR #23200:
URL: https://github.com/apache/airflow/pull/23200
--
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:
github-actions[bot] commented on PR #23200:
URL: https://github.com/apache/airflow/pull/23200#issuecomment-1107863667
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and take the
github-actions[bot] commented on PR #23193:
URL: https://github.com/apache/airflow/pull/23193#issuecomment-1107863317
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and take the
potiuk opened a new pull request, #23201:
URL: https://github.com/apache/airflow/pull/23201
This change protects against accidental left-overs (root-owned)
that might be created in the repository, in case a job is
re-run on a self-hosted runnner from a previous build.
ephraimbuddy commented on code in PR #23193:
URL: https://github.com/apache/airflow/pull/23193#discussion_r857141900
##
BREEZE.rst:
##
@@ -445,64 +448,81 @@ Airflow Breeze is a bash script serving as a
"swiss-army-knife" of Airflow testi
hood it uses other scripts that you
mrkaye97 commented on issue #23176:
URL: https://github.com/apache/airflow/issues/23176#issuecomment-1107860345
Makes sense, thanks @uranusjr!
I just thought this would be worth pointing out for others in the future who
had the same issue. It took us a while to debug and wasn't
potiuk opened a new pull request, #23200:
URL: https://github.com/apache/airflow/pull/23200
The moto library 3.1.6 extracted MotoAPI to dedicated module
(How about SemVer?).
https://github.com/spulec/moto/pull/5055
This broke our S3/CloudWatch tests.
This PR bumps
ktleung2017 commented on issue #13542:
URL: https://github.com/apache/airflow/issues/13542#issuecomment-1107854050
https://github.com/apache/airflow/issues/21455#issue-1128336470
The fix here solves our stuck in "scheduled" problem. If you are using
2.2.2, you may give it a try.
--
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 79d1a9d1cb Force installing Breeze on CI (#23196)
potiuk merged PR #23196:
URL: https://github.com/apache/airflow/pull/23196
--
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 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 438814f9e5 Add cleanup of docker-context-files
potiuk merged PR #23197:
URL: https://github.com/apache/airflow/pull/23197
--
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:
alexott opened a new pull request, #23199:
URL: https://github.com/apache/airflow/pull/23199
This PR includes following changes:
* Document missed parameters for `DatabricksSubmitRunOperator` and
`DatabricksRunNowOperator`
* Add support for new parameters in
alexott commented on code in PR #3:
URL: https://github.com/apache/airflow/pull/3#discussion_r857098184
##
docs/apache-airflow-providers-databricks/index.rst:
##
@@ -80,7 +80,8 @@ PIP requirements
PIP package Version required
github-actions[bot] commented on PR #23196:
URL: https://github.com/apache/airflow/pull/23196#issuecomment-1107778732
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and take the
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 ba973a7388 Fix constraint generation on CI
potiuk merged PR #23194:
URL: https://github.com/apache/airflow/pull/23194
--
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:
65 matches
Mail list logo