[GitHub] [airflow-checks-action] dependabot[bot] opened a new pull request, #7: Bump minimist from 1.2.5 to 1.2.8

2023-03-05 Thread via GitHub
dependabot[bot] opened a new pull request, #7: URL: https://github.com/apache/airflow-checks-action/pull/7 Bumps [minimist](https://github.com/minimistjs/minimist) from 1.2.5 to 1.2.8. Changelog Sourced from https://github.com/minimistjs/minimist/blob/main/CHANGELOG.md";>minimist's

[GitHub] [airflow-checks-action] dependabot[bot] opened a new pull request, #8: Bump node-fetch from 2.6.1 to 2.6.9

2023-04-21 Thread via GitHub
dependabot[bot] opened a new pull request, #8: URL: https://github.com/apache/airflow-checks-action/pull/8 Bumps [node-fetch](https://github.com/node-fetch/node-fetch) from 2.6.1 to 2.6.9. Release notes Sourced from https://github.com/node-fetch/node-fetch/releases";>node-fetch's

[GitHub] [airflow-checks-action] dependabot[bot] opened a new pull request, #9: Bump ansi-regex from 4.1.0 to 4.1.1

2023-04-21 Thread via GitHub
dependabot[bot] opened a new pull request, #9: URL: https://github.com/apache/airflow-checks-action/pull/9 Bumps [ansi-regex](https://github.com/chalk/ansi-regex) from 4.1.0 to 4.1.1. Commits https://github.com/chalk/ansi-regex/commit/64735d25eb839b55bc9fae3877edb702b4c92ca2";>6

Re: [PR] Add S3 to github sync script and workflow [airflow-site-archive]

2025-04-27 Thread via GitHub
potiuk commented on PR #1: URL: https://github.com/apache/airflow-site-archive/pull/1#issuecomment-2833984439 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 un

Re: [PR] Add S3 to github sync script and workflow [airflow-site-archive]

2025-04-27 Thread via GitHub
potiuk commented on PR #1: URL: https://github.com/apache/airflow-site-archive/pull/1#issuecomment-2833989448 I think we should prioritise some of the speed-ups we can do for our workflows - this one. ARM images and ARC is now top of my priority i think. -- This is an automated message

Re: [PR] Add S3 to github sync script and workflow [airflow-site-archive]

2025-04-27 Thread via GitHub
gopidesupavan commented on PR #1: URL: https://github.com/apache/airflow-site-archive/pull/1#issuecomment-2834163934 > I think we should prioritise some of the speed-ups we can do for our workflows - this one. ARM images and ARC is now top of my priority i think. Yeah offcourse, can

Re: [PR] Change notifications to comm...@airflow.apache.org [airflow-site-archive]

2025-05-02 Thread via GitHub
Copilot commented on code in PR #5: URL: https://github.com/apache/airflow-site-archive/pull/5#discussion_r2071513927 ## .asf.yaml: ## @@ -0,0 +1,62 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file

Re: [PR] Add S3/Github/S3 sync scripts and workflows [airflow-site-archive]

2025-05-01 Thread via GitHub
gopidesupavan merged PR #1: URL: https://github.com/apache/airflow-site-archive/pull/1 -- 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: dev-unsubscr...@a

[PR] Update run_with_pool function to use map for list [airflow-site-archive]

2025-05-01 Thread via GitHub
gopidesupavan opened a new pull request, #2: URL: https://github.com/apache/airflow-site-archive/pull/2 (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 unsu

Re: [PR] Update run_with_pool function to use map for list [airflow-site-archive]

2025-05-01 Thread via GitHub
potiuk merged PR #2: URL: https://github.com/apache/airflow-site-archive/pull/2 -- 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: dev-unsubscr...@airflow.

[PR] Add s3 to github sync script and workflow [airflow-site-archive]

2025-04-27 Thread via GitHub
gopidesupavan opened a new pull request, #1: URL: https://github.com/apache/airflow-site-archive/pull/1 (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 unsu

[PR] Copy all historical docs to airflow-site-archive [airflow-site-archive]

2025-05-01 Thread via GitHub
gopidesupavan opened a new pull request, #3: URL: https://github.com/apache/airflow-site-archive/pull/3 (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 unsu

Re: [PR] Copy all historical docs to airflow-site-archive [airflow-site-archive]

