dirrao commented on PR #41693:
URL: https://github.com/apache/airflow/pull/41693#issuecomment-2339670429
> Actually I think your changes are fine. It seems inevitable to force
Airflow 3 to use the latest FAB version provider. See comment
[here](https://github.com/apache/airflow/pull/42042#i
sid-habu commented on issue #35803:
URL: https://github.com/apache/airflow/issues/35803#issuecomment-2339665443
@hussein-awala With your merged PR, is it now possible to dynamically set
the `pool_slot` of a task based on an XCOM return value of an upstream task?
I am trying to custom
dirrao commented on PR #39908:
URL: https://github.com/apache/airflow/pull/39908#issuecomment-2339648771
Hi @kaxil / @ferruzzi,
I'm awaiting final approval to proceed with the merge. Could you please take
a look when you get a chance?
--
This is an automated message from the Apache Git
richard-iovanisci opened a new issue, #42125:
URL: https://github.com/apache/airflow/issues/42125
### Apache Airflow Provider(s)
hashicorp
### Versions of Apache Airflow Providers
3.7.1
### Apache Airflow version
2.9.3
### Operating System
Linu
mf-hyunsoo-kang commented on PR #42058:
URL: https://github.com/apache/airflow/pull/42058#issuecomment-2339547698
Sorry, I didn't push the changes.
`field_a` and `field_b` are renamed to `foo` and `bar`, since they have more
consistency with other documentations and they have shorter
bbovenzi opened a new pull request, #42124:
URL: https://github.com/apache/airflow/pull/42124
Move sort, filter and pagination table state to url parameters
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob
jscheffl commented on PR #42058:
URL: https://github.com/apache/airflow/pull/42058#issuecomment-2339522509
Still some failures in static checks. Have you tried running `pre-commit` in
your workspace or `breeze static-checks`?
--
This is an automated message from the Apache Git Service.
To
bbovenzi merged PR #42119:
URL: https://github.com/apache/airflow/pull/42119
--
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
This is an automated email from the ASF dual-hosted git repository.
bbovenzi 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 afc05322283 Improve DAGs table UI (#42119)
afc053
kand617 commented on issue #42107:
URL: https://github.com/apache/airflow/issues/42107#issuecomment-2339379582
Im on 2.8.3 and can confirm similar behaviour. Suddenly I see a large chunk
of scheduled tasks be marked for failure.
No logs or anything.
--
This is an automated message fro
github-actions[bot] closed pull request #40911: ISSUE-40525: Fix backfilling
CLI command where DAG files are missing
URL: https://github.com/apache/airflow/pull/40911
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
github-actions[bot] commented on issue #41121:
URL: https://github.com/apache/airflow/issues/41121#issuecomment-2339368897
This issue has been automatically marked as stale because it has been open
for 14 days with no response from the author. It will be closed in next 7 days
if no further
github-actions[bot] commented on issue #41340:
URL: https://github.com/apache/airflow/issues/41340#issuecomment-2339368838
This issue has been automatically marked as stale because it has been open
for 14 days with no response from the author. It will be closed in next 7 days
if no further
bugraoz93 commented on code in PR #59:
URL: https://github.com/apache/airflow-ci-infra/pull/59#discussion_r1751051531
##
runner/Dockerfile:
##
@@ -0,0 +1,35 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTIC
vincbeck commented on PR #41693:
URL: https://github.com/apache/airflow/pull/41693#issuecomment-2339084086
Actually I think your changes are fine. It seems inevitable to force Airflow
3 to use the latest FAB version provider. See comment
[here](https://github.com/apache/airflow/pull/42042#i
vincbeck commented on PR #42087:
URL: https://github.com/apache/airflow/pull/42087#issuecomment-2339063307
Static checks are failing, could you please fix them? See [documentation
](https://github.com/apache/airflow/blob/main/contributing-docs/08_static_code_checks.rst)
--
This is an auto
potiuk commented on issue #41525:
URL: https://github.com/apache/airflow/issues/41525#issuecomment-2339029396
cc: @o-nikolas
--
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.
This is an automated email from the ASF dual-hosted git repository.
vincbeck pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 0a81eb08d57 Fix failing compatibility test: importing missing saml for
old airflows (#42113)
add ff718bd7e3a Aw
dead-1ine commented on PR #42102:
URL: https://github.com/apache/airflow/pull/42102#issuecomment-2339008950
After further investigation, I found that the removal of `executor=None` was
intentional in the `v2-10-test` branch. Therefore, I'm closing this PR and will
open a new one to update t
This is an automated email from the ASF dual-hosted git repository.
vincbeck 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 0a81eb08d57 Fix failing compatibility test: impor
bbovenzi commented on code in PR #42095:
URL: https://github.com/apache/airflow/pull/42095#discussion_r1750644676
##
airflow/ui/src/components/DataTable.tsx:
##
@@ -0,0 +1,202 @@
+/*!
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch v2-10-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/v2-10-test by this push:
new 5371ffc67f4 Fix details tab log url det
potiuk closed issue #41838: Sometimes PubSubPullSensor doesn't pull messages
even if the PubSub subscription has unacked messages
URL: https://github.com/apache/airflow/issues/41838
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
boring-cyborg[bot] commented on PR #42115:
URL: https://github.com/apache/airflow/pull/42115#issuecomment-2338556348
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 Contributors'
R7L208 opened a new pull request, #42115:
URL: https://github.com/apache/airflow/pull/42115
---
The Databricks Provider did not implement `on_kill` or another mechanism to
cancel either tasks generated by `_CreateDatabricksWorkflowOperator` or SQL
queries subm
raphaelauv commented on issue #39332:
URL: https://github.com/apache/airflow/issues/39332#issuecomment-2338457310
I have the problem with 2.10.1 and only deterministic dags ( not using any
dynamic config for the dag authoring )
and
`airflow_dag_processing_total_parse_time` is
jkramer-ginkgo commented on issue #41981:
URL: https://github.com/apache/airflow/issues/41981#issuecomment-2338443302
@potiuk It is still broken in `2.10.1`.. I should've mentioned I had tested
against a manually patched version with
https://github.com/apache/airflow/pull/41605
--
This i
pierrejeambrun commented on PR #42063:
URL: https://github.com/apache/airflow/pull/42063#issuecomment-2338442838
Thank you for the pull request.
That seems a little odd that the normal string "Something" is considered a
string (most things are then truthy), but casting it to lower som
jcrobak commented on issue #42111:
URL: https://github.com/apache/airflow/issues/42111#issuecomment-2338425708
here's a quick repro with docker-compose, in case you need it:
```
curl -LfO
'https://airflow.apache.org/docs/apache-airflow/2.10.1/docker-compose.yaml'
sed -e '/\
potiuk merged PR #42108:
URL: https://github.com/apache/airflow/pull/42108
--
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
pierrejeambrun opened a new pull request, #42114:
URL: https://github.com/apache/airflow/pull/42114
Backporting of https://github.com/apache/airflow/pull/42104
--
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
pierrejeambrun merged PR #42104:
URL: https://github.com/apache/airflow/pull/42104
--
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
ephraimbuddy commented on issue #42111:
URL: https://github.com/apache/airflow/issues/42111#issuecomment-2338358642
> cc: @utkarsharma2 @ephraimbuddy -> I think we need a quick 2.10.2
Yeah. Looking into this
--
This is an automated message from the Apache Git Service.
To respond to
potiuk commented on issue #42111:
URL: https://github.com/apache/airflow/issues/42111#issuecomment-2338321721
> I'm experiencing this issue with 2.10.1, too. I was able to add some
logging and it appears that https://github.com/apache/airflow/pull/41433 is the
culprit. If the value of airfl
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 7c3dea44e7e fix indentation for unit test contribut
jcrobak commented on issue #42111:
URL: https://github.com/apache/airflow/issues/42111#issuecomment-2338303854
I'm experiencing this issue with 2.10.1, too. I was able to add some logging
and it appears that https://github.com/apache/airflow/pull/41433 is the
culprit. If the value of `airfl
ldacey opened a new issue, #42111:
URL: https://github.com/apache/airflow/issues/42111
### Apache Airflow version
2.10.0
### If "Other Airflow 2 version" selected, which one?
_No response_
### What happened?
I updated to Airflow 2.10.1 and the DAGs in my UI
potiuk commented on PR #42110:
URL: https://github.com/apache/airflow/pull/42110#issuecomment-2338276812
Just in time for the upcoming contribution workshop @RNHTTR !
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
potiuk opened a new pull request, #42110:
URL: https://github.com/apache/airflow/pull/42110
The header levels in the unit test contribution docs has been messed up.
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https:
potiuk commented on PR #41916:
URL: https://github.com/apache/airflow/pull/41916#issuecomment-2338248978
> @eladkal Hi! Could you please help me understand what I should do to fix
the failing tests? :(
There are regular unit tests failing and they should be fixed. For
compatibility t
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 22bb433a8f9 Deprecated metrics configuration remove
potiuk merged PR #42060:
URL: https://github.com/apache/airflow/pull/42060
--
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 opened a new pull request, #42108:
URL: https://github.com/apache/airflow/pull/42108
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pul
boring-cyborg[bot] commented on PR #42106:
URL: https://github.com/apache/airflow/pull/42106#issuecomment-2338226346
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.
boring-cyborg[bot] commented on issue #42107:
URL: https://github.com/apache/airflow/issues/42107#issuecomment-2338220140
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
potiuk commented on code in PR #42004:
URL: https://github.com/apache/airflow/pull/42004#discussion_r1750302628
##
airflow/auth/managers/simple/simple_auth_manager.py:
##
@@ -0,0 +1,166 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor li
boring-cyborg[bot] commented on PR #42106:
URL: https://github.com/apache/airflow/pull/42106#issuecomment-2338174880
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 Contributors'
raphaelauv opened a new issue, #42105:
URL: https://github.com/apache/airflow/issues/42105
### Apache Airflow version
2.10.0
### If "Other Airflow 2 version" selected, which one?
_No response_
### What happened?
I have regularly this error in the logs of the
pierrejeambrun opened a new pull request, #42104:
URL: https://github.com/apache/airflow/pull/42104
Use the same regexp that we have for the legacy log page which is more
complex and robust. This will also enforce consistancy between the two pages.
Cf screenshot the url was not properly mat
boring-cyborg[bot] commented on PR #42103:
URL: https://github.com/apache/airflow/pull/42103#issuecomment-2338145485
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 Contributors'
potiuk merged PR #42091:
URL: https://github.com/apache/airflow/pull/42091
--
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
dead-1ine opened a new pull request, #42102:
URL: https://github.com/apache/airflow/pull/42102
Upon checking the blame history, I found that there were no specific changes
that removed the executor=None keyword argument from this method. This argument
was present in version 2.9, but for rea
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 4950c62743 Fix the system test: dataflow_native_jav
potiuk commented on PR #42101:
URL: https://github.com/apache/airflow/pull/42101#issuecomment-2338045273
Backport of #42090
--
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, #42101:
URL: https://github.com/apache/airflow/pull/42101
…090)
Universal Pathlib 0.2.4 adds extra validation for Paths and our integration
with local file paths Does not work with it. The 0.2.5 version fixes it.
Tracked in https://github.com/f
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 2517da4769 Limit universal-pathlib below 0.2.4 as i
potiuk merged PR #42090:
URL: https://github.com/apache/airflow/pull/42090
--
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 #42090:
URL: https://github.com/apache/airflow/pull/42090#issuecomment-2338040619
Random k8S failure. Merging.
--
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
michaelimas1 commented on PR #41078:
URL: https://github.com/apache/airflow/pull/41078#issuecomment-2337943429
@eladkal anything else missing from my side to get this approved? it's my
first PR here so i'm not sure what's the process
--
This is an automated message from the Apache Git Ser
potiuk commented on PR #42098:
URL: https://github.com/apache/airflow/pull/42098#issuecomment-2337920513
Cool. I found this one https://github.com/OpenAPITools/openapi-diff - maybe
we can also (later when we get to be much closer) to employ that one to produce
the diff ?
--
This is an au
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.git
The following commit(s) were added to refs/heads/main by this push:
new 49e011f6d1 Add openapi generation hook (#42
gopidesupavan commented on code in PR #42080:
URL: https://github.com/apache/airflow/pull/42080#discussion_r1750066174
##
airflow/providers/google/cloud/triggers/pubsub.py:
##
@@ -106,7 +106,10 @@ async def run(self) -> AsyncIterator[TriggerEvent]: #
type: ignore[override]
VladaZakharova commented on issue #41194:
URL: https://github.com/apache/airflow/issues/41194#issuecomment-2337829130
> Can i take this? 👀 If possible, please assign this issue to me. (it will
be done until day after tomorrow)
sure, thank you :)
--
This is an automated message
jrderuiter commented on PR #41639:
URL: https://github.com/apache/airflow/pull/41639#issuecomment-2337828740
I see some of the static checks are failing, can you have a look at this
@basvandriel? Afterwards we should be fine to merge if @BasPH agrees.
--
This is an automated message from
potiuk closed issue #42025: Test Airflow Release step is flaky.
URL: https://github.com/apache/airflow/issues/42025
--
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 unsubscri
harjeevanmaan commented on PR #42086:
URL: https://github.com/apache/airflow/pull/42086#issuecomment-2337789559
cc @utkarsharma2 @uranusjr
--
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 spe
MaksYermak commented on code in PR #42080:
URL: https://github.com/apache/airflow/pull/42080#discussion_r1749993868
##
airflow/providers/google/cloud/triggers/pubsub.py:
##
@@ -106,7 +106,10 @@ async def run(self) -> AsyncIterator[TriggerEvent]: #
type: ignore[override]
molcay opened a new pull request, #42099:
URL: https://github.com/apache/airflow/pull/42099
This PR is adding a logic to use a shared folder for the
`example_dataflow_native_java` system test.
---
**^ Add meaningful description above**
Read the **[Pul
pierrejeambrun opened a new pull request, #42098:
URL: https://github.com/apache/airflow/pull/42098
Add a new pre-commit hook and utility script to persist to file the FastAPI
openapi spec.
This will be useful for a follow up PR that will generate front-end types
and query based on t
sc-anssi commented on PR #40414:
URL: https://github.com/apache/airflow/pull/40414#issuecomment-2337585317
> @sc-anssi So then I would say... if you fix the static checks then we
could merge it and back-port to a 2.10.2 release...
Spelling is fixed, please let me know if anything else
pierrejeambrun commented on PR #42095:
URL: https://github.com/apache/airflow/pull/42095#issuecomment-2337371757
I don't think we planned on adding coverage for front-end tests in a near
future but that might be a great addition indeed.
There are some static checks failure to fix.
Pad71 opened a new issue, #42096:
URL: https://github.com/apache/airflow/issues/42096
### Apache Airflow version
Other Airflow 2 version (please specify below)
### If "Other Airflow 2 version" selected, which one?
2.10.1
### What happened?
There was a bug in
github-actions[bot] commented on issue #33078:
URL: https://github.com/apache/airflow/issues/33078#issuecomment-2337286288
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
73 matches
Mail list logo
