Re: [PR] Bumping min version of azure-storage-blob to 12.26.0 [airflow]

2025-07-16 Thread via GitHub
gopidesupavan commented on code in PR #53440: URL: https://github.com/apache/airflow/pull/53440#discussion_r2212396067 ## providers/celery/src/airflow/providers/celery/executors/celery_executor.py: ## @@ -474,6 +474,8 @@ def try_adopt_task_instances(self, tis: Sequence[TaskInst

[I] Add side/navbar_color config for Airflow 3.x [airflow]

2025-07-16 Thread via GitHub
r-richmond opened a new issue, #53443: URL: https://github.com/apache/airflow/issues/53443 ### Description In Airflow 2.x `NAVBAR_COLOR` controlled the color of the navbar on every UI page. This made it very useful for clearly marking different airflow environments (prod, staging, de

Re: [PR] Bumping min version of azure-storage-blob to 12.26.0 [airflow]

2025-07-16 Thread via GitHub
amoghrajesh commented on PR #53440: URL: https://github.com/apache/airflow/pull/53440#issuecomment-3082718629 Ah wrong import, checked their repo and adding the right one. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

(airflow) branch main updated (d8e6f4bb756 -> dc63c2560ee)

2025-07-16 Thread gopidesu
This is an automated email from the ASF dual-hosted git repository. gopidesu pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from d8e6f4bb756 feat(airflowctl): Include Airflow CLI Output Structure and Integrate (#53424) add dc63c2560ee Add t

Re: [PR] Add type ignore unreachable for get_autocommit in jdbc provider [airflow]

2025-07-16 Thread via GitHub
gopidesupavan merged PR #53434: URL: https://github.com/apache/airflow/pull/53434 -- 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

Re: [PR] [v3-0-test] Add --warn-unreachable flag to mypy as a optional flag (#53398) [airflow]

2025-07-16 Thread via GitHub
gopidesupavan closed pull request #53428: [v3-0-test] Add --warn-unreachable flag to mypy as a optional flag (#53398) URL: https://github.com/apache/airflow/pull/53428 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

Re: [I] AwsBaseWaiterTrigger blocks event loop on initialization [airflow]

2025-07-16 Thread via GitHub
kien-truong commented on issue #53359: URL: https://github.com/apache/airflow/issues/53359#issuecomment-3082691370 `boto3` service loader can also introduce long blocking upon service initialization: https://github.com/aio-libs/aiobotocore/issues/1199 -- This is an automated messag

Re: [D] Airflow 3.0.2 configure_logging from airflow.sdk.log ignores logging_config_class [airflow]

2025-07-16 Thread via GitHub
GitHub user r-richmond added a comment to the discussion: Airflow 3.0.2 configure_logging from airflow.sdk.log ignores logging_config_class Based on Ash's comments I submitted https://github.com/apache/airflow/issues/53442 to make it easier for others to find the current information about lo

Re: [D] Airflow 3.0.2 configure_logging from airflow.sdk.log ignores logging_config_class [airflow]

2025-07-16 Thread via GitHub
GitHub user r-richmond deleted a comment on the discussion: Airflow 3.0.2 configure_logging from airflow.sdk.log ignores logging_config_class Based on Ash's comments I submitted https://github.com/apache/airflow/issues/53442 to make it easier for others to find the current information about

Re: [D] Airflow 3.0.2 configure_logging from airflow.sdk.log ignores logging_config_class [airflow]

2025-07-16 Thread via GitHub
GitHub user r-richmond added a comment to the discussion: Airflow 3.0.2 configure_logging from airflow.sdk.log ignores logging_config_class Based on Ash's comments I submitted https://github.com/apache/airflow/issues/53442 to make it easier for others to find the current information about lo

[I] Airflow Logging Configs are not Used [airflow]

2025-07-16 Thread via GitHub
r-richmond opened a new issue, #53442: URL: https://github.com/apache/airflow/issues/53442 ### Apache Airflow version 3.0.3 ### If "Other Airflow 2 version" selected, which one? _No response_ ### What happened? Airflow log configs are currently not used.

Re: [I] Airflow 3 Connections Page HTTP 500 [airflow]

2025-07-16 Thread via GitHub
r-richmond commented on issue #53091: URL: https://github.com/apache/airflow/issues/53091#issuecomment-3082651365 I had a similar issue when using helm chart deploy. Explicitly setting the fernet key & then wiping & re-creating the connections & variables resolved it for me as well. -- T

Re: [PR] Bumping min version of azure-storage-blob to 12.26.0 [airflow]

2025-07-16 Thread via GitHub
amoghrajesh commented on code in PR #53440: URL: https://github.com/apache/airflow/pull/53440#discussion_r2212312884 ## providers/microsoft/azure/src/airflow/providers/microsoft/azure/hooks/wasb.py: ## @@ -712,7 +713,7 @@ async def get_blobs_list_async( include: list[st

Re: [PR] Bumping min version of azure-storage-blob to 12.26.0 [airflow]

2025-07-16 Thread via GitHub
eladkal commented on code in PR #53440: URL: https://github.com/apache/airflow/pull/53440#discussion_r2212290701 ## providers/microsoft/azure/src/airflow/providers/microsoft/azure/hooks/wasb.py: ## @@ -712,7 +713,7 @@ async def get_blobs_list_async( include: list[str] |

Re: [PR] Bumping min version of azure-storage-blob to 12.26.0 [airflow]

2025-07-16 Thread via GitHub
eladkal commented on code in PR #53440: URL: https://github.com/apache/airflow/pull/53440#discussion_r2212290701 ## providers/microsoft/azure/src/airflow/providers/microsoft/azure/hooks/wasb.py: ## @@ -712,7 +713,7 @@ async def get_blobs_list_async( include: list[str] |

Re: [PR] Bumping min version of azure-storage-blob to 12.26.0 [airflow]

2025-07-16 Thread via GitHub
eladkal commented on code in PR #53440: URL: https://github.com/apache/airflow/pull/53440#discussion_r2212290701 ## providers/microsoft/azure/src/airflow/providers/microsoft/azure/hooks/wasb.py: ## @@ -712,7 +713,7 @@ async def get_blobs_list_async( include: list[str] |

Re: [PR] Add source dag run to asset [airflow]

2025-07-16 Thread via GitHub
kgw7401 commented on code in PR #53357: URL: https://github.com/apache/airflow/pull/53357#discussion_r2212291443 ## task-sdk/src/airflow/sdk/execution_time/comms.py: ## @@ -390,6 +406,48 @@ def source_task_instance(self) -> AssetEventSourceTaskInstance | None: map_

Re: [PR] Fix: simplify parsing of Apprise config in AppriseHook [airflow]

2025-07-16 Thread via GitHub
mrohith3693 commented on PR #53441: URL: https://github.com/apache/airflow/pull/53441#issuecomment-3082523559 Hi 👋 This PR fixes a TypeError in AppriseHook.get_config_from_conn() caused when the Apprise connection's config is already a dictionary (e.g. from env vars like AIRFLOW_CONN_APPRIS

Re: [PR] adding docs for wait till dag run finishes [airflow]

2025-07-16 Thread via GitHub
Bowrna commented on code in PR #53346: URL: https://github.com/apache/airflow/pull/53346#discussion_r2212261253 ## airflow-core/docs/core-concepts/dag-run.rst: ## @@ -260,6 +260,28 @@ Example of a parameterized DAG: **Note**: The parameters from ``dag_run.conf`` can only be u

Re: [I] Fix/Cleanup MyPy unreachable code warnings Across Providers [airflow]

2025-07-16 Thread via GitHub
jscheffl commented on issue #53395: URL: https://github.com/apache/airflow/issues/53395#issuecomment-3082469011 UPDATE on update: `breeze down --cleanup-mypy-cache` helps. Now list is showing back again for sftp/snowflake :-D -- This is an automated message from the Apache Git Service. To

Re: [PR] Fix: simplify parsing of Apprise config in AppriseHook [airflow]

2025-07-16 Thread via GitHub
boring-cyborg[bot] commented on PR #53441: URL: https://github.com/apache/airflow/pull/53441#issuecomment-3082463919 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'

[PR] Fix: simplify parsing of Apprise config in AppriseHook [airflow]

2025-07-16 Thread via GitHub
mrohith3693 opened a new pull request, #53441: URL: https://github.com/apache/airflow/pull/53441 ### Simplify Parsing of apprise_default Connection ID in AppriseHook This PR improves the robustness of AppriseHook.get_config_from_conn() to correctly handle cases where the config fiel

[PR] Bumping min version of azure-storage-blob to 12.26.0 [airflow]

2025-07-16 Thread via GitHub
amoghrajesh opened a new pull request, #53440: URL: https://github.com/apache/airflow/pull/53440 Using this opportunity to bump of azure-blob-storage library to 12.26 released yesterday. We have been using 3+ year old one https://github.com/user-attachments/assets/3

Re: [PR] Add dag delete button in dag details page [airflow]

2025-07-16 Thread via GitHub
Jasperora commented on code in PR #53064: URL: https://github.com/apache/airflow/pull/53064#discussion_r2212073733 ## airflow-core/src/airflow/ui/src/components/DagActions/DeleteDagButton.tsx: ## @@ -51,7 +57,7 @@ const DeleteDagButton = ({ dagDisplayName, dagId, withText = tru

Re: [PR] Add dag delete button in dag details page [airflow]

2025-07-16 Thread via GitHub
Jasperora commented on code in PR #53064: URL: https://github.com/apache/airflow/pull/53064#discussion_r2212073733 ## airflow-core/src/airflow/ui/src/components/DagActions/DeleteDagButton.tsx: ## @@ -51,7 +57,7 @@ const DeleteDagButton = ({ dagDisplayName, dagId, withText = tru

Re: [PR] Add dag delete button in dag details page [airflow]

2025-07-16 Thread via GitHub
Jasperora commented on code in PR #53064: URL: https://github.com/apache/airflow/pull/53064#discussion_r2212073733 ## airflow-core/src/airflow/ui/src/components/DagActions/DeleteDagButton.tsx: ## @@ -51,7 +57,7 @@ const DeleteDagButton = ({ dagDisplayName, dagId, withText = tru

Re: [PR] Add dag delete button in dag details page [airflow]

2025-07-16 Thread via GitHub
Jasperora commented on code in PR #53064: URL: https://github.com/apache/airflow/pull/53064#discussion_r2212072807 ## airflow-core/src/airflow/ui/src/layouts/Details/DetailsLayout.tsx: ## @@ -77,6 +78,9 @@ export const DetailsLayout = ({ children, error, isLoading, tabs }: Prop

Re: [I] Secret Manager to Fetch secrets from a chain of cross account. [airflow]

2025-07-16 Thread via GitHub
Poornima030596 commented on issue #53361: URL: https://github.com/apache/airflow/issues/53361#issuecomment-3082268620 Airflow can fetch secrets only from one AWS account - > providers/amazon/src/airflow/providers/amazon/aws/secrets/secrets_manager.py -- This is an automated message fr

Re: [I] Secret Manager to Fetch secrets from a chain of cross account. [airflow]

2025-07-16 Thread via GitHub
Poornima030596 commented on issue #53361: URL: https://github.com/apache/airflow/issues/53361#issuecomment-3082267034 This is not an issue with Astronomer as Astronomer just provides airflow as a service and uses the base code of airflow. It uses this providers/amazon/src/airflow/provide

Re: [I] 3.0.3 xcom_pull inconsistent content returned when using file storage [airflow]

2025-07-16 Thread via GitHub
puzzle9 commented on issue #53432: URL: https://github.com/apache/airflow/issues/53432#issuecomment-3082242730 > Are you using a custom XCom backend? If so, which one? ```env xcom_backend = airflow.providers.common.io.xcom.backend.XComObjectStorageBackend xcom_objectstorage_path

Re: [PR] Refactoring connection in httphook [airflow]

2025-07-16 Thread via GitHub
kyungjunleeme commented on PR #53407: URL: https://github.com/apache/airflow/pull/53407#issuecomment-3082235695 @jscheffl Since you recently shared an idea that inspired this refactoring direction, I wanted to mention you here. If you have a moment, I’d really appreciate hearing y

Re: [PR] Refactoring connection in httphook [airflow]

2025-07-16 Thread via GitHub
kyungjunleeme commented on code in PR #53407: URL: https://github.com/apache/airflow/pull/53407#discussion_r2212022460 ## providers/http/src/airflow/providers/http/hooks/http.py: ## @@ -172,7 +175,7 @@ def get_conn( :return: A configured requests.Session object.

Re: [PR] Refactoring connection in httphook [airflow]

2025-07-16 Thread via GitHub
kyungjunleeme commented on code in PR #53407: URL: https://github.com/apache/airflow/pull/53407#discussion_r2212006473 ## providers/http/src/airflow/providers/http/hooks/http.py: ## @@ -172,7 +175,7 @@ def get_conn( :return: A configured requests.Session object.

[I] Unable to show both enabled and disabled dags at the same time (the `All` option is unresponsive) [airflow]

2025-07-16 Thread via GitHub
zachliu opened a new issue, #53439: URL: https://github.com/apache/airflow/issues/53439 ### Apache Airflow version 3.0.3 ### If "Other Airflow 2 version" selected, which one? _No response_ ### What happened? The `All, Enabled, Disabled` drop-down menu on the

Re: [PR] Refactoring connection in httphook [airflow]

2025-07-16 Thread via GitHub
jroachgolf84 commented on code in PR #53407: URL: https://github.com/apache/airflow/pull/53407#discussion_r2211956334 ## providers/http/src/airflow/providers/http/hooks/http.py: ## @@ -172,7 +175,7 @@ def get_conn( :return: A configured requests.Session object.

Re: [I] 404 Not Found for `EmptyOperator` docs for version 2.7+ [airflow]

2025-07-16 Thread via GitHub
jroachgolf84 commented on issue #52643: URL: https://github.com/apache/airflow/issues/52643#issuecomment-3082074915 Bumping this back up - what is the best way to go about remedy'ing this? I'm not sure how `redirects.txt` is used... -- This is an automated message from the Apache Git Serv

Re: [PR] feat(airflowctl): Include Airflow CLI Output Structure and Integrate [airflow]

2025-07-16 Thread via GitHub
bugraoz93 merged PR #53424: URL: https://github.com/apache/airflow/pull/53424 -- 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

(airflow) branch main updated: feat(airflowctl): Include Airflow CLI Output Structure and Integrate (#53424)

2025-07-16 Thread bugraoz
This is an automated email from the ASF dual-hosted git repository. bugraoz 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 d8e6f4bb756 feat(airflowctl): Include Airflow CLI

Re: [I] Prettify Generated Command Outputs [airflow]

2025-07-16 Thread via GitHub
bugraoz93 closed issue #52791: Prettify Generated Command Outputs URL: https://github.com/apache/airflow/issues/52791 -- 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 unsubsc

Re: [I] Provide access privilege required for `cncf.kubernetes` operators/executors [airflow]

2025-07-16 Thread via GitHub
yranjan06 commented on issue #40210: URL: https://github.com/apache/airflow/issues/40210#issuecomment-3082049031 Hi @potiuk @xelathan , I’m a beginner in data engineering and Kubernetes and would love to take this up as my first contribution. @xelathan – just checking if you’ve al

Re: [I] 3.0.3 xcom_pull inconsistent content returned when using file storage [airflow]

2025-07-16 Thread via GitHub
jroachgolf84 commented on issue #53432: URL: https://github.com/apache/airflow/issues/53432#issuecomment-3082014927 Are you using a custom XCom backend? If so, which one? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [I] Fix/Cleanup MyPy unreachable code warnings Across Providers [airflow]

2025-07-16 Thread via GitHub
jroachgolf84 commented on issue #53395: URL: https://github.com/apache/airflow/issues/53395#issuecomment-3082012031 Is there a flag/argument that we can include in this command (`./scripts/ci/pre_commit/mypy_folder.py providers/snowflake`) that allows us to run "all files without cache"?

Re: [I] Missing Metrics After Migrating from StatsD to OpenTelemetry [airflow]

2025-07-16 Thread via GitHub
zettelin commented on issue #41822: URL: https://github.com/apache/airflow/issues/41822#issuecomment-3081932096 I made a personal workaround so that people who already migrated to Otel could use this until it is officially fixed. ### Cause of issue (As I understand) Flushing fr

Re: [PR] Serialize the modified DAG to DB immediately without wating period [airflow]

2025-07-16 Thread via GitHub
github-actions[bot] commented on PR #48474: URL: https://github.com/apache/airflow/pull/48474#issuecomment-3081919083 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

Re: [PR] Helm Chart: Add gitConfig option for gitSync and upgrade NOTES.txt for Airflow v3 [airflow]

2025-07-16 Thread via GitHub
github-actions[bot] closed pull request #50418: Helm Chart: Add gitConfig option for gitSync and upgrade NOTES.txt for Airflow v3 URL: https://github.com/apache/airflow/pull/50418 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

Re: [I] Task groups relationships falsely decleared as circular in airflow3.0.2 [airflow]

2025-07-16 Thread via GitHub
github-actions[bot] closed issue #52245: Task groups relationships falsely decleared as circular in airflow3.0.2 URL: https://github.com/apache/airflow/issues/52245 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [I] Task groups relationships falsely decleared as circular in airflow3.0.2 [airflow]

2025-07-16 Thread via GitHub
github-actions[bot] commented on issue #52245: URL: https://github.com/apache/airflow/issues/52245#issuecomment-3081918567 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

Re: [I] LatestOnlyOperator fails in airflow-providers-standard >=1.1.0 [airflow]

2025-07-16 Thread via GitHub
leakec commented on issue #50911: URL: https://github.com/apache/airflow/issues/50911#issuecomment-3081909775 For anyone looking to reproduce the problem, a simple way to do so is run the following DAG in an airflow instance: ```python from airflow import DAG from datetime import da

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on PR #46891: URL: https://github.com/apache/airflow/pull/46891#issuecomment-3081874457 Pushed changes trying out the comments in places I am not sure. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211857618 ## Dockerfile.ci: ## @@ -1185,7 +1190,9 @@ function check_force_lowest_dependencies() { # --no-binary is needed in order to avoid libxml and xmlsec using di

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211854419 ## airflow-core/tests/unit/serialization/test_dag_serialization.py: ## @@ -1229,10 +1234,6 @@ def test_extra_serialized_field_and_operator_links( link = s

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211852077 ## airflow-core/tests/unit/cli/commands/test_api_server_command.py: ## @@ -296,16 +297,31 @@ def test_run_command_daemon( ) ]

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211851312 ## airflow-core/tests/unit/cli/commands/test_api_server_command.py: ## @@ -296,16 +297,31 @@ def test_run_command_daemon( ) ]

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211849777 ## airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_dag_run.py: ## @@ -82,6 +88,7 @@ @pytest.fixture(autouse=True) @provide_session def setup(request

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211848475 ## airflow-core/src/airflow/api_fastapi/core_api/app.py: ## @@ -124,6 +127,13 @@ def init_flask_plugins(app: FastAPI) -> None: try: from airflow.provider

[PR] Restore tooltips on grid view [airflow]

2025-07-16 Thread via GitHub
bbovenzi opened a new pull request, #53438: URL: https://github.com/apache/airflow/pull/53438 Chakra's tooltips are now more performant than when we first built the 3.0 Grid view, but there were still a few issues with it when it came specifically to the grid view. Primarily with how we han

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211842627 ## airflow-core/pyproject.toml: ## @@ -35,7 +35,12 @@ name = "apache-airflow-core" description = "Core packages for Apache Airflow, schedule and API server" readme =

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211841088 ## providers/apache/beam/pyproject.toml: ## @@ -57,20 +57,21 @@ requires-python = ">=3.10" # After you modify the dependencies, and rebuild your Breeze CI image with

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211842165 ## scripts/ci/pre_commit/update_airflow_pyproject_toml.py: ## @@ -132,6 +141,23 @@ def find_min_provider_version(provider_id: str) -> tuple[Version | None, str]: P

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211838550 ## providers/amazon/pyproject.toml: ## @@ -95,9 +96,12 @@ dependencies = [ "s3fs>=2023.10.0", ] "python3-saml" = [ -"python3-saml>=1.16.0", -"xmlsec>=1.

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211833529 ## dev/breeze/tests/test_selective_checks.py: ## @@ -1236,7 +1236,15 @@ def test_excluded_providers(): { "excluded-providers-as-string": json.dum

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211831396 ## .pre-commit-config.yaml: ## @@ -1014,7 +1014,12 @@ repos: name: Update Airflow's meta-package pyproject.toml language: python entry: ./sc

Re: [I] [Filters] Add filtering support to Search - Dags [airflow]

2025-07-16 Thread via GitHub
AryanK1511 commented on issue #53041: URL: https://github.com/apache/airflow/issues/53041#issuecomment-3081792996 I would love to work on this issue. -- 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

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211828429 ## airflow-core/tests/unit/api_fastapi/conftest.py: ## @@ -126,6 +126,7 @@ def configure_git_connection_for_dag_bundle(session): login="", ) session

Re: [I] AIP-68 Share url params with plugin [airflow]

2025-07-16 Thread via GitHub
AryanK1511 commented on issue #53381: URL: https://github.com/apache/airflow/issues/53381#issuecomment-3081777158 > Sorry, this one may be quite complicated but we need a lot of help updating all our table views if you want to pick one of those up! [#53040](https://github.com/apache/airflow

Re: [PR] AIP-86 - Implement basic functionality for deadline alerts async callbacks [airflow]

2025-07-16 Thread via GitHub
ramitkataria commented on code in PR #53201: URL: https://github.com/apache/airflow/pull/53201#discussion_r2211823482 ## airflow-core/src/airflow/models/trigger.py: ## @@ -366,9 +386,9 @@ def get_sorted_triggers(cls, capacity: int, alive_triggerer_ids: list[int] | Sel

Re: [I] Listeners not firing on task success [airflow]

2025-07-16 Thread via GitHub
tanujdargan commented on issue #53162: URL: https://github.com/apache/airflow/issues/53162#issuecomment-3081751858 My apologies I was testing on the dev branch but I seem to be able to replicate the issue on the current dev version as well. I'll get back to you on this shortly -- This is

Re: [PR] Add Python 3.13 support for Airflow. [airflow]

2025-07-16 Thread via GitHub
potiuk commented on code in PR #46891: URL: https://github.com/apache/airflow/pull/46891#discussion_r2211815756 ## airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_plugins.py: ## @@ -24,6 +24,13 @@ pytestmark = pytest.mark.db_test +try: +import flask_appb

(airflow) branch main updated: Don't hit API for "all group ids" (#53116)

2025-07-16 Thread bbovenzi
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 0138a31bc91 Don't hit API for "all group ids" (#5

Re: [PR] Don't hit API for "all group ids" [airflow]

2025-07-16 Thread via GitHub
bbovenzi merged PR #53116: URL: https://github.com/apache/airflow/pull/53116 -- 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

Re: [I] AIP-68 Share url params with plugin [airflow]

2025-07-16 Thread via GitHub
bbovenzi commented on issue #53381: URL: https://github.com/apache/airflow/issues/53381#issuecomment-3081740654 Sorry, this one may be quite complicated but we need a lot of help updating all our table views if you want to pick one of those up! https://github.com/apache/airflow/issues/53040

Re: [I] LatestOnlyOperator fails in airflow-providers-standard >=1.1.0 [airflow]

2025-07-16 Thread via GitHub
potiuk commented on issue #50911: URL: https://github.com/apache/airflow/issues/50911#issuecomment-3081731251 I guess then that might be somethig @uranusjr might be best to answer. If that clearly reproducible in latest version of airflow - I marked it as "affected_version:3.0" -- This

Re: [D] Adding support for the new Apache Spark Kubernetes Operator [airflow]

2025-07-16 Thread via GitHub
GitHub user potiuk added a comment to the discussion: Adding support for the new Apache Spark Kubernetes Operator Sure. If someone wants to implement it - it's a good idea to contribute it. Feel free to do it or find someone who can do it if you want it. GitHub link: https://github.com/apach

Re: [D] Airflow 3.0: Future of Traditional DAG Context + Operators Approach vs TaskFlow API [airflow]

2025-07-16 Thread via GitHub
GitHub user potiuk edited a comment on the discussion: Airflow 3.0: Future of Traditional DAG Context + Operators Approach vs TaskFlow API Unless it's stated as deprecated, there are no plans to remove feature from Airflow. This might change in the future though - Airflow is a living thing and

Re: [D] Airflow 3.0: Future of Traditional DAG Context + Operators Approach vs TaskFlow API [airflow]

2025-07-16 Thread via GitHub
GitHub user potiuk edited a comment on the discussion: Airflow 3.0: Future of Traditional DAG Context + Operators Approach vs TaskFlow API Unless it's stated as deprecated, there are no plans to remove feature from Airflow. This might change in the future though - Airflow is a living thing and

Re: [D] Airflow 3.0: Future of Traditional DAG Context + Operators Approach vs TaskFlow API [airflow]

2025-07-16 Thread via GitHub
GitHub user potiuk added a comment to the discussion: Airflow 3.0: Future of Traditional DAG Context + Operators Approach vs TaskFlow API Unless it's stated as deprecated, there are no plans to remove feature from Airflow. This might change in the future though - Airflow is a living thing and

Re: [I] Fix/Cleanup MyPy unreachable code warnings Across Providers [airflow]

2025-07-16 Thread via GitHub
gopidesupavan commented on issue #53395: URL: https://github.com/apache/airflow/issues/53395#issuecomment-3081654848 i hopping for no new bugs get introduced while fixing these :) and i feel here its not required to fix all those ones mentioned few may be false positives those can ignored a

Re: [I] Fix/Cleanup MyPy unreachable code warnings Across Providers [airflow]

2025-07-16 Thread via GitHub
gopidesupavan commented on issue #53395: URL: https://github.com/apache/airflow/issues/53395#issuecomment-3081635611 yeah mypy behaving slightly different always, agree running with cleancache with full providers atleast behaves like CI. may i should update that in the steps above? -- T

Re: [D] High CPU and database load caused by Airflow 3 dag-processor and scheduler [airflow]

2025-07-16 Thread via GitHub
GitHub user michaelosthege added a comment to the discussion: High CPU and database load caused by Airflow 3 dag-processor and scheduler Turns out the dag-processor loop is fine, but the healthcheck was the issue! My `dag-processor` healthcheck (copied from the docker-compose example) was: ```

Re: [I] Fix/Cleanup MyPy unreachable code warnings Across Providers [airflow]

2025-07-16 Thread via GitHub
potiuk commented on issue #53395: URL: https://github.com/apache/airflow/issues/53395#issuecomment-3081388499 But maybe others can come up with better ideas :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

Re: [I] Fix/Cleanup MyPy unreachable code warnings Across Providers [airflow]

2025-07-16 Thread via GitHub
potiuk commented on issue #53395: URL: https://github.com/apache/airflow/issues/53395#issuecomment-3081366009 Unfortunately - mypy is known to use heuristics to determine "guessed" types of variables and yes - it is know it will produce different results when different sets of files are pre

[PR] Fix unreachable code errors in Vault provider [airflow]

2025-07-16 Thread via GitHub
dominikhei opened a new pull request, #53437: URL: https://github.com/apache/airflow/pull/53437 --- related: https://github.com/apache/airflow/issues/53395 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [D] High CPU and database load caused by Airflow 3 dag-processor and scheduler [airflow]

2025-07-16 Thread via GitHub
GitHub user michaelosthege edited a comment on the discussion: High CPU and database load caused by Airflow 3 dag-processor and scheduler ~According to the documentation, [`BaseSelector.select` is supposed to wait until some registered file objects become ready, **or the timeout expires**](ht

(airflow) branch main updated: Fix unreachable code mypy warnings in edge3 provider (#53430)

2025-07-16 Thread jscheffl
This is an automated email from the ASF dual-hosted git repository. jscheffl 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 81519c62854 Fix unreachable code mypy warnings in

Re: [PR] Fix unreachable code mypy warnings in edge3 provider [airflow]

2025-07-16 Thread via GitHub
jscheffl merged PR #53430: URL: https://github.com/apache/airflow/pull/53430 -- 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

Re: [I] ValidationError: dag_versions should be a valid dictionary or object [airflow]

2025-07-16 Thread via GitHub
AryanK1511 commented on issue #52660: URL: https://github.com/apache/airflow/issues/52660#issuecomment-3081163904 Hey @dabeeru, I will look into this -- 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

Re: [PR] Don't hit API for "all group ids" [airflow]

2025-07-16 Thread via GitHub
dstandish commented on code in PR #53116: URL: https://github.com/apache/airflow/pull/53116#discussion_r2211671137 ## airflow-core/src/airflow/ui/src/layouts/Details/Graph/Graph.tsx: ## @@ -71,18 +73,32 @@ export const Graph = () => { "gray.800", ]); - const { openGro

Re: [I] Fix/Cleanup MyPy unreachable code warnings Across Providers [airflow]

2025-07-16 Thread via GitHub
dominikhei commented on issue #53395: URL: https://github.com/apache/airflow/issues/53395#issuecomment-3081126547 > I'm giving it a whirl right now running for all providers, I'll keep you posted. > > EDIT: > > Running "globally" for all providers shows no issues found in all s

Re: [PR] Fix unreachable code mypy warnings in standard provider [airflow]

2025-07-16 Thread via GitHub
jscheffl commented on code in PR #53431: URL: https://github.com/apache/airflow/pull/53431#discussion_r2211667888 ## providers/standard/src/airflow/providers/standard/operators/branch.py: ## @@ -56,9 +56,7 @@ def _expand_task_group_roots( if TYPE_CHECKING:

Re: [PR] Fix jdbc provider unreachable code [airflow]

2025-07-16 Thread via GitHub
gopidesupavan commented on PR #53434: URL: https://github.com/apache/airflow/pull/53434#issuecomment-3081019141 after checking `jaydebeapi` there might be some drivers doesnt support the autocommit , so i think it make sense to keep that false , my bad closing this one .. -- This is an a

Re: [I] AIP-68 Share url params with plugin [airflow]

2025-07-16 Thread via GitHub
AryanK1511 commented on issue #53381: URL: https://github.com/apache/airflow/issues/53381#issuecomment-3081092492 Hello! Can I work on this issue?? I am very new to contributing to the UI, however I would like to take up this challenge. -- This is an automated message from the Ap

Re: [PR] Fix jdbc provider unreachable code [airflow]

2025-07-16 Thread via GitHub
gopidesupavan commented on PR #53434: URL: https://github.com/apache/airflow/pull/53434#issuecomment-3081092174 an example it simulates that behaviours ``` import traceback import warnings from contextlib import contextmanager @contextmanager def suppress_and_warn(*e

Re: [PR] Fix jdbc provider unreachable code [airflow]

2025-07-16 Thread via GitHub
gopidesupavan commented on PR #53434: URL: https://github.com/apache/airflow/pull/53434#issuecomment-3081065830 > As for the future I still I would consider a PR makes sense to (1) add a type:ignore to suppress the warnign and (2) also add a comment why this is needed -for the future yo

Re: [PR] Fix jdbc provider unreachable code [airflow]

2025-07-16 Thread via GitHub
jscheffl commented on PR #53434: URL: https://github.com/apache/airflow/pull/53434#issuecomment-3081042741 As for the future I still I would consider a PR makes sense to (1) add a type:ignore to suppress the warnign and (2) also add a comment why this is needed -for the future you/me th

Re: [PR] Fix unreachable code mypy warnings in standard provider [airflow]

2025-07-16 Thread via GitHub
jscheffl commented on code in PR #53431: URL: https://github.com/apache/airflow/pull/53431#discussion_r2211652317 ## providers/standard/src/airflow/providers/standard/sensors/date_time.py: ## @@ -99,9 +99,6 @@ def poke(self, context: Context) -> bool: @property def _

Re: [PR] Fix jdbc provider unreachable code [airflow]

2025-07-16 Thread via GitHub
gopidesupavan closed pull request #53434: Fix jdbc provider unreachable code URL: https://github.com/apache/airflow/pull/53434 -- 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. T

Re: [PR] Support system theme [airflow]

2025-07-16 Thread via GitHub
RoyLee1224 commented on PR #52649: URL: https://github.com/apache/airflow/pull/52649#issuecomment-3080716079 @bbovenzi Sure, I'll give it a try this week. -- 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 t

Re: [PR] Navigate Grid view via keyboard [airflow]

2025-07-16 Thread via GitHub
RoyLee1224 commented on PR #51784: URL: https://github.com/apache/airflow/pull/51784#issuecomment-3080693379 @bbovenzi Sure, I'll give it a try this week. -- 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 t

(airflow) branch constraints-main updated: Updating constraints. Github run id:16328573564

2025-07-16 Thread github-bot
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 f23f2126061 Updating co

Re: [I] Errors are not sent to Sentry in Airflow 3.0.2 [airflow]

2025-07-16 Thread via GitHub
opeida commented on issue #52368: URL: https://github.com/apache/airflow/issues/52368#issuecomment-3080545081 I've just tested on 3.0.3. Not working -- 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 t

  1   2   3   4   >