2025-05-01 Thread via GitHub
gopidesupavan commented on PR #3: URL: https://github.com/apache/airflow-site-archive/pull/3#issuecomment-2845916259 It will be easy after switch to S3, we just have to nuke the folders from s3 :) so it will not display in documentation page. -- This is an automated message from the Apac

Re: [PR] Copy all historical docs to airflow-site-archive [airflow-site-archive]

2025-05-01 Thread via GitHub
potiuk commented on PR #3: URL: https://github.com/apache/airflow-site-archive/pull/3#issuecomment-2846279558 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 unsu

Re: [PR] Copy all historical docs to airflow-site-archive [airflow-site-archive]

2025-05-01 Thread via GitHub
potiuk merged PR #3: URL: https://github.com/apache/airflow-site-archive/pull/3 -- 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: dev-unsubscr...@airflow.

Re: [PR] Copy all historical docs to airflow-site-archive [airflow-site-archive]

2025-05-02 Thread via GitHub
gopidesupavan commented on PR #3: URL: https://github.com/apache/airflow-site-archive/pull/3#issuecomment-2846781510 > Cool. But this is for "all" - I guess what we will be doing most of the time is just syncing "latest" airflow or "latest" providers ? Yeah for latest changes it will

[PR] Change notifications to comm...@airflow.apache.org [airflow-site-archive]

2025-05-02 Thread via GitHub
jscheffl opened a new pull request, #5: URL: https://github.com/apache/airflow-site-archive/pull/5 I noticed that all PRs are sent to dev@airflow.apache.org - probably we don't want to spam the devlist - we have comm...@airflow.apache.org on the other repo for this. Replicating the confi

Re: [PR] Change notifications to comm...@airflow.apache.org [airflow-site-archive]

2025-05-02 Thread via GitHub
jscheffl commented on code in PR #5: URL: https://github.com/apache/airflow-site-archive/pull/5#discussion_r2071523554 ## .asf.yaml: ## @@ -0,0 +1,62 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file

Re: [PR] Change notifications to comm...@airflow.apache.org [airflow-site-archive]

2025-05-02 Thread via GitHub
gopidesupavan commented on code in PR #5: URL: https://github.com/apache/airflow-site-archive/pull/5#discussion_r2071516820 ## .asf.yaml: ## @@ -0,0 +1,62 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE

Re: [PR] Change notifications to comm...@airflow.apache.org [airflow-site-archive]

2025-05-02 Thread via GitHub
potiuk commented on code in PR #5: URL: https://github.com/apache/airflow-site-archive/pull/5#discussion_r2071522398 ## .asf.yaml: ## @@ -0,0 +1,62 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +

Re: [PR] Change notifications to comm...@airflow.apache.org [airflow-site-archive]

2025-05-02 Thread via GitHub
potiuk merged PR #5: URL: https://github.com/apache/airflow-site-archive/pull/5 -- 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: dev-unsubscr...@airflow.

Re: [PR] Copy all historical docs to airflow-site-archive [airflow-site-archive]

2025-05-02 Thread via GitHub
potiuk commented on PR #3: URL: https://github.com/apache/airflow-site-archive/pull/3#issuecomment-2846606118 > Awesome Indeed looking forward to sub-5 minutes updates to the docs :) -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Copy all historical docs to airflow-site-archive [airflow-site-archive]

2025-05-02 Thread via GitHub
kaxil commented on PR #3: URL: https://github.com/apache/airflow-site-archive/pull/3#issuecomment-2846576632 Awesome -- 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 uns

[PR] Fix nospace issue [airflow-site-archive]

2025-05-02 Thread via GitHub
gopidesupavan opened a new pull request, #4: URL: https://github.com/apache/airflow-site-archive/pull/4 (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 unsu

Re: [PR] Copy all historical docs to airflow-site-archive [airflow-site-archive]

2025-05-02 Thread via GitHub
gopidesupavan commented on PR #3: URL: https://github.com/apache/airflow-site-archive/pull/3#issuecomment-2846670297 > > Awesome > > Indeed looking forward to sub-5 minutes updates to the docs :) Yeah, the whole sync is taking around 20min, this is really good :) https://githu

Re: [PR] Copy all historical docs to airflow-site-archive [airflow-site-archive]

