eladkal commented on code in PR #39336:
URL: https://github.com/apache/airflow/pull/39336#discussion_r1597486765
##
airflow/providers/dbt/cloud/CHANGELOG.rst:
##
@@ -28,6 +28,11 @@
Changelog
-
+main
+.
+
+In Airflow 2.10.0, we fix the way try_number works, so tha
YounHS commented on PR #39355:
URL: https://github.com/apache/airflow/pull/39355#issuecomment-2106116615
@jscheffl
sry, I checked this prob!
I initially thought that I would only need to modify the airflow/www side,
but for "security", "browse", and "admin", modifying views.py wou
rawwar commented on code in PR #39295:
URL: https://github.com/apache/airflow/pull/39295#discussion_r1597539214
##
airflow/providers/databricks/operators/databricks.py:
##
@@ -1056,13 +1073,24 @@ def monitor_databricks_job(self) -> None:
)
if not run_state.
rawwar commented on code in PR #39295:
URL: https://github.com/apache/airflow/pull/39295#discussion_r1597539214
##
airflow/providers/databricks/operators/databricks.py:
##
@@ -1056,13 +1073,24 @@ def monitor_databricks_job(self) -> None:
)
if not run_state.
rawwar closed pull request #39572: use `sqlqlchemy_url` property in `get_uri`
for `mssql` provider
URL: https://github.com/apache/airflow/pull/39572
--
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 t
rawwar opened a new pull request, #39572:
URL: https://github.com/apache/airflow/pull/39572
related: #38195
use sqlalchemy's URL object to create a URI in the `get_uri` method.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
rawwar commented on issue #39571:
URL: https://github.com/apache/airflow/issues/39571#issuecomment-2106074922
Can you confirm if this is happening consistently when you run 200+ tasks in
parallel
--
This is an automated message from the Apache Git Service.
To respond to the message, pleas
github-actions[bot] commented on PR #38001:
URL: https://github.com/apache/airflow/pull/38001#issuecomment-2106066273
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 pull request #38281: New generaic SQL transfer
operator proposal
URL: https://github.com/apache/airflow/pull/38281
--
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
github-actions[bot] commented on PR #38497:
URL: https://github.com/apache/airflow/pull/38497#issuecomment-2106066256
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
paramjeet01 opened a new issue, #39571:
URL: https://github.com/apache/airflow/issues/39571
### Apache Airflow version
Other Airflow 2 version (please specify below)
### If "Other Airflow 2 version" selected, which one?
2.8.3
### What happened?
Statsd stoppe
potiuk commented on PR #39513:
URL: https://github.com/apache/airflow/pull/39513#issuecomment-2106034576
Yep. All green. So now just waiting for comments / approvals.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
potiuk commented on PR #39513:
URL: https://github.com/apache/airflow/pull/39513#issuecomment-2106028192
OK. I removed it out of `DRAFT` state.
Looks like this one should get `green` - which means that we can run full
test suite against Airflow 2.9.1 - but we will need to get some res
potiuk commented on code in PR #39513:
URL: https://github.com/apache/airflow/pull/39513#discussion_r1597505481
##
tests/test_utils/compat.py:
##
@@ -0,0 +1,47 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NO
potiuk commented on code in PR #39513:
URL: https://github.com/apache/airflow/pull/39513#discussion_r1597505397
##
tests/providers/google/cloud/operators/test_dataproc.py:
##
@@ -1064,11 +1064,10 @@ def test_create_cluster_operator_extra_links(dag_maker,
create_task_instance_of
potiuk commented on code in PR #39513:
URL: https://github.com/apache/airflow/pull/39513#discussion_r1597505127
##
tests/providers/amazon/aws/executors/ecs/test_ecs_executor.py:
##
@@ -367,6 +368,7 @@ def test_stopped_tasks(self):
class TestAwsEcsExecutor:
"""Tests the AWS
RNHTTR opened a new pull request, #39570:
URL: https://github.com/apache/airflow/pull/39570
Closes: #35935
--
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,
potiuk commented on PR #39565:
URL: https://github.com/apache/airflow/pull/39565#issuecomment-2106007577
Ah . I see :)
--
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 un
jscheffl opened a new pull request, #39569:
URL: https://github.com/apache/airflow/pull/39569
This PR adds a feature to trigger DAG run handling, if called from Grid view
it will redirect to the new DAG Run that was generated.
If any task or detail tab was selected, the selection is carri
eladkal commented on PR #39565:
URL: https://github.com/apache/airflow/pull/39565#issuecomment-2106000758
> Am I right that you skipped all the other providers where only templates
were reapplied ?
>
> I think you should update all other provider changes and mark them as
"doc-only" -
boring-cyborg[bot] commented on PR #39568:
URL: https://github.com/apache/airflow/pull/39568#issuecomment-210531
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'
kisssam opened a new pull request, #39568:
URL: https://github.com/apache/airflow/pull/39568
---
This updates the label value validation logic for BigQuery label values as
documented in the [public
documentation](https://cloud.google.com/bigquery/docs/labels-intro#req
pedro-cf commented on issue #37935:
URL: https://github.com/apache/airflow/issues/37935#issuecomment-2105999397
> In 2.8.2 the main problem was the new DAG run was "hidden". This is what
in 2.9.1 now is fixed. The new DAG run appears after triggering.
That's interesting you mention it
kisssam opened a new issue, #39567:
URL: https://github.com/apache/airflow/issues/39567
### Apache Airflow Provider(s)
google
### Versions of Apache Airflow Providers
apache-airflow-providers-google==10.17.0
### Apache Airflow version
airflow-2.7.3
boring-cyborg[bot] commented on issue #39567:
URL: https://github.com/apache/airflow/issues/39567#issuecomment-2105998513
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
jscheffl commented on issue #37935:
URL: https://github.com/apache/airflow/issues/37935#issuecomment-2105998506
Okay, then I assume I now understand the mis-understanding. Please correct
me if I am right:
In 2.8.2 the main problem was the new DAG run was "hidden". This is what in
2.9
jscheffl opened a new pull request, #39566:
URL: https://github.com/apache/airflow/pull/39566
WIP PoC for AIP-68
--
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 unsubscr
potiuk commented on PR #39565:
URL: https://github.com/apache/airflow/pull/39565#issuecomment-2105981649
Am I right that you skipped all the other providers where only templates
were reapplied ?
I think you should update all other provider changes and mark them as
"doc-only" - the th
eladkal opened a new pull request, #39565:
URL: https://github.com/apache/airflow/pull/39565
Ad hoc release for the providers we didn't release in the 1st wave + the new
iceberg provider
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
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 1416d3662d Updating con
eladkal commented on code in PR #39336:
URL: https://github.com/apache/airflow/pull/39336#discussion_r1597486765
##
airflow/providers/dbt/cloud/CHANGELOG.rst:
##
@@ -28,6 +28,11 @@
Changelog
-
+main
+.
+
+In Airflow 2.10.0, we fix the way try_number works, so tha
eladkal commented on code in PR #39336:
URL: https://github.com/apache/airflow/pull/39336#discussion_r1597486765
##
airflow/providers/dbt/cloud/CHANGELOG.rst:
##
@@ -28,6 +28,11 @@
Changelog
-
+main
+.
+
+In Airflow 2.10.0, we fix the way try_number works, so tha
pedro-cf commented on issue #37935:
URL: https://github.com/apache/airflow/issues/37935#issuecomment-2105951470
Updated the issue description. Would be cool if someone could update the
label since it affects version 2.9.1 @eladkal
--
This is an automated message from the Apache Git Servi
potiuk commented on code in PR #39513:
URL: https://github.com/apache/airflow/pull/39513#discussion_r1597471910
##
tests/providers/amazon/aws/links/test_base_aws.py:
##
@@ -203,7 +204,10 @@ def test_link_serialize(self):
"""Test: Operator links should exist for serializ
TiDeane opened a new pull request, #39564:
URL: https://github.com/apache/airflow/pull/39564
Implements a small checkbox named "Use AND logic" under the "filter DAGs by
tag" box, allowing users to toggle between filtering DAGs using AND logic or OR
logic. If there are no matching DAGs, it r
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 defe4590e9 yandex provider: bump version for yq
Taragolis merged PR #39548:
URL: https://github.com/apache/airflow/pull/39548
--
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
dylanbstorey commented on PR #39559:
URL: https://github.com/apache/airflow/pull/39559#issuecomment-2105740417
Yeah that should be all.
--
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 specif
potiuk commented on PR #39563:
URL: https://github.com/apache/airflow/pull/39563#issuecomment-2105731291
Nice :)
--
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 unsubscr
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 092153c2e2 Update INSTALL (#39563)
092153c2e2 is de
potiuk merged PR #39563:
URL: https://github.com/apache/airflow/pull/39563
--
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
shahar1 commented on PR #39563:
URL: https://github.com/apache/airflow/pull/39563#issuecomment-2105727883
cc: @potiuk (changing this file doesn't assign reviewers automatically)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
shahar1 opened a new pull request, #39563:
URL: https://github.com/apache/airflow/pull/39563
Proofread the `INSTALL` file
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/contributing-
potiuk commented on code in PR #39513:
URL: https://github.com/apache/airflow/pull/39513#discussion_r1597433753
##
tests/providers/amazon/aws/links/test_base_aws.py:
##
@@ -203,7 +204,10 @@ def test_link_serialize(self):
"""Test: Operator links should exist for serializ
potiuk commented on code in PR #39513:
URL: https://github.com/apache/airflow/pull/39513#discussion_r1597430270
##
tests/providers/amazon/aws/links/test_base_aws.py:
##
@@ -203,7 +204,10 @@ def test_link_serialize(self):
"""Test: Operator links should exist for serializ
potiuk commented on code in PR #39513:
URL: https://github.com/apache/airflow/pull/39513#discussion_r1597429662
##
tests/providers/amazon/aws/links/test_base_aws.py:
##
@@ -203,7 +204,10 @@ def test_link_serialize(self):
"""Test: Operator links should exist for serializ
potiuk commented on code in PR #39513:
URL: https://github.com/apache/airflow/pull/39513#discussion_r1597429528
##
tests/providers/amazon/aws/links/test_base_aws.py:
##
@@ -203,7 +204,10 @@ def test_link_serialize(self):
"""Test: Operator links should exist for serializ
tankim opened a new issue, #39562:
URL: https://github.com/apache/airflow/issues/39562
### Description
When configuring Airflow using a Helm chart, the `core` section allows you
to set the `plugins_folder`. This setting is applied equally across all web
servers and schedulers. Many u
boring-cyborg[bot] commented on issue #39562:
URL: https://github.com/apache/airflow/issues/39562#issuecomment-2105683617
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
Taragolis commented on PR #39548:
URL: https://github.com/apache/airflow/pull/39548#issuecomment-2105680384
I've add missing changes
--
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 c
potiuk commented on code in PR #39513:
URL: https://github.com/apache/airflow/pull/39513#discussion_r1597427202
##
tests/providers/amazon/aws/links/test_base_aws.py:
##
@@ -203,7 +204,10 @@ def test_link_serialize(self):
"""Test: Operator links should exist for serializ
Taragolis commented on code in PR #39513:
URL: https://github.com/apache/airflow/pull/39513#discussion_r1597426660
##
tests/providers/amazon/aws/links/test_base_aws.py:
##
@@ -203,7 +204,10 @@ def test_link_serialize(self):
"""Test: Operator links should exist for seria
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 86caf84934 Remove Airflow 2.6 back compact code (#39558)
add 5fd1128085 Update uv to 0.42 (#39561)
No new revisio
potiuk merged PR #39561:
URL: https://github.com/apache/airflow/pull/39561
--
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 merged PR #39558:
URL: https://github.com/apache/airflow/pull/39558
--
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.
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 86caf84934 Remove Airflow 2.6 back compact code (#
potiuk opened a new pull request, #39561:
URL: https://github.com/apache/airflow/pull/39561
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pul
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 a22d042ace fix boring-cyborg.yml (#39557)
a22d042ac
potiuk merged PR #39557:
URL: https://github.com/apache/airflow/pull/39557
--
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
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 e637367958 Group id is mandatory configurtion optio
potiuk merged PR #39559:
URL: https://github.com/apache/airflow/pull/39559
--
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
boring-cyborg[bot] commented on PR #39560:
URL: https://github.com/apache/airflow/pull/39560#issuecomment-2105622673
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'
iamtodor opened a new pull request, #39560:
URL: https://github.com/apache/airflow/pull/39560
Replace `Connection.get` with `Connection. get_connection_from_secrets`
Resolves https://github.com/apache/airflow/issues/39475
---
**^ Add meaningful descrip
YounHS commented on code in PR #39355:
URL: https://github.com/apache/airflow/pull/39355#discussion_r1597388058
##
airflow/www/templates/appbuilder/navbar.html:
##
@@ -50,6 +50,13 @@
{% include 'appbuilder/navbar_menu.html' %}
+
Review Comment:
YounHS commented on code in PR #39355:
URL: https://github.com/apache/airflow/pull/39355#discussion_r1597386702
##
airflow/www/views.py:
##
@@ -719,8 +719,17 @@ def render_template(self, *args, **kwargs):
**kwargs,
)
+class CustomAirflow(AirflowBaseView):
potiuk commented on PR #39559:
URL: https://github.com/apache/airflow/pull/39559#issuecomment-2105612755
cc: @dylanbstorey -> our integration tests started to fail with latest
`confluent_kafka` -> and I **think** this PR should fix it, but would love your
pair of eyes on it.
--
This is
potiuk opened a new pull request, #39559:
URL: https://github.com/apache/airflow/pull/39559
The group.id parameter has been optional before 2.4.0 version of
confluent_kafka relased on 7th of May. This started to fail our integration
tests (cool).
This PR adds "group.id" as extra fiel
eladkal commented on issue #37583:
URL: https://github.com/apache/airflow/issues/37583#issuecomment-2105608675
@Ram-tripath please share real example of the issue `" {Ke" " `
i think the real issue you are trying to solve is not about hidden things
but more about having Json linter simil
eladkal opened a new pull request, #39558:
URL: https://github.com/apache/airflow/pull/39558
(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, e-
69 matches
Mail list logo
