SatishChGit closed pull request #39217: Updates to Teradata Provider
URL: https://github.com/apache/airflow/pull/39217
--
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
amoghrajesh opened a new pull request, #39359:
URL: https://github.com/apache/airflow/pull/39359
Due to missing validation check for PR users, sometimes, it is possible that
empty "@" can get into the issue content. See attached screenshot:
I confirmed my patch is in the rc1.
bmoon4 opened a new issue, #39356:
URL: https://github.com/apache/airflow/issues/39356
### Apache Airflow version
Other Airflow 2 version (please specify below)
### If "Other Airflow 2 version" selected, which one?
2.8.4
### What happened?
I am using
gaurav7261 commented on PR #39110:
URL: https://github.com/apache/airflow/pull/39110#issuecomment-2089364303
Hi @Lee-W @pankajkoti , raised PR for runnow operator separately as
discussed, please review https://github.com/apache/airflow/pull/39354
--
This is an automated message from the
boring-cyborg[bot] commented on PR #39355:
URL: https://github.com/apache/airflow/pull/39355#issuecomment-2089360402
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
YounHS opened a new pull request, #39355:
URL: https://github.com/apache/airflow/pull/39355
related: #11334
We've been seeing requests for a dark mode feature in the Airflow UI for a
while now.
I tested it using docker-compose.yaml.
After testing, I was able to set
bmoon4 commented on issue #37442:
URL: https://github.com/apache/airflow/issues/37442#issuecomment-2089343933
Hi, I'm experiencing same issue. Airflow is not picking up variables from a
connection (`smtp_default`). It is just looking for `env vars` such as
`AIRFLOW__SMTP__` or conf
gaurav7261 opened a new pull request, #39354:
URL: https://github.com/apache/airflow/pull/39354
1. logic for getting main notebook exception via `get_run_output` api
calling and raising the exception with all failed tasks errors
**^ Add meaningful
github-actions[bot] commented on PR #37087:
URL: https://github.com/apache/airflow/pull/37087#issuecomment-2089321356
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
github-actions[bot] closed pull request #38069: Add the possibility of custom
probes in flower helm deployment
URL: https://github.com/apache/airflow/pull/38069
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
github-actions[bot] commented on PR #37151:
URL: https://github.com/apache/airflow/pull/37151#issuecomment-2089321318
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
o-nikolas commented on code in PR #38658:
URL: https://github.com/apache/airflow/pull/38658#discussion_r1586906672
##
tests/providers/amazon/aws/operators/test_emr_containers.py:
##
@@ -144,6 +144,20 @@ def test_operator_defer(self, mock_submit_job,
mock_check_query_status):
o-nikolas commented on PR #38658:
URL: https://github.com/apache/airflow/pull/38658#issuecomment-2089264782
Tests need to pass after your rebase from main. Otherwise looks good to merge
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
VertexToEdge commented on code in PR #39345:
URL: https://github.com/apache/airflow/pull/39345#discussion_r1586902396
##
airflow/www/app.py:
##
@@ -120,6 +120,10 @@ def create_app(config=None, testing=False):
)
cookie_samesite_config = "Lax"
dstandish commented on code in PR #39336:
URL: https://github.com/apache/airflow/pull/39336#discussion_r1586874582
##
tests/sensors/test_base.py:
##
@@ -382,9 +398,12 @@ def _get_tis():
# Task is cleared
sensor.clear()
sensor_ti, dummy_ti = _get_tis()
dstandish commented on code in PR #39336:
URL: https://github.com/apache/airflow/pull/39336#discussion_r1586872663
##
airflow/models/taskinstance.py:
##
@@ -280,14 +280,13 @@ def clear_task_instances(
ti.refresh_from_task(task)
if
dondaum commented on issue #39352:
URL: https://github.com/apache/airflow/issues/39352#issuecomment-2089204662
Would be happy to take it over.
--
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
dondaum commented on PR #39299:
URL: https://github.com/apache/airflow/pull/39299#issuecomment-2089195859
> > Fixed but still raise a warning
> >
beobest2 commented on code in PR #38658:
URL: https://github.com/apache/airflow/pull/38658#discussion_r1586861317
##
tests/providers/amazon/aws/operators/test_emr_containers.py:
##
@@ -144,6 +144,20 @@ def test_operator_defer(self, mock_submit_job,
mock_check_query_status):
dondaum commented on issue #39346:
URL: https://github.com/apache/airflow/issues/39346#issuecomment-2089183283
#38961 is good. ✅
--
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
jedcunningham merged PR #39350:
URL: https://github.com/apache/airflow/pull/39350
--
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.
jedcunningham pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 29a9a24a5f Refactor tests re sensor reschedule mode and try_number
(#39351)
add 4910c0439b Add `jmespath`
SangwanP commented on issue #22006:
URL: https://github.com/apache/airflow/issues/22006#issuecomment-2089068306
I have an alternative solution; someone with a deeper knowledge of the code
base can evaluate it:
If one is able to 'disable' the schedule on a DAG, the 'drain' functionality
KimboTodd commented on issue #37752:
URL: https://github.com/apache/airflow/issues/37752#issuecomment-2089067948
We need better monitoring in astronomer and this would help a lot.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
eladkal commented on issue #22006:
URL: https://github.com/apache/airflow/issues/22006#issuecomment-209489
@jscheffl The way I imagine it is that the drain status is temporary (for
the duration of the drain). once drain is finished it should be switched to
pause. So at any given time
jscheffl commented on issue #22006:
URL: https://github.com/apache/airflow/issues/22006#issuecomment-200568
I feel linke when going into details of database, DAG model, REST API,
CLI... UI Experience this is a very complex thing. I agree that this option is
useful.
On the other hand
rasulkarimov commented on PR #39031:
URL: https://github.com/apache/airflow/pull/39031#issuecomment-2088870335
> look good, can you fix current tests to validate your changes?
Thank you! Fixed tests.
--
This is an automated message from the Apache Git Service.
To respond to the
Anush008 commented on issue #39346:
URL: https://github.com/apache/airflow/issues/39346#issuecomment-2088866714
> Provider [qdrant:
1.1.0rc1](https://pypi.org/project/apache-airflow-providers-qdrant/1.1.0rc1)
https://github.com/apache/airflow/pull/39273
Tested. Works as
This is an automated email from the ASF dual-hosted git repository.
dstandish pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 413698ed1d Use correct prefixes for distribution packages in provider
documentation (#39341)
add 29a9a24a5f
dstandish merged PR #39351:
URL: https://github.com/apache/airflow/pull/39351
--
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:
eladkal commented on issue #22006:
URL: https://github.com/apache/airflow/issues/22006#issuecomment-2088864728
I prefer option 1.
We need to remember that pausing DAG can also be done from API/CLI and not
just the UI.
We can simply deprecate `is_paused_upon_creation` and replace it
jedcunningham commented on code in PR #39351:
URL: https://github.com/apache/airflow/pull/39351#discussion_r1586579778
##
tests/sensors/test_base.py:
##
@@ -284,62 +284,59 @@ def test_ok_with_reschedule(self, make_sensor,
time_machine, task_reschedules_fo
if
gopidesupavan commented on issue #39346:
URL: https://github.com/apache/airflow/issues/39346#issuecomment-2088845850
@eladkal https://github.com/apache/airflow/pull/39300 tested and working
correctly.
--
This is an automated message from the Apache Git Service.
To respond to the message,
Taragolis opened a new issue, #39352:
URL: https://github.com/apache/airflow/issues/39352
### Body
Originally reported in
[Slack](https://apache-airflow.slack.com/archives/C06K9Q5G2UA/p1714533835490519)
**Jed Cunningham 7:23 AM**
Feels like we might have missed mocking
o-nikolas commented on code in PR #39212:
URL: https://github.com/apache/airflow/pull/39212#discussion_r1586556327
##
airflow/providers/amazon/aws/executors/ecs/ecs_executor.py:
##
@@ -400,7 +400,7 @@ def attempt_task_runs(self):
else:
task =
dstandish commented on code in PR #39351:
URL: https://github.com/apache/airflow/pull/39351#discussion_r1586546901
##
tests/sensors/test_base.py:
##
@@ -284,62 +284,59 @@ def test_ok_with_reschedule(self, make_sensor,
time_machine, task_reschedules_fo
if
Bowrna commented on PR #38683:
URL: https://github.com/apache/airflow/pull/38683#issuecomment-2088810739
I have added the changes and included the toggle_dag_callback inside models.
But I am not sure how I can test the changes that adds a new table. Can anyone
guide me with any sample PR
dstandish commented on code in PR #39351:
URL: https://github.com/apache/airflow/pull/39351#discussion_r1586529415
##
tests/sensors/test_base.py:
##
@@ -284,62 +284,59 @@ def test_ok_with_reschedule(self, make_sensor,
time_machine, task_reschedules_fo
if
dstandish commented on code in PR #39351:
URL: https://github.com/apache/airflow/pull/39351#discussion_r1586529415
##
tests/sensors/test_base.py:
##
@@ -284,62 +284,59 @@ def test_ok_with_reschedule(self, make_sensor,
time_machine, task_reschedules_fo
if
dstandish commented on code in PR #39351:
URL: https://github.com/apache/airflow/pull/39351#discussion_r1586525580
##
tests/sensors/test_base.py:
##
@@ -284,62 +284,59 @@ def test_ok_with_reschedule(self, make_sensor,
time_machine, task_reschedules_fo
if
jedcunningham commented on code in PR #39351:
URL: https://github.com/apache/airflow/pull/39351#discussion_r1586522534
##
tests/sensors/test_base.py:
##
@@ -284,62 +284,59 @@ def test_ok_with_reschedule(self, make_sensor,
time_machine, task_reschedules_fo
if
jscheffl commented on issue #39326:
URL: https://github.com/apache/airflow/issues/39326#issuecomment-2088781071
Checked and my patch is in.
--
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
jedcunningham commented on code in PR #39351:
URL: https://github.com/apache/airflow/pull/39351#discussion_r1586519165
##
tests/sensors/test_base.py:
##
@@ -284,62 +284,59 @@ def test_ok_with_reschedule(self, make_sensor,
time_machine, task_reschedules_fo
if
shahar1 commented on code in PR #39315:
URL: https://github.com/apache/airflow/pull/39315#discussion_r1586508577
##
airflow/providers/google/cloud/operators/bigquery.py:
##
@@ -1026,7 +1045,10 @@ def execute(self, context: Context):
if not self.table_project_id:
jedcunningham commented on code in PR #39350:
URL: https://github.com/apache/airflow/pull/39350#discussion_r1586509024
##
airflow/providers/amazon/provider.yaml:
##
@@ -103,6 +103,7 @@ dependencies:
# We should investigate if we can upgrade to a newer version of lxml and
shahar1 commented on code in PR #39315:
URL: https://github.com/apache/airflow/pull/39315#discussion_r1586508577
##
airflow/providers/google/cloud/operators/bigquery.py:
##
@@ -1026,7 +1045,10 @@ def execute(self, context: Context):
if not self.table_project_id:
dstandish opened a new pull request, #39351:
URL: https://github.com/apache/airflow/pull/39351
It's a little clearer what's going on when we're not in a loop or behind an
`assert...` helper function.
Additionally I add asserts immediately after calling `clear`.
This will make
syedahsn commented on code in PR #39245:
URL: https://github.com/apache/airflow/pull/39245#discussion_r1586500169
##
airflow/providers/amazon/aws/sensors/bedrock.py:
##
@@ -211,3 +262,149 @@ def execute(self, context: Context) -> Any:
)
else:
boraberke commented on PR #38868:
URL: https://github.com/apache/airflow/pull/38868#issuecomment-2088750430
@Lee-W,
I have tested and here how it works:
When first run already uses `steps_override, schema_override,
additional_run_config` and the first run is failed, rerunning
syedahsn commented on code in PR #39245:
URL: https://github.com/apache/airflow/pull/39245#discussion_r1586497806
##
airflow/providers/amazon/aws/sensors/bedrock.py:
##
@@ -211,3 +262,149 @@ def execute(self, context: Context) -> Any:
)
else:
shahar1 commented on code in PR #39315:
URL: https://github.com/apache/airflow/pull/39315#discussion_r1586490522
##
airflow/providers/google/cloud/operators/bigquery.py:
##
@@ -1035,31 +1057,45 @@ def execute(self, context: Context):
)
if not
shahar1 commented on code in PR #39315:
URL: https://github.com/apache/airflow/pull/39315#discussion_r1586490522
##
airflow/providers/google/cloud/operators/bigquery.py:
##
@@ -1035,31 +1057,45 @@ def execute(self, context: Context):
)
if not
Taragolis commented on code in PR #39350:
URL: https://github.com/apache/airflow/pull/39350#discussion_r1586488684
##
airflow/providers/fab/provider.yaml:
##
@@ -50,6 +50,7 @@ dependencies:
- flask-appbuilder==4.4.1
- flask-login>=0.6.2
- google-re2>=1.0
+ - jmsepath
shahar1 commented on code in PR #39315:
URL: https://github.com/apache/airflow/pull/39315#discussion_r1586489531
##
airflow/providers/google/cloud/triggers/bigquery.py:
##
@@ -235,7 +237,11 @@ async def run(self) -> AsyncIterator[TriggerEvent]: #
type: ignore[override]
Taragolis commented on PR #39349:
URL: https://github.com/apache/airflow/pull/39349#issuecomment-2088718716
You need to remove this line:
https://github.com/apache/airflow/blob/c434c6b98911d71f27910d7d30cf966462157612/tests/always/test_project_structure.py#L75
--
This is an
wlinamchurch commented on issue #39346:
URL: https://github.com/apache/airflow/issues/39346#issuecomment-2088714308
@eladkal https://github.com/apache/airflow/pull/38911 tested and working
correctly.
--
This is an automated message from the Apache Git Service.
To respond to the message,
Taragolis commented on issue #35442:
URL: https://github.com/apache/airflow/issues/35442#issuecomment-2088704233
There is pretty simple rule there if providers module placed into the
`airflow/providers/amazon/aws/operators/emr.py` then tests should be placed
into the
mateuslatrova opened a new pull request, #39349:
URL: https://github.com/apache/airflow/pull/39349
Related: https://github.com/apache/airflow/issues/35442
This PR adds unit tests for the EksCreateClusterTrigger trigger from the
Airflow AWS provider
Taragolis commented on code in PR #39345:
URL: https://github.com/apache/airflow/pull/39345#discussion_r1586433872
##
airflow/www/app.py:
##
@@ -120,6 +120,10 @@ def create_app(config=None, testing=False):
)
cookie_samesite_config = "Lax"
bbovenzi commented on PR #39345:
URL: https://github.com/apache/airflow/pull/39345#issuecomment-2088631413
Nice catch! We could probably cache it for a few weeks. Assets will only
change between airflow version updates.
--
This is an automated message from the Apache Git Service.
To
dabla commented on issue #39346:
URL: https://github.com/apache/airflow/issues/39346#issuecomment-2088602198
I know there is still an issue with the MSGraphSensor in that sense that
it’s not working correctly in an async way and sometimes an event loop closed
error can occur. This
Taragolis commented on PR #39341:
URL: https://github.com/apache/airflow/pull/39341#issuecomment-2088588913
> LGTM. Looks like this is due to recent pip support for underscore in
packager names.
This one about normalisation described in
[PEP-625](https://peps.python.org/pep-0625/)
eladkal commented on PR #39305:
URL: https://github.com/apache/airflow/pull/39305#issuecomment-2088586224
Supressed by https://github.com/apache/airflow/pull/39341
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
eladkal closed pull request #39305:
[fix](doc)apache_airflow_providers_apache_flink download link failure
URL: https://github.com/apache/airflow/pull/39305
--
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.
eladkal 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 413698ed1d Use correct prefixes for distribution
eladkal merged PR #39341:
URL: https://github.com/apache/airflow/pull/39341
--
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:
TJaniF commented on issue #39144:
URL: https://github.com/apache/airflow/issues/39144#issuecomment-2088554851
> is there a workaround for this until the fix is deployed? I can't run my
dev environment locally
If you are referring to the Astro CLI local environment you can set
nathadfield commented on issue #39346:
URL: https://github.com/apache/airflow/issues/39346#issuecomment-2088553502
@eladkal #39248 and #38736 tested and working correctly.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
vincbeck commented on issue #39144:
URL: https://github.com/apache/airflow/issues/39144#issuecomment-2088492522
Besides removing the `@has_access` from the view you defined in your plugin,
I dont see any. I am not saying this is something you should do. It really
depends on your setup and
mateuslatrova commented on issue #35442:
URL: https://github.com/apache/airflow/issues/35442#issuecomment-2088490071
> @mateuslatrova Some test modules from your list do not exists or named
differently
>
> ### Exists
> I've marked this as completed
>
> *
tagoad commented on issue #39144:
URL: https://github.com/apache/airflow/issues/39144#issuecomment-2088477101
is there a workaround for this until the fix is deployed? I can't run my dev
environment locally
--
This is an automated message from the Apache Git Service.
To respond to the
This is an automated email from the ASF dual-hosted git repository.
eladkal 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 fd9f9efe68 Fix Jira CHANGELOG.rst (#39347)
eladkal merged PR #39347:
URL: https://github.com/apache/airflow/pull/39347
--
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:
pankajastro commented on PR #37279:
URL: https://github.com/apache/airflow/pull/37279#issuecomment-2088470139
hmm, this will require small changes PR
https://github.com/apache/airflow/pull/39348
--
This is an automated message from the Apache Git Service.
To respond to the message,
pankajastro opened a new pull request, #39348:
URL: https://github.com/apache/airflow/pull/39348
---
**^ Add meaningful description above**
Read the **[Pull Request
This is an automated email from the ASF dual-hosted git repository.
eladkal pushed a change to branch add-documentation-2024-05-01
in repository https://gitbox.apache.org/repos/asf/airflow-site.git
from 07f08cdfe1 Add documentation for packages - 2024-05-01
add ead07a1760 Fix duplicate
eladkal opened a new pull request, #39347:
URL: https://github.com/apache/airflow/pull/39347
(no comment)
--
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,
eladkal opened a new issue, #39346:
URL: https://github.com/apache/airflow/issues/39346
### Body
I have a kind request for all the contributors to the latest provider
packages release.
Could you please help us to test the RC versions of the providers?
The guidelines on how
eladkal opened a new pull request, #1009:
URL: https://github.com/apache/airflow-site/pull/1009
(no comment)
--
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
This is an automated email from the ASF dual-hosted git repository.
eladkal pushed a change to branch add-documentation-2024-05-01
in repository https://gitbox.apache.org/repos/asf/airflow-site.git
at 07f08cdfe1 Add documentation for packages - 2024-05-01
This branch includes the
VertexToEdge opened a new pull request, #39345:
URL: https://github.com/apache/airflow/pull/39345
close #39344
Above Airflow 2.3.0, webserver's static file caching is disabled because of
major updated Flask's breaking change. In Flask 2.0.x, default value of
boring-cyborg[bot] commented on PR #39345:
URL: https://github.com/apache/airflow/pull/39345#issuecomment-2088394243
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
Added:
dev/airflow/providers/apache_airflow_providers_salesforce-5.7.0.tar.gz.sha512
==
---
dev/airflow/providers/apache_airflow_providers_salesforce-5.7.0.tar.gz.sha512
(added)
+++
Added:
dev/airflow/providers/apache_airflow_providers_http-4.11.0-py3-none-any.whl.asc
==
---
dev/airflow/providers/apache_airflow_providers_http-4.11.0-py3-none-any.whl.asc
(added)
+++
Added:
dev/airflow/providers/apache_airflow_providers_apache_livy-3.8.0-py3-none-any.whl.sha512
==
---
dev/airflow/providers/apache_airflow_providers_apache_livy-3.8.0-py3-none-any.whl.sha512
(added)
+++
Author: eladkal
Date: Wed May 1 11:53:48 2024
New Revision: 68905
Log:
Add artifacts for Airflow Providers 2024-05-01
Added:
dev/airflow/providers/apache_airflow_providers_airbyte-3.8.0-py3-none-any.whl
(with props)
This is an automated email from the ASF dual-hosted git repository.
eladkal pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 7a0fa64997 Use `upload_files_v2` Slack SDK method by default in Slack
Operators (#39340)
add fe4605a10e Prepare
eladkal merged PR #39328:
URL: https://github.com/apache/airflow/pull/39328
--
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 issue #39344:
URL: https://github.com/apache/airflow/issues/39344#issuecomment-2088326239
> if there is a better way to do this without changing the code
Better way contribute this changes to Airflow codebase. I do not sure that
you could provide additional
This is an automated email from the ASF dual-hosted git repository.
eladkal 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 7a0fa64997 Use `upload_files_v2` Slack SDK method
eladkal merged PR #39340:
URL: https://github.com/apache/airflow/pull/39340
--
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 issue #39344:
URL: https://github.com/apache/airflow/issues/39344#issuecomment-2088277101
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
VertexToEdge opened a new issue, #39344:
URL: https://github.com/apache/airflow/issues/39344
### Apache Airflow version
Other Airflow 2 version (please specify below)
### If "Other Airflow 2 version" selected, which one?
2.3.x ~ 2.9.x
### What happened?
Taragolis closed issue #39342: New release covering latest main
URL: https://github.com/apache/airflow/issues/39342
--
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
1 - 100 of 136 matches
Mail list logo