2025-05-02 Thread via GitHub
gopidesupavan commented on PR #3: URL: https://github.com/apache/airflow-site-archive/pull/3#issuecomment-2846671823 We can parallelise with more process, at present it runs 4 parallel process. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Fix nospace issue [airflow-site-archive]

2025-05-02 Thread via GitHub
gopidesupavan commented on PR #4: URL: https://github.com/apache/airflow-site-archive/pull/4#issuecomment-2846673414 Some teething issues :) -- 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

Re: [PR] Copy all historical docs to airflow-site-archive [airflow-site-archive]

2025-05-01 Thread via GitHub
gopidesupavan commented on PR #3: URL: https://github.com/apache/airflow-site-archive/pull/3#issuecomment-2845898841 Fixed all the ASF non complaint CSP issues. -- 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: [PR] Copy all historical docs to airflow-site-archive [airflow-site-archive]

2025-05-01 Thread via GitHub
gopidesupavan commented on PR #3: URL: https://github.com/apache/airflow-site-archive/pull/3#issuecomment-2845900566 @kaxil will workout separate for the old version archival, after switching to S3 -- This is an automated message from the Apache Git Service. To respond to the message, p

Re: [PR] Fix nospace issue [airflow-site-archive]

2025-05-02 Thread via GitHub
potiuk merged PR #4: URL: https://github.com/apache/airflow-site-archive/pull/4 -- 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: dev-unsubscr...@airflow.

Re: [PR] Copy all historical docs to airflow-site-archive [airflow-site-archive]

2025-05-02 Thread via GitHub
potiuk commented on PR #3: URL: https://github.com/apache/airflow-site-archive/pull/3#issuecomment-2846751947 Cool. But this is for "all" - I guess what we will be doing most of the time is just syncing "latest" airflow or "latest" providers ? -- This is an automated message from the Apa

Re: [I] DAGs disappear sometime after callback execution [airflow]

2025-09-05 Thread via GitHub
boring-cyborg[bot] commented on issue #55315: URL: https://github.com/apache/airflow/issues/55315#issuecomment-3259853859 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

Re: [PR] Complete Hebrew gaps 2025-09-01 [airflow]

2025-09-05 Thread via GitHub
shahar1 merged PR #55123: URL: https://github.com/apache/airflow/pull/55123 -- 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: [PR] Implement `batch_is_authorized_` methods in AWS auth manager [airflow]

2025-09-05 Thread via GitHub
o-nikolas commented on code in PR #55307: URL: https://github.com/apache/airflow/pull/55307#discussion_r2326124827 ## providers/amazon/src/airflow/providers/amazon/aws/auth_manager/aws_auth_manager.py: ## @@ -244,25 +247,93 @@ def _has_access_to_menu_item(request: IsAuthorizedR

Re: [PR] Add CloudComposerTriggerDAGRunOperator for Cloud Composer service [airflow]

2025-09-05 Thread via GitHub
potiuk merged PR #55256: URL: https://github.com/apache/airflow/pull/55256 -- 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

[PR] 🩹 fix: Use GoogleBaseAsyncHook [airflow]

2025-09-05 Thread via GitHub
morgan-dgk opened a new pull request, #55316: URL: https://github.com/apache/airflow/pull/55316 When run with `deferrable=True`, the `CloudRunExecuteJobOperator` fails with the error `You cannot use AsyncToSync in the same thread as an async event loop - just await the async function direct

Re: [PR] Add worker maintenance mode functionality to Edge3 provider UI [airflow]

2025-09-05 Thread via GitHub
jscheffl commented on code in PR #55301: URL: https://github.com/apache/airflow/pull/55301#discussion_r2326096542 ## providers/edge3/src/airflow/providers/edge3/worker_api/routes/ui.py: ## @@ -100,3 +105,57 @@ def jobs( jobs=result, total_entries=len(result),

Re: [PR] Add worker maintenance mode functionality to Edge3 provider UI [airflow]

2025-09-05 Thread via GitHub
dheerajturaga commented on code in PR #55301: URL: https://github.com/apache/airflow/pull/55301#discussion_r2326208332 ## providers/edge3/src/airflow/providers/edge3/worker_api/routes/ui.py: ## @@ -100,3 +105,57 @@ def jobs( jobs=result, total_entries=len(resul

[I] Task Instances not correctly displayed in task group header in grid view [airflow]

