Lee-W commented on code in PR #34652:
URL: https://github.com/apache/airflow/pull/34652#discussion_r1339602913
##
airflow/sensors/external_task.py:
##
@@ -163,9 +163,11 @@ def __init__(
total_states = set(self.allowed_states + self.skipped_states +
self.failed_states)
Lee-W commented on code in PR #34652:
URL: https://github.com/apache/airflow/pull/34652#discussion_r1339602707
##
airflow/sensors/external_task.py:
##
@@ -163,9 +163,11 @@ def __init__(
total_states = set(self.allowed_states + self.skipped_states +
self.failed_states)
utkarsharma2 commented on code in PR #34652:
URL: https://github.com/apache/airflow/pull/34652#discussion_r1339594679
##
airflow/sensors/external_task.py:
##
@@ -163,9 +163,11 @@ def __init__(
total_states = set(self.allowed_states + self.skipped_states +
self.failed_s
utkarsharma2 commented on PR #34652:
URL: https://github.com/apache/airflow/pull/34652#issuecomment-1738537373
> No need to add this patch in the core operators, we already fixed it, and
for that we add the comment (TODO: remove this if block when
min_airflow_version is set to higher than 2
Lee-W commented on code in PR #34652:
URL: https://github.com/apache/airflow/pull/34652#discussion_r1339583073
##
airflow/sensors/external_task.py:
##
@@ -163,9 +163,11 @@ def __init__(
total_states = set(self.allowed_states + self.skipped_states +
self.failed_states)
utkarsharma2 commented on code in PR #34652:
URL: https://github.com/apache/airflow/pull/34652#discussion_r1339576279
##
airflow/sensors/external_task.py:
##
@@ -163,9 +163,11 @@ def __init__(
total_states = set(self.allowed_states + self.skipped_states +
self.failed_s
utkarsharma2 commented on code in PR #34652:
URL: https://github.com/apache/airflow/pull/34652#discussion_r1339576279
##
airflow/sensors/external_task.py:
##
@@ -163,9 +163,11 @@ def __init__(
total_states = set(self.allowed_states + self.skipped_states +
self.failed_s
utkarsharma2 commented on code in PR #34652:
URL: https://github.com/apache/airflow/pull/34652#discussion_r1339572230
##
airflow/sensors/external_task.py:
##
@@ -163,9 +163,11 @@ def __init__(
total_states = set(self.allowed_states + self.skipped_states +
self.failed_s
utkarsharma2 commented on code in PR #34652:
URL: https://github.com/apache/airflow/pull/34652#discussion_r1339572230
##
airflow/sensors/external_task.py:
##
@@ -163,9 +163,11 @@ def __init__(
total_states = set(self.allowed_states + self.skipped_states +
self.failed_s
jens-scheffler-bosch commented on issue #34634:
URL: https://github.com/apache/airflow/issues/34634#issuecomment-1738511649
> Or perhaps you are saying "kill switch is dangerous but we should have it
anyway"?
I was referring to the "danger" that if suh kill switch would be implemented
Joffreybvn commented on code in PR #34606:
URL: https://github.com/apache/airflow/pull/34606#discussion_r1339558401
##
airflow/providers/http/operators/http.py:
##
@@ -49,6 +50,12 @@ class SimpleHttpOperator(BaseOperator):
:param data: The data to pass. POST-data in POST/PU
eladkal commented on code in PR #34514:
URL: https://github.com/apache/airflow/pull/34514#discussion_r1339558030
##
airflow/providers/google/cloud/operators/datafusion.py:
##
@@ -34,21 +34,12 @@
from airflow.providers.google.cloud.operators.cloud_base import
GoogleCloudBaseOpe
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 e050c92354 Updating con
This is an automated email from the ASF dual-hosted git repository.
jedcunningham 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 19450e03f5 Restrict `astroid` version < 3 (#
jedcunningham merged PR #34658:
URL: https://github.com/apache/airflow/pull/34658
--
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...@ai
github-actions[bot] closed pull request #33192: Add marker for a test to run ti
with test mode
URL: https://github.com/apache/airflow/pull/33192
--
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 s
o-nikolas commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1339330087
##
airflow/api_connexion/security.py:
##
@@ -55,3 +68,166 @@ def decorated(*args, **kwargs):
return cast(T, decorated)
return requires_access_decora
jedcunningham opened a new issue, #34660:
URL: https://github.com/apache/airflow/issues/34660
### Body
We are kindly requesting that contributors to [Apache Airflow Helm Chart
1.11.0rc1](https://dist.apache.org/repos/dist/dev/airflow/helm-chart/1.11.0rc1)
help test the RC.
Ple
This is an automated email from the ASF dual-hosted git repository.
jedcunningham pushed a change to annotated tag helm-chart/1.11.0rc1
in repository https://gitbox.apache.org/repos/asf/airflow.git
*** WARNING: tag helm-chart/1.11.0rc1 was modified! ***
from ee2109ff4a (commit)
to 014
Author: jedcunningham
Date: Wed Sep 27 23:29:31 2023
New Revision: 64231
Log:
Remove old Helm Chart release: 1.10.0rc1
Removed:
dev/airflow/helm-chart/1.10.0rc1/
Author: jedcunningham
Date: Wed Sep 27 23:28:46 2023
New Revision: 64230
Log:
Add artifacts for Helm Chart 1.11.0rc1
Added:
dev/airflow/helm-chart/1.11.0rc1/
dev/airflow/helm-chart/1.11.0rc1/airflow-1.11.0.tgz (with props)
dev/airflow/helm-chart/1.11.0rc1/airflow-1.11.0.tgz.asc
This is an automated email from the ASF dual-hosted git repository.
jedcunningham 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 2abebe3a7b Add helm chart 1.11.0 changelog f
jedcunningham merged PR #34655:
URL: https://github.com/apache/airflow/pull/34655
--
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...@ai
ferruzzi commented on code in PR #34324:
URL: https://github.com/apache/airflow/pull/34324#discussion_r1339321927
##
docs/apache-airflow/core-concepts/executor/index.rst:
##
@@ -78,3 +71,134 @@ There are two types of executor - those that run tasks
*locally* (inside the ``s
..
o-nikolas commented on code in PR #34324:
URL: https://github.com/apache/airflow/pull/34324#discussion_r1339320655
##
docs/apache-airflow/core-concepts/executor/index.rst:
##
@@ -78,3 +71,134 @@ There are two types of executor - those that run tasks
*locally* (inside the ``s
.
Taragolis opened a new pull request, #34658:
URL: https://github.com/apache/airflow/pull/34658
Our `main` branch CI unable to build docs due to an error, see:
https://github.com/readthedocs/sphinx-autoapi/issues/407
```console
apache-airflow-providers-jenkins
dstandish commented on issue #34634:
URL: https://github.com/apache/airflow/issues/34634#issuecomment-1738136970
> Still is a valid feature having a "simple" administrative "halt" of
scheduler. If somebody wants to contribute this is valid...
You write `Having a single "kill switch" m
jens-scheffler-bosch commented on issue #34634:
URL: https://github.com/apache/airflow/issues/34634#issuecomment-1738125495
I feel "turning all off" is might be very simple but if you have a selective
set enabled, then re-enabling still is a tedious job. If I would need to make
this I proba
dstandish opened a new pull request, #34656:
URL: https://github.com/apache/airflow/pull/34656
Introduced in #34120.
--
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
jens-scheffler-bosch commented on issue #34641:
URL: https://github.com/apache/airflow/issues/34641#issuecomment-1738122440
Oh, yeah. You can easily help and this even with no complex dev env setup.
Use the page
https://airflow.apache.org/docs/apache-airflow/stable/core-concepts/params.html
tnyz commented on PR #34654:
URL: https://github.com/apache/airflow/pull/34654#issuecomment-1738115366
>Optional dictionary for arbitrary parameters to the boto API
create_application call.
so I think this statement is not accurate as you cannot put release_version
in this dictionary
dstandish commented on PR #32572:
URL: https://github.com/apache/airflow/pull/32572#issuecomment-1738114465
Hi @vincbeck
Can you help me understand why max_tis_per_query should be no greater than
parallelism?
--
This is an automated message from the Apache Git Service.
To respond
tnyz commented on PR #34654:
URL: https://github.com/apache/airflow/pull/34654#issuecomment-1738107974
> Could you add a bit more details about this best practices? Unfortunately
I can’t find any info, according to the [boto3
doc](https://boto3.amazonaws.com/v1/documentation/api/latest/refe
dstandish commented on PR #34120:
URL: https://github.com/apache/airflow/pull/34120#issuecomment-1738099095
@hussein-awala i see these warnings now:
```
/Users/dstandish/code/airflow/airflow/models/taskinstance.py:3432 SAWarning:
Can't validate argument 'foreign_key'; can't locate any
Taragolis commented on PR #34654:
URL: https://github.com/apache/airflow/pull/34654#issuecomment-1738094904
Could you add a bit more details about this best practices?
Unfortunately I can’t find any info, according to the [boto3
doc](https://boto3.amazonaws.com/v1/documentation/api/latest
jedcunningham commented on PR #34639:
URL: https://github.com/apache/airflow/pull/34639#issuecomment-1738093225
I really like this idea, but I suspect this will have to be done on
[airflow-site](https://github.com/apache/airflow-site/). This repo only has the
latest-and-greatest, and this w
jedcunningham commented on code in PR #34579:
URL: https://github.com/apache/airflow/pull/34579#discussion_r1339224522
##
tests/providers/cncf/kubernetes/triggers/test_pod.py:
##
@@ -334,3 +335,37 @@ def
test_define_container_state_should_execute_successfully(
)
ferruzzi merged PR #34653:
URL: https://github.com/apache/airflow/pull/34653
--
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.
ferruzzi pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 493431023f Release notes for helm chart 1.11.0 (#34618)
add e80077 Clarify Amazon Lambda invocation and sens
hussein-awala commented on issue #34644:
URL: https://github.com/apache/airflow/issues/34644#issuecomment-1738076701
> Expected key current-context in kube-config
Deferrable and no deferrable operators use the exact same method to load the
kube config file since 7.0.0, I'm surprised t
jedcunningham opened a new pull request, #34655:
URL: https://github.com/apache/airflow/pull/34655
🤦♂️ forgot to generate this for 1.11.0
--
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
This is an automated email from the ASF dual-hosted git repository.
jedcunningham 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 493431023f Release notes for helm chart 1.11
jedcunningham merged PR #34618:
URL: https://github.com/apache/airflow/pull/34618
--
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...@ai
Taragolis commented on code in PR #34653:
URL: https://github.com/apache/airflow/pull/34653#discussion_r1339197054
##
docs/apache-airflow-providers-amazon/operators/lambda.rst:
##
@@ -68,10 +100,10 @@ Sensors
.. _howto/sensor:LambdaFunctionStateSensor:
-Wait on an Amazon La
tnyz opened a new pull request, #34654:
URL: https://github.com/apache/airflow/pull/34654
as best practice users should be able to provision their serverless app
using the config api without having to define/duplicate job type and release
version in the airflow abstration
hussein-awala merged PR #34645:
URL: https://github.com/apache/airflow/pull/34645
--
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...@ai
ferruzzi commented on code in PR #34653:
URL: https://github.com/apache/airflow/pull/34653#discussion_r1339187733
##
docs/apache-airflow-providers-amazon/operators/lambda.rst:
##
@@ -57,6 +57,38 @@ Invoke an AWS Lambda function
To invoke an AWS lambda function you can use
:cl
ferruzzi commented on code in PR #34324:
URL: https://github.com/apache/airflow/pull/34324#discussion_r1339180017
##
docs/apache-airflow/core-concepts/executor/index.rst:
##
@@ -78,3 +71,134 @@ There are two types of executor - those that run tasks
*locally* (inside the ``s
..
ferruzzi commented on code in PR #34324:
URL: https://github.com/apache/airflow/pull/34324#discussion_r1339169842
##
docs/apache-airflow/core-concepts/executor/index.rst:
##
@@ -78,3 +71,134 @@ There are two types of executor - those that run tasks
*locally* (inside the ``s
..
vincbeck commented on code in PR #34324:
URL: https://github.com/apache/airflow/pull/34324#discussion_r1339165472
##
docs/apache-airflow/core-concepts/executor/index.rst:
##
@@ -78,3 +71,134 @@ There are two types of executor - those that run tasks
*locally* (inside the ``s
..
JonnyWaffles commented on issue #31544:
URL: https://github.com/apache/airflow/issues/31544#issuecomment-1738010381
Hi team, apologies for the delayed response @potiuk!
After reading through the security model, I don't really understand how
force running a task is non-compliant, but t
pierrejeambrun commented on PR #34568:
URL: https://github.com/apache/airflow/pull/34568#issuecomment-1738005108
Yep, what Hussein said, not a breaking change
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
vincbeck commented on code in PR #34653:
URL: https://github.com/apache/airflow/pull/34653#discussion_r1339135104
##
docs/apache-airflow-providers-amazon/operators/lambda.rst:
##
@@ -57,6 +57,38 @@ Invoke an AWS Lambda function
To invoke an AWS lambda function you can use
:cl
jedcunningham commented on code in PR #34618:
URL: https://github.com/apache/airflow/pull/34618#discussion_r1339130019
##
chart/RELEASE_NOTES.rst:
##
@@ -23,6 +23,111 @@ Run ``helm repo update`` before upgrading the chart to the
latest version.
.. towncrier release notes sta
o-nikolas commented on code in PR #34324:
URL: https://github.com/apache/airflow/pull/34324#discussion_r1339125938
##
docs/apache-airflow/core-concepts/executor/debug.rst:
##
@@ -15,8 +15,47 @@
specific language governing permissions and limitations
under the License.
o-nikolas commented on code in PR #34324:
URL: https://github.com/apache/airflow/pull/34324#discussion_r1339125938
##
docs/apache-airflow/core-concepts/executor/debug.rst:
##
@@ -15,8 +15,47 @@
specific language governing permissions and limitations
under the License.
Taragolis opened a new pull request, #34653:
URL: https://github.com/apache/airflow/pull/34653
Follow up for troubleshooting in Slack
- Add information how to deal with timeout error during sync Amazon Lambda
invocation
- Add information for workaround for sensing a
uranusjr merged PR #34637:
URL: https://github.com/apache/airflow/pull/34637
--
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.
uranusjr 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 e5325cd2b7 Unify mapped task group lookup logic (
hussein-awala commented on code in PR #34652:
URL: https://github.com/apache/airflow/pull/34652#discussion_r1339084563
##
airflow/sensors/external_task.py:
##
@@ -163,9 +163,11 @@ def __init__(
total_states = set(self.allowed_states + self.skipped_states +
self.failed_
hussein-awala commented on code in PR #34568:
URL: https://github.com/apache/airflow/pull/34568#discussion_r1339079159
##
airflow/api_connexion/schemas/task_instance_schema.py:
##
@@ -180,7 +180,7 @@ def validate_form(self, data, **kwargs):
class SetSingleTaskInstanceStateFormS
hussein-awala commented on PR #34568:
URL: https://github.com/apache/airflow/pull/34568#issuecomment-1737929509
> Also, are we considering this change to the default a breaking change to
the API or a bugfix?
>
> CC @eladkal
Based on this issue #34563, we cannot use the endpoint
hussein-awala commented on code in PR #34392:
URL: https://github.com/apache/airflow/pull/34392#discussion_r1339053143
##
airflow/ti_deps/deps/trigger_rule_dep.py:
##
@@ -379,7 +379,7 @@ def _evaluate_direct_relatives() -> Iterator[TIDepStatus]:
if skipped:
TJaniF commented on PR #34392:
URL: https://github.com/apache/airflow/pull/34392#issuecomment-1737849365
Thank you! Hmm... good point! My thinking was that since currently a
`failed` upstream task causes the all_skipped-task to end up in `skipped`, then
`upstream_failed` should do the same
jens-scheffler-bosch commented on PR #34392:
URL: https://github.com/apache/airflow/pull/34392#issuecomment-1737838802
PR from code and attempt looks good. But I would assume if upstream failed
is the status of previous tasks, then the task under this fix should also
rather be going into "u
utkarsharma2 opened a new pull request, #34652:
URL: https://github.com/apache/airflow/pull/34652
This PR intends to solve the issue that the soft_fail argument in
ExternalTaskSensor is not respected.
Also closes: https://github.com/apache/airflow/issues/34497
--
This is an aut
Taragolis closed issue #34646: Connections and variables are not being created
based on extraSecrets and extraConfigMaps
URL: https://github.com/apache/airflow/issues/34646
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
vincbeck commented on code in PR #34349:
URL: https://github.com/apache/airflow/pull/34349#discussion_r1338967863
##
airflow/auth/managers/base_auth_manager.py:
##
@@ -50,6 +50,10 @@ def get_cli_commands() -> list[CLICommand]:
"""
return []
+def get_api_b
Calder-Ty commented on PR #34568:
URL: https://github.com/apache/airflow/pull/34568#issuecomment-1737802596
> Also, are we considering this change to the default a breaking change to
the API or a bugfix?
>
> CC @eladkal
Technically it is, however, I think we can be fairly sure
o-nikolas commented on PR #34568:
URL: https://github.com/apache/airflow/pull/34568#issuecomment-1737793184
Also, are we considering this change to the default a breaking change to the
API or a bugfix?
CC @eladkal
--
This is an automated message from the Apache Git Service.
To res
jedcunningham commented on code in PR #34349:
URL: https://github.com/apache/airflow/pull/34349#discussion_r1338936861
##
airflow/auth/managers/base_auth_manager.py:
##
@@ -50,6 +50,10 @@ def get_cli_commands() -> list[CLICommand]:
"""
return []
+def get_
Taragolis commented on code in PR #34627:
URL: https://github.com/apache/airflow/pull/34627#discussion_r1338907495
##
airflow/providers/microsoft/azure/operators/container_instances.py:
##
@@ -319,6 +319,9 @@ def _monitor_logging(self, resource_group: str, name: str)
-> int:
Taragolis commented on code in PR #34627:
URL: https://github.com/apache/airflow/pull/34627#discussion_r1338907495
##
airflow/providers/microsoft/azure/operators/container_instances.py:
##
@@ -319,6 +319,9 @@ def _monitor_logging(self, resource_group: str, name: str)
-> int:
jedcunningham commented on PR #34647:
URL: https://github.com/apache/airflow/pull/34647#issuecomment-1737728724
@utkarsharma2, it lives here now:
https://github.com/apache/airflow/blob/fb92ff8486f21b61a840ddc4414429c3a9adfc88/airflow/auth/managers/fab/cli_commands/user_command.py#L153
--
jedcunningham closed issue #34482: KubernetesPodOperator shutting down istio
sidecar but not deleting pod upon failure
URL: https://github.com/apache/airflow/issues/34482
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
This is an automated email from the ASF dual-hosted git repository.
jedcunningham 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 fb92ff8486 Fix Pod not being removed after i
jedcunningham merged PR #34500:
URL: https://github.com/apache/airflow/pull/34500
--
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...@ai
This is an automated email from the ASF dual-hosted git repository.
dstandish 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 e81bb48779 Fail dag test if defer without trigge
dstandish merged PR #34619:
URL: https://github.com/apache/airflow/pull/34619
--
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
krisfur commented on code in PR #34627:
URL: https://github.com/apache/airflow/pull/34627#discussion_r1338860691
##
airflow/providers/microsoft/azure/operators/container_instances.py:
##
@@ -319,6 +319,9 @@ def _monitor_logging(self, resource_group: str, name: str)
-> int:
krisfur commented on code in PR #34627:
URL: https://github.com/apache/airflow/pull/34627#discussion_r1338860691
##
airflow/providers/microsoft/azure/operators/container_instances.py:
##
@@ -319,6 +319,9 @@ def _monitor_logging(self, resource_group: str, name: str)
-> int:
SamWheating commented on PR #34172:
URL: https://github.com/apache/airflow/pull/34172#issuecomment-1737699234
I actually can't trigger the workflow - we'll have to wait for a committer
to come along and do the final check + merge.
--
This is an automated message from the Apache Git Servic
Taragolis commented on code in PR #34627:
URL: https://github.com/apache/airflow/pull/34627#discussion_r1338841760
##
airflow/providers/microsoft/azure/operators/container_instances.py:
##
@@ -319,6 +319,9 @@ def _monitor_logging(self, resource_group: str, name: str)
-> int:
krisfur commented on code in PR #34627:
URL: https://github.com/apache/airflow/pull/34627#discussion_r1338839397
##
airflow/providers/microsoft/azure/operators/container_instances.py:
##
@@ -319,6 +319,9 @@ def _monitor_logging(self, resource_group: str, name: str)
-> int:
krisfur commented on code in PR #34627:
URL: https://github.com/apache/airflow/pull/34627#discussion_r1338805545
##
airflow/providers/microsoft/azure/operators/container_instances.py:
##
@@ -319,6 +319,9 @@ def _monitor_logging(self, resource_group: str, name: str)
-> int:
Taragolis commented on code in PR #34627:
URL: https://github.com/apache/airflow/pull/34627#discussion_r1338800572
##
airflow/providers/microsoft/azure/operators/container_instances.py:
##
@@ -319,6 +319,9 @@ def _monitor_logging(self, resource_group: str, name: str)
-> int:
Taragolis commented on code in PR #34627:
URL: https://github.com/apache/airflow/pull/34627#discussion_r1338797309
##
airflow/providers/microsoft/azure/operators/container_instances.py:
##
@@ -319,6 +319,9 @@ def _monitor_logging(self, resource_group: str, name: str)
-> int:
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 07fe1d2a69 Refactor dedent nested loops (#34409)
vincbeck merged PR #34409:
URL: https://github.com/apache/airflow/pull/34409
--
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
vincbeck commented on PR #34466:
URL: https://github.com/apache/airflow/pull/34466#issuecomment-1737631885
There are still some static checks failures. You should be able to fix them
automatically by looking at
[documentation](https://github.com/apache/airflow/blob/main/STATIC_CODE_CHECKS.r
belaytzev commented on issue #34646:
URL: https://github.com/apache/airflow/issues/34646#issuecomment-1737588388
> It not how it initially designed, airflow just lookup in configured
storage during the execution, see [Secrets
Backend](https://airflow.apache.org/docs/apache-airflow/stable/se
vincbeck commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1338731975
##
airflow/api_connexion/endpoints/dag_endpoint.py:
##
@@ -180,7 +180,7 @@ def patch_dags(limit, session, offset=0, only_active=True,
tags=None, dag_id_pat
ret
vincbeck commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1338734563
##
airflow/api_connexion/endpoints/xcom_endpoint.py:
##
@@ -39,14 +39,7 @@
from airflow.api_connexion.types import APIResponse
-@security.requires_access(
-
vincbeck commented on code in PR #34317:
URL: https://github.com/apache/airflow/pull/34317#discussion_r1338731975
##
airflow/api_connexion/endpoints/dag_endpoint.py:
##
@@ -180,7 +180,7 @@ def patch_dags(limit, session, offset=0, only_active=True,
tags=None, dag_id_pat
ret
Taragolis commented on issue #34646:
URL: https://github.com/apache/airflow/issues/34646#issuecomment-1737541634
It not how it initially designed, airflow just lookup in configured storage
during the execution, see [Secrets
Backend](https://airflow.apache.org/docs/apache-airflow/stable/secu
vincbeck commented on code in PR #34349:
URL: https://github.com/apache/airflow/pull/34349#discussion_r1338719267
##
airflow/auth/managers/base_auth_manager.py:
##
@@ -50,6 +50,10 @@ def get_cli_commands() -> list[CLICommand]:
"""
return []
+def get_api_b
phanikumv commented on code in PR #34643:
URL: https://github.com/apache/airflow/pull/34643#discussion_r1338695414
##
tests/providers/databricks/hooks/test_databricks.py:
##
@@ -158,6 +162,11 @@ def repair_run_endpoint(host):
"""
return f"https://{host}/api/2.1/jobs/ru
fuatcakici commented on issue #34604:
URL: https://github.com/apache/airflow/issues/34604#issuecomment-1737453705
I have potentially also found another error which you may want to know.
After manually setting the `simple_task` in the first DAG run to FAILED, if I
manually set that whole DAG
mnacharov commented on code in PR #34342:
URL: https://github.com/apache/airflow/pull/34342#discussion_r1338645269
##
airflow/providers/cncf/kubernetes/pod_template_file_examples/git_sync_v4_template.yaml:
##
Review Comment:
ok
--
This is an automated message from the A
mnacharov commented on code in PR #34342:
URL: https://github.com/apache/airflow/pull/34342#discussion_r1338644564
##
chart/templates/_helpers.yaml:
##
@@ -197,6 +197,7 @@ If release name contains chart name it will be used as a
full name.
imagePullPolicy: {{ .Values.images.
1 - 100 of 150 matches
Mail list logo