2025-09-05 Thread via GitHub
jscheffl opened a new issue, #55319: URL: https://github.com/apache/airflow/issues/55319 ### Apache Airflow version main (development) ### If "Other Airflow 2 version" selected, which one? _No response_ ### What happened? If I take a look to a task group the

Re: [PR] Fix PL translation gaps - 2025-09-05 [airflow]

2025-09-05 Thread via GitHub
potiuk commented on PR #55293: URL: https://github.com/apache/airflow/pull/55293#issuecomment-3259563087 I am sorry guys... But I could not resist :) https://github.com/user-attachments/assets/d3666bdd-42b6-49ab-af20-a8114b3d861a"; /> I **REALLY** wanted an easter egg in Airflo

Re: [I] Status of testing Providers that were prepared on September 05, 2025 [airflow]

2025-09-05 Thread via GitHub
jscheffl commented on issue #55285: URL: https://github.com/apache/airflow/issues/55285#issuecomment-3259629264 Tested the changes in Edge UI and looks good - a lot of features missing but might need an ad-hoc then again together with 3.1 release or so. -- This is an automated message fro

Re: [I] Better trigger visibility for watchers [airflow]

2025-09-05 Thread via GitHub
dstandish commented on issue #55314: URL: https://github.com/apache/airflow/issues/55314#issuecomment-3259851896 cc @vincbeck @Lee-W @jroachgolf84 @cmarteepants -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[I] DAGs disappear sometime after callback execution [airflow]

2025-09-05 Thread via GitHub
kirillsights opened a new issue, #55315: URL: https://github.com/apache/airflow/issues/55315 ### Apache Airflow version 3.0.6 ### If "Other Airflow 2 version" selected, which one? _No response_ ### What happened? After upgrade to airflow 3, system started ex

Re: [PR] Make `batch_is_authorized_` methods team aware [airflow]

2025-09-05 Thread via GitHub
o-nikolas commented on code in PR #55306: URL: https://github.com/apache/airflow/pull/55306#discussion_r2326108782 ## airflow-core/src/airflow/api_fastapi/core_api/security.py: ## @@ -255,21 +255,28 @@ def inner( request: BulkBody[PoolBody], user: GetUserDep,

Re: [PR] Include simple context in triggerer async callback [airflow]

2025-09-05 Thread via GitHub
ferruzzi commented on code in PR #55241: URL: https://github.com/apache/airflow/pull/55241#discussion_r2325559579 ## task-sdk/src/airflow/sdk/definitions/deadline.py: ## @@ -120,8 +120,10 @@ class Callback(ABC): path: str kwargs: dict | None -def __init__(self, c

Re: [PR] Add worker maintenance mode functionality to Edge3 provider UI [airflow]

2025-09-05 Thread via GitHub
dheerajturaga commented on code in PR #55301: URL: https://github.com/apache/airflow/pull/55301#discussion_r2326338148 ## providers/edge3/src/airflow/providers/edge3/plugins/www/src/pages/WorkerPage.tsx: ## @@ -16,22 +16,178 @@ * specific language governing permissions and lim

[PR] [v3-0-test] modify asset ui doc (#55152) [airflow]

2025-09-06 Thread via GitHub
github-actions[bot] opened a new pull request, #55326: URL: https://github.com/apache/airflow/pull/55326 (cherry picked from commit 9b65b5b630f79d393980332d97c781c795334c5c) Co-authored-by: Xch1 <34787736+xch...@users.noreply.github.com> Signed-off-by: Xch1 Co-authored-by: Xch1

Re: [PR] modify asset ui doc [airflow]

2025-09-06 Thread via GitHub
boring-cyborg[bot] commented on PR #55152: URL: https://github.com/apache/airflow/pull/55152#issuecomment-3262269015 Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions.

Re: [PR] Do not fail when translation-changing commit runs as canary run [airflow]

2025-09-06 Thread via GitHub
potiuk merged PR #55325: URL: https://github.com/apache/airflow/pull/55325 -- 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

Re: [PR] Do not fail when translation-changing commit runs as canary run [airflow]

2025-09-06 Thread via GitHub
github-actions[bot] commented on PR #55325: URL: https://github.com/apache/airflow/pull/55325#issuecomment-3262285821 ### Backport failed to create: v3-0-test. View the failure log Run details Status Branch Result

Re: [PR] modify asset ui doc [airflow]

2025-09-06 Thread via GitHub
github-actions[bot] commented on PR #55152: URL: https://github.com/apache/airflow/pull/55152#issuecomment-3262270807 ### Backport successfully created: v3-0-test Status Branch Result

Re: [I] RTL UI orientation (Intermittent issue) [airflow]

2025-09-06 Thread via GitHub
viiccwen commented on issue #55263: URL: https://github.com/apache/airflow/issues/55263#issuecomment-3261510747 Could you share how to do that? bcz I couldn't reproduce. 🤡 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Fix the write-to-es feature [airflow]

2025-09-06 Thread via GitHub
Owen-CH-Leung commented on code in PR #53821: URL: https://github.com/apache/airflow/pull/53821#discussion_r2327491621 ## providers/elasticsearch/src/airflow/providers/elasticsearch/log/es_task_handler.py: ## @@ -411,46 +448,16 @@ def _format_msg(self, hit: Hit):

[PR] fix: allow slash in variable key when retrieving variable value with task-sdk [airflow]

2025-09-06 Thread via GitHub
vanducng opened a new pull request, #55324: URL: https://github.com/apache/airflow/pull/55324 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#p

Re: [PR] Check if plural forms are correct for languages. [airflow]

2025-09-06 Thread via GitHub
shahar1 commented on code in PR #55327: URL: https://github.com/apache/airflow/pull/55327#discussion_r2327650220 ## dev/i18n/check_translations_completeness.py: ## @@ -467,9 +485,9 @@ def cli(language: str | None = None, add_missing: bool = False): summary, )

Re: [PR] Check if plural forms are correct for languages. [airflow]

2025-09-06 Thread via GitHub
shahar1 commented on code in PR #55327: URL: https://github.com/apache/airflow/pull/55327#discussion_r2327650220 ## dev/i18n/check_translations_completeness.py: ## @@ -467,9 +485,9 @@ def cli(language: str | None = None, add_missing: bool = False): summary, )

[PR] Do not fail when translation-changing commit runs as canary run [airflow]

2025-09-06 Thread via GitHub
potiuk opened a new pull request, #55325: URL: https://github.com/apache/airflow/pull/55325 When we run with "translation-changing" commit in a canary run, we should not fail the build as the change has already been deliberately merged (likely with "allow translation changes" label).

Re: [I] Expensive Trigger Query Does Not Use Index (even after forcing a hint) Due to Additional Column [airflow]

2025-09-06 Thread via GitHub
potiuk commented on issue #40674: URL: https://github.com/apache/airflow/issues/40674#issuecomment-3262451357 Assuming fixed in Airflow 3. Does not seem worth / critical to backport to 2. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] Add worker maintenance mode functionality to Edge3 provider UI [airflow]

2025-09-06 Thread via GitHub
jscheffl commented on code in PR #55301: URL: https://github.com/apache/airflow/pull/55301#discussion_r232598 ## providers/edge3/src/airflow/providers/edge3/worker_api/routes/ui.py: ## @@ -100,3 +105,57 @@ def jobs( jobs=result, total_entries=len(result),

[PR] Fix typo introduced in #55125 [airflow]

2025-09-06 Thread via GitHub
ashb opened a new pull request, #55329: URL: https://github.com/apache/airflow/pull/55329 ``` error: unexpected argument '--no-python-download' found tip: a similar argument exists: '--no-python-downloads' ``` --- **^ Add meaningful description a

[PR] Fix grid view to handle long task name [airflow]

2025-09-06 Thread via GitHub
guan404ming opened a new pull request, #55332: URL: https://github.com/apache/airflow/pull/55332 ## Related Issue Closes https://github.com/apache/airflow/issues/55171 ## How Limit the width for task name https://github.com/user-attachments/ass

[PR] Exposing DAG Parsing duration in the UI [airflow]

2025-09-06 Thread via GitHub
SamWheating opened a new pull request, #54752: URL: https://github.com/apache/airflow/pull/54752 closes: https://github.com/apache/airflow/issues/54141 This PR adds a `last_parse_time` column to the DAG table, and updates it during the DAG processing loop. This field is then expos

[PR] Update minimum size for Gantt [airflow]

2025-09-06 Thread via GitHub
guan404ming opened a new pull request, #55330: URL: https://github.com/apache/airflow/pull/55330 ## Related Issue Closes #55244 ## Why Bars with the minimal size (10px) don't display properly when `responsive: true` in Chart.js. ## How We ne

Re: [PR] Fix the misalignment of gantt view progress bars (#55061) [airflow]

2025-09-06 Thread via GitHub
guan404ming commented on PR #55291: URL: https://github.com/apache/airflow/pull/55291#issuecomment-3262521834 Closes https://github.com/apache/airflow/issues/55061 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] Fix typo introduced in #55125 [airflow]

2025-09-06 Thread via GitHub
github-actions[bot] commented on PR #55329: URL: https://github.com/apache/airflow/pull/55329#issuecomment-3262443326 ### Backport failed to create: v3-0-test. View the failure log Run details Status Branch Result

Re: [PR] Fix the misalignment of gantt view progress bars (#55061) [airflow]

2025-09-06 Thread via GitHub
guan404ming commented on code in PR #55291: URL: https://github.com/apache/airflow/pull/55291#discussion_r2327797697 ## airflow-core/src/airflow/ui/src/layouts/Details/Gantt/Gantt.tsx: ## @@ -185,9 +187,9 @@ export const Gantt = ({ limit }: Props) => { minBarLength: M

Re: [PR] Update color palette and leverage Chakra semantic tokens [airflow]

2025-09-06 Thread via GitHub
odaneau-astro commented on code in PR #53981: URL: https://github.com/apache/airflow/pull/53981#discussion_r2327733608 ## airflow-core/src/airflow/ui/src/pages/Dag/Calendar/CalendarCell.tsx: ## @@ -38,13 +41,15 @@ export const CalendarCell = ({ backgroundColor, content, index,

Re: [PR] Add _two form for the Hebrew translation [airflow]

2025-09-06 Thread via GitHub
potiuk commented on PR #55331: URL: https://github.com/apache/airflow/pull/55331#issuecomment-3262498002 Technical approval of course... It's all Hebrew to me :) -- 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: [PR] Exposing DAG Parsing duration in the UI [airflow]

2025-09-06 Thread via GitHub
potiuk commented on PR #54752: URL: https://github.com/apache/airflow/pull/54752#issuecomment-3262482665 Closed/reopened to re-run with the label applied. -- 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] Check for Deprecated SQLA Query Class Usage in Prevent Deprecated SQLA Hook [airflow]

2025-09-06 Thread via GitHub
Prab-27 commented on PR #55274: URL: https://github.com/apache/airflow/pull/55274#issuecomment-3262595028 @potiuk , Could you please let me know if there are any other deprecated query forms I should add? 1 query attribute .query() - added 2 class of session.orm -> Query - implementin

Re: [PR] Check for Deprecated SQLA Query Class Usage in Prevent Deprecated SQLA Hook [airflow]

2025-09-06 Thread via GitHub
potiuk commented on PR #55274: URL: https://github.com/apache/airflow/pull/55274#issuecomment-3262600478 > @potiuk , Could you please let me know if there are any other deprecated query forms I should add? Hard to say - I am not well versed in SQLAlchemy 2 deprecations - but maybe @D

Re: [PR] Improve cache usage for prek hooks [airflow]

2025-09-06 Thread via GitHub
shahar1 commented on code in PR #55328: URL: https://github.com/apache/airflow/pull/55328#discussion_r2327921656 ## .github/actions/install-prek/action.yml: ## @@ -91,3 +94,17 @@ runs: working-directory: ${{ github.workspace }} env: SKIP: ${{ inputs.skip-p

Re: [I] Add speicification of version in README if provider's extra has specified version [airflow]

2025-09-06 Thread via GitHub
potiuk commented on issue #55207: URL: https://github.com/apache/airflow/issues/55207#issuecomment-3262695543 Those files are generated by release manager, you can also re-apply templates to all providers via `breeze release-management prepare-providers-documentation --apply-templates-only`

[PR] Optimize Gantt group expansion with debouncing and deferred rendering [airflow]

2025-09-06 Thread via GitHub
guan404ming opened a new pull request, #55334: URL: https://github.com/apache/airflow/pull/55334 ## Related Issue Closes #55250 ## Why - Group toggle operations cause UI freezes during rapid interactions - Expensive flattenNodes calculations block UI updat

Re: [PR] Remove duplicate TI.next_method from the select statement [airflow]

2025-09-06 Thread via GitHub
gopidesupavan merged PR #55311: URL: https://github.com/apache/airflow/pull/55311 -- 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] Change CloudComposerAsyncHook to inherit from GoogleBaseAsyncHook instead of GoogleBaseHook. [airflow]

2025-09-06 Thread via GitHub
eladkal merged PR #54976: URL: https://github.com/apache/airflow/pull/54976 -- 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: [PR] Implement simpler and faster freeze check for translations [airflow]

2025-09-06 Thread via GitHub
potiuk merged PR #55154: URL: https://github.com/apache/airflow/pull/55154 -- 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

Re: [PR] Implement simpler and faster freeze check for translations [airflow]

2025-09-06 Thread via GitHub
potiuk commented on PR #55154: URL: https://github.com/apache/airflow/pull/55154#issuecomment-3262097187 no backport -- 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 unsu

Re: [I] Google Stackdriver Logging not working as expected [airflow]

2025-09-06 Thread via GitHub
potiuk commented on issue #39613: URL: https://github.com/apache/airflow/issues/39613#issuecomment-3262104870 Agreed. -- 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 uns

[PR] Fix missing 'ti_successes' and related metrics in Airflow 3.0 Task SDKBug/missing ti successes metric [airflow]

2025-09-06 Thread via GitHub
HsiuChuanHsu opened a new pull request, #55322: URL: https://github.com/apache/airflow/pull/55322 ## Description This PR addresses issue #52336 by implementing missing task success and duration metrics in the Airflow 3.0 Task SDK. **Metrics Restored**

Re: [PR] Implement simpler and faster freeze check for translations [airflow]

2025-09-06 Thread via GitHub
github-actions[bot] commented on PR #55154: URL: https://github.com/apache/airflow/pull/55154#issuecomment-3262083021 ### Backport failed to create: v3-0-test. View the failure log Run details Status Branch Result

Re: [PR] Fix sync to async issue when getting connection for SQLExecuteQueryTrigger [airflow]

2025-09-06 Thread via GitHub
potiuk commented on PR #55094: URL: https://github.com/apache/airflow/pull/55094#issuecomment-3262093421 > @eladkal Do you prefer using asyncio.to_thread instead of sync_to_async from asgiref? I believe it's good - we are using sync_to_async in other places already. -- This is an a

Re: [PR] Implement simpler and faster freeze check for translations [airflow]

2025-09-06 Thread via GitHub
potiuk commented on PR #55154: URL: https://github.com/apache/airflow/pull/55154#issuecomment-3262081403 Known failure - being fixed in main, -- 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

[PR] Hover effect added to footer social icons [airflow-site]

2025-09-06 Thread via GitHub
Virajjai opened a new pull request, #1237: URL: https://github.com/apache/airflow-site/pull/1237 Closes #1095 Added the hover effect to the social icon in footer section. Changes are in css file. -- This is an automated message from the Apache Git Service. To respond to the message

Re: [I] RuntimeError in trigger creating hooks, Airflow 3.0.3 [airflow]

2025-09-06 Thread via GitHub
eladkal closed issue #53447: RuntimeError in trigger creating hooks, Airflow 3.0.3 URL: https://github.com/apache/airflow/issues/53447 -- 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 co

Re: [PR] Complete translation of tasks.json to Spanish (2025-09-06) [airflow]

2025-09-06 Thread via GitHub
ecodina commented on PR #55313: URL: https://github.com/apache/airflow/pull/55313#issuecomment-3261694270 Hi @shahar1. I'm away this weekend, but will try to review everything before the 9th. -- This is an automated message from the Apache Git Service. To respond to the message, please lo

Re: [PR] Fix sync_hook_class and unit tests of Cloud Run async hook [airflow]

2025-09-06 Thread via GitHub
shahar1 commented on PR #55321: URL: https://github.com/apache/airflow/pull/55321#issuecomment-3261844584 > Hmm. Does not seem to fix things :( Just realized I might have introduced other issues as I fixed tests of another hook (which basically needs the same treatment as the original

Re: [PR] Add GOVERNANCE.md to our repo [airflow]

2025-09-06 Thread via GitHub
potiuk merged PR #55245: URL: https://github.com/apache/airflow/pull/55245 -- 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

Re: [PR] Check if plural forms are correct for languages. [airflow]

2025-09-06 Thread via GitHub
potiuk commented on PR #55327: URL: https://github.com/apache/airflow/pull/55327#issuecomment-3262337974 It turned out that we had bad plural forms for: * ca: @jscheffl @bugraoz93 @ecodina * he: @eladkal @shahar1 @romsharon98 @Dev-iL * es: @jscheffl @bugraoz93 @ecodina * fr:

Re: [PR] Make absolutely sure that uv uses system-installed Python in the image [airflow]

2025-09-06 Thread via GitHub
github-actions[bot] commented on PR #55125: URL: https://github.com/apache/airflow/pull/55125#issuecomment-3262342769 ### Backport failed to create: v3-0-test. View the failure log Run details Status Branch Result

Re: [PR] Check if plural forms are correct for languages. [airflow]

2025-09-06 Thread via GitHub
potiuk commented on PR #55327: URL: https://github.com/apache/airflow/pull/55327#issuecomment-3262428113 There you go: nice report, straight in the PR: https://github.com/apache/airflow/actions/runs/17516289587/job/49754216405?pr=55327#step:4:1 -- This is an automated message from the Apa

Re: [PR] Improving cache usage for prek hooks [airflow]

2025-09-06 Thread via GitHub
potiuk commented on PR #55328: URL: https://github.com/apache/airflow/pull/55328#issuecomment-3262406870 I need to measure the gain, but I feel it's worth it -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

Re: [PR] Make `batch_is_authorized_` methods team aware [airflow]

2025-09-06 Thread via GitHub
jason810496 commented on code in PR #55306: URL: https://github.com/apache/airflow/pull/55306#discussion_r2327647815 ## airflow-core/src/airflow/api_fastapi/core_api/routes/public/import_error.py: ## @@ -179,19 +179,21 @@ def get_import_errors( limit=limit, ses

Re: [PR] Fix typo introduced in #55125 [airflow]

2025-09-06 Thread via GitHub
potiuk merged PR #55329: URL: https://github.com/apache/airflow/pull/55329 -- 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

Re: [PR] Update color palette and leverage Chakra semantic tokens [airflow]

2025-09-06 Thread via GitHub
odaneau-astro commented on code in PR #53981: URL: https://github.com/apache/airflow/pull/53981#discussion_r2327733608 ## airflow-core/src/airflow/ui/src/pages/Dag/Calendar/CalendarCell.tsx: ## @@ -38,13 +41,15 @@ export const CalendarCell = ({ backgroundColor, content, index,

Re: [PR] Fix typo introduced in #55125 [airflow]

2025-09-06 Thread via GitHub
potiuk commented on PR #55329: URL: https://github.com/apache/airflow/pull/55329#issuecomment-3262443463 oups -- 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

Re: [PR] Add update_mask support for bulk PATCH APIs [airflow]

2025-09-06 Thread via GitHub
jason810496 commented on code in PR #54597: URL: https://github.com/apache/airflow/pull/54597#discussion_r2327728390 ## airflow-core/src/airflow/api_fastapi/core_api/routes/public/hitl.py: ## @@ -64,7 +64,7 @@ def _get_task_instance( dag_run_id: str, task_id: str,

Re: [I] Not clear when and what to clean on Redis [airflow]

2025-09-06 Thread via GitHub
potiuk commented on issue #55215: URL: https://github.com/apache/airflow/issues/55215#issuecomment-3262460924 > My plan is to add a “**REDIS MAINTENANCE**” section to the docs that explains common issues , cleanup strategies like setting TTLs, and monitoring best practices. Even if some of

Re: [PR] [RFC] Feat(ui): add italian translation + CODEOWNER [airflow]

2025-09-06 Thread via GitHub
potiuk commented on PR #52342: URL: https://github.com/apache/airflow/pull/52342#issuecomment-326246 For italian it seems to be `_one, _many, _other` -- 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

Re: [PR] Exposing DAG Parsing duration in the UI [airflow]

2025-09-06 Thread via GitHub
potiuk closed pull request #54752: Exposing DAG Parsing duration in the UI URL: https://github.com/apache/airflow/pull/54752 -- 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   2   3   4   5   6   7   8   9   10   >