(airflow) annotated tag 2.9.0rc3 updated (50f22ffe91 -> 65abff8a7b)

2024-04-06 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to annotated tag 2.9.0rc3 in repository https://gitbox.apache.org/repos/asf/airflow.git *** WARNING: tag 2.9.0rc3 was modified! *** from 50f22ffe91 (commit) to 65abff8a7b (tag)

svn commit: r68348 - /dev/airflow/2.9.0rc2/

2024-04-06 Thread ephraimanierobi
Author: ephraimanierobi Date: Sun Apr 7 05:37:19 2024 New Revision: 68348 Log: Remove old release: 2.9.0rc2 Removed: dev/airflow/2.9.0rc2/

svn commit: r68347 - /dev/airflow/2.9.0rc3/

2024-04-06 Thread ephraimanierobi
Author: ephraimanierobi Date: Sun Apr 7 05:36:28 2024 New Revision: 68347 Log: Add artifacts for Airflow 2.9.0rc3 Added: dev/airflow/2.9.0rc3/ dev/airflow/2.9.0rc3/apache-airflow-2.9.0-source.tar.gz (with props) dev/airflow/2.9.0rc3/apache-airflow-2.9.0-source.tar.gz.asc

(airflow) annotated tag constraints-2.9.0rc3 updated (4642ff83ce -> a193953f94)

2024-04-06 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to annotated tag constraints-2.9.0rc3 in repository https://gitbox.apache.org/repos/asf/airflow.git *** WARNING: tag constraints-2.9.0rc3 was modified! *** from 4642ff83ce (commit) to

(airflow) branch v2-9-stable updated (920921ee39 -> 50f22ffe91)

2024-04-06 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch v2-9-stable in repository https://gitbox.apache.org/repos/asf/airflow.git from 920921ee39 Update RELEASE_NOTES.rst add a3eb5f73d6 Load providers configuration when gunicorn

Re: [PR] Sync v2-9-stable with v2-9-test to release 2.9.0 [airflow]

2024-04-06 Thread via GitHub
ephraimbuddy merged PR #38798: URL: https://github.com/apache/airflow/pull/38798 -- 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:

Re: [PR] Adds cancel previous run parameter [airflow]

2024-04-06 Thread via GitHub
Lee-W commented on PR #38702: URL: https://github.com/apache/airflow/pull/38702#issuecomment-2041286647 As many have approved this, I plan to merge it today or tomorrow. Please let me know if anyone want to take a deeper look  -- This is an automated message from the Apache Git

[PR] Add six tests for AWS triggers [airflow]

2024-04-06 Thread via GitHub
slycyberguy opened a new pull request, #38810: URL: https://github.com/apache/airflow/pull/38810 Unit tests added for amazon/aws/triggers: rds, lambda_function, glue_crawler, emr, batch, athena --- **^ Add meaningful description above** Read the

Re: [PR] Add tests to four services in AWS util [airflow]

2024-04-06 Thread via GitHub
boring-cyborg[bot] commented on PR #38809: URL: https://github.com/apache/airflow/pull/38809#issuecomment-2041280857 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

[PR] Add tests to four services in AWS util [airflow]

2024-04-06 Thread via GitHub
slycyberguy opened a new pull request, #38809: URL: https://github.com/apache/airflow/pull/38809 Tests for tags, sagemaker, sqs and rds in Amazon/aws/util are being added --- **^ Add meaningful description above** Read the **[Pull Request

Re: [PR] Add metrics_allow_list and metrics_block_list to statsd deployments env [airflow]

2024-04-06 Thread via GitHub
Xiroo commented on code in PR #38697: URL: https://github.com/apache/airflow/pull/38697#discussion_r1554627220 ## chart/templates/statsd/statsd-deployment.yaml: ## @@ -104,8 +104,17 @@ spec: - "--statsd.mapping-config=/etc/statsd-exporter/mappings.yml"

Re: [I] Connection Extra Field is not encrypted/masked on editing [airflow]

2024-04-06 Thread via GitHub
Bowrna commented on issue #11218: URL: https://github.com/apache/airflow/issues/11218#issuecomment-2041274004 @potiuk Unfortunately I forgot to fix this issue and it has been pending for a long time. I remember seeing your fix around this and then it striked me that I have kept this open

Re: [PR] adding callback hook for on/off state of dag [airflow]

2024-04-06 Thread via GitHub
Bowrna commented on code in PR #38683: URL: https://github.com/apache/airflow/pull/38683#discussion_r1554794562 ## airflow/models/dag.py: ## @@ -690,6 +694,7 @@ def __init__( ) self.dagrun_timeout = dagrun_timeout self.sla_miss_callback =

[PR] fix spelling in pyproject.toml [airflow]

2024-04-06 Thread via GitHub
rawwar opened a new pull request, #38808: URL: https://github.com/apache/airflow/pull/38808 fix spelling in `pyproject.toml` -- 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.

[PR] Paginate Airflow task logs [airflow]

2024-04-06 Thread via GitHub
RNHTTR opened a new pull request, #38807: URL: https://github.com/apache/airflow/pull/38807 It's relatively easy for the webserver to get overwhelmed with large log files served from remote blob storage. Instead of just throwing more memory at the webserver, this seeks to

Re: [PR] Configure HttpHook's auth_type from Connection [airflow]

2024-04-06 Thread via GitHub
github-actions[bot] commented on PR #35591: URL: https://github.com/apache/airflow/pull/35591#issuecomment-2041247962 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

Re: [PR] [WIP] Pinecone provider support for `pinecone-client`>=3 [airflow]

2024-04-06 Thread via GitHub
github-actions[bot] commented on PR #37307: URL: https://github.com/apache/airflow/pull/37307#issuecomment-2041247952 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

Re: [PR] Feature: Return rowcount from SQL Operators [airflow]

2024-04-06 Thread via GitHub
github-actions[bot] commented on PR #37381: URL: https://github.com/apache/airflow/pull/37381#issuecomment-2041247943 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

Re: [PR] Add secret caching for Airflow CLI [airflow]

2024-04-06 Thread via GitHub
github-actions[bot] commented on PR #37551: URL: https://github.com/apache/airflow/pull/37551#issuecomment-2041247934 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

[PR] fix: try002 for provider openlineage [airflow]

2024-04-06 Thread via GitHub
dondaum opened a new pull request, #38806: URL: https://github.com/apache/airflow/pull/38806 related: #38761 fix TRY002 for provider openlineage ### [openlineage](https://github.com/apache/airflow/tree/main/airflow/providers/openlineage) - [x]

[PR] fix: try002 for provider microsoft azure [airflow]

2024-04-06 Thread via GitHub
dondaum opened a new pull request, #38805: URL: https://github.com/apache/airflow/pull/38805 related: #38761 fix TRY002 for provider microsoft azure ### [microsoft.azure](https://github.com/apache/airflow/tree/main/airflow/providers/microsoft/azure) -

[PR] fix: try002 for provider imap [airflow]

2024-04-06 Thread via GitHub
dondaum opened a new pull request, #38804: URL: https://github.com/apache/airflow/pull/38804 related: #38761 fix TRY002 for provider imap ### [imap](https://github.com/apache/airflow/tree/main/airflow/providers/imap) - [x]

Re: [I] Default credentials cause warnings on Google Chrome after logging in to Airflow's UI [airflow]

2024-04-06 Thread via GitHub
shahar1 commented on issue #38797: URL: https://github.com/apache/airflow/issues/38797#issuecomment-2041213783 > I would rather address this issue to the Google, because it annoys everywhere, even if host is localhost, and there is no easy way to turn it of for specific dev endpoint. The

[PR] fix: try002 for provider google [airflow]

2024-04-06 Thread via GitHub
dondaum opened a new pull request, #38803: URL: https://github.com/apache/airflow/pull/38803 related: #38761 fix TRY002 for provider google ### [google](https://github.com/apache/airflow/tree/main/airflow/providers/google) - [x]

Re: [PR] (img/edge_label_example.png): changed imaged to new label view [airflow]

2024-04-06 Thread via GitHub
idantepper commented on PR #38802: URL: https://github.com/apache/airflow/pull/38802#issuecomment-2041211928 Hi, I saw that the image in the document wasn't updated so I took the example dag that was given their and started airflow and took a screenshot of the graph -- This is an

[PR] (img/edge_label_example.png): changed imaged to new label view [airflow]

2024-04-06 Thread via GitHub
idantepper opened a new pull request, #38802: URL: https://github.com/apache/airflow/pull/38802 --- **^ Add meaningful description above** Read the **[Pull Request

(airflow) branch constraints-2-9 updated: Upgrade constraints to rc1

2024-04-06 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch constraints-2-9 in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-2-9 by this push: new 4642ff83ce Upgrade

(airflow) branch constraints-2-9 updated: Updating constraints. Github run id:8583693042

2024-04-06 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch constraints-2-9 in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-2-9 by this push: new b86d9678fb Updating

Re: [I] The Airflow Scheduler and Airflow Triggerer are failing to load the openlineage plugin with Custom extractors [airflow]

2024-04-06 Thread via GitHub
rylativity commented on issue #38037: URL: https://github.com/apache/airflow/issues/38037#issuecomment-2041209079 I am running into an issue with the openlineage plugin too, however it is occuring when I try to install the Datahub plugin (`acryl-datahub-airflow-plugin[plugin-v2]`) to

Re: [I] Databricks provider does not support Azure managed identity if more than one identity exists (e.g. ADF, AKS) [airflow]

2024-04-06 Thread via GitHub
ghjklw commented on issue #38762: URL: https://github.com/apache/airflow/issues/38762#issuecomment-2041208864 I'm facing exactly the same problem and I was just starting to look into it myself! Although I considered the same solution as you initially, I feel like just adding these

[PR] fix: try002 for provider fab [airflow]

2024-04-06 Thread via GitHub
dondaum opened a new pull request, #38801: URL: https://github.com/apache/airflow/pull/38801 related: #38761 fix TRY002 for provider fab ### [fab](https://github.com/apache/airflow/tree/main/airflow/providers/fab) - [x]

[PR] fix: try002 for provider common sql [airflow]

2024-04-06 Thread via GitHub
dondaum opened a new pull request, #38800: URL: https://github.com/apache/airflow/pull/38800 related: #38761 fix TRY002 for provider common sql ### [common.sql](https://github.com/apache/airflow/tree/main/airflow/providers/common/sql) - [x]

[PR] fix: try002 for provider cncf kubernetes [airflow]

2024-04-06 Thread via GitHub
dondaum opened a new pull request, #38799: URL: https://github.com/apache/airflow/pull/38799 related: #38761 fix TRY002 for provider cncf kubernetes ### [cncf.kubernetes](https://github.com/apache/airflow/tree/main/airflow/providers/cncf/kubernetes) -

(airflow) branch v2-9-test updated (3a9391c3a1 -> 50f22ffe91)

2024-04-06 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git from 3a9391c3a1 Update hatchling to latest version (1.22.5) (#38780) add 50f22ffe91 fixup! fixup! Update

Re: [PR] fix: try002 for provider amazon [airflow]

2024-04-06 Thread via GitHub
dondaum commented on PR #38789: URL: https://github.com/apache/airflow/pull/38789#issuecomment-2041200553 > Could you merge main? Done. -- 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

Re: [I] Automate links to configuration of providers [airflow]

2024-04-06 Thread via GitHub
hussein-awala commented on issue #34306: URL: https://github.com/apache/airflow/issues/34306#issuecomment-2041197092 @poorvirohidekar assigned you. -- 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

(airflow) branch v2-9-test updated (d40b751fbc -> 3a9391c3a1)

2024-04-06 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git discard d40b751fbc Update hatchling to latest version (1.22.5) (#38780) add 3a9391c3a1 Update hatchling to latest

(airflow) branch v2-9-test updated (9930804829 -> d40b751fbc)

2024-04-06 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git discard 9930804829 fixup! Update RELEASE_NOTES.rst discard 636ad7b26d Load providers configuration when gunicorn workers

(airflow) branch v2-9-test updated (b38575a5a4 -> 9930804829)

2024-04-06 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git discard b38575a5a4 fixup! Update RELEASE_NOTES.rst discard a3eb5f73d6 Load providers configuration when

Re: [PR] fix: try002 for provider amazon [airflow]

2024-04-06 Thread via GitHub
hussein-awala commented on PR #38789: URL: https://github.com/apache/airflow/pull/38789#issuecomment-2041183389 Could you merge 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 specific

[PR] Sync v2-9-stable with v2-9-test to release 2.9.0 [airflow]

2024-04-06 Thread via GitHub
ephraimbuddy opened a new pull request, #38798: URL: https://github.com/apache/airflow/pull/38798 Time for `2.9.0rc3`! -- 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

(airflow) 02/02: fixup! Update RELEASE_NOTES.rst

2024-04-06 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit b38575a5a445ea36fb8fdb1c6fd75ffe7543ad56 Author: Ephraim Anierobi AuthorDate: Sat Apr 6 20:58:16 2024

(airflow) 01/02: Load providers configuration when gunicorn workers start (#38795)

2024-04-06 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git commit a3eb5f73d617b7482fdcc81a888e3599f1f63d75 Author: Jarek Potiuk AuthorDate: Sat Apr 6 21:46:01 2024 +0200

(airflow) branch v2-9-test updated (920921ee39 -> b38575a5a4)

2024-04-06 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch v2-9-test in repository https://gitbox.apache.org/repos/asf/airflow.git from 920921ee39 Update RELEASE_NOTES.rst new a3eb5f73d6 Load providers configuration when gunicorn

Re: [PR] Force loading provider configuration when using config.as_dict [airflow]

2024-04-06 Thread via GitHub
ephraimbuddy closed pull request #38793: Force loading provider configuration when using config.as_dict URL: https://github.com/apache/airflow/pull/38793 -- 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] Force loading provider configuration when using config.as_dict [airflow]

2024-04-06 Thread via GitHub
ephraimbuddy commented on PR #38793: URL: https://github.com/apache/airflow/pull/38793#issuecomment-2041177995 Better fix: https://github.com/apache/airflow/pull/38795 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

(airflow) branch main updated (4d65f216de -> 042c2acaed)

2024-04-06 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from 4d65f216de fix: try002 for provider apache cassandra (#38791) add 042c2acaed Load providers

Re: [PR] Load providers configuration when gunicorn workers start [airflow]

2024-04-06 Thread via GitHub
ephraimbuddy merged PR #38795: URL: https://github.com/apache/airflow/pull/38795 -- 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:

Re: [I] Default credentials cause warnings on Google Chrome after logging in to Airflow's UI [airflow]

2024-04-06 Thread via GitHub
Taragolis commented on issue #38797: URL: https://github.com/apache/airflow/issues/38797#issuecomment-2041173359 I would rather address this issue to the Google, because it annoys everywhere, even if host is localhost, and there is no easy way to turn it of for specific dev endpoint. The

Re: [PR] Fix loading configuration from providers for config endpoints [airflow]

2024-04-06 Thread via GitHub
potiuk closed pull request #38794: Fix loading configuration from providers for config endpoints URL: https://github.com/apache/airflow/pull/38794 -- 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

Re: [PR] Migrate to connexion v3 [airflow]

2024-04-06 Thread via GitHub
Satoshi-Sh commented on PR #37638: URL: https://github.com/apache/airflow/pull/37638#issuecomment-2041152227 I'm working on `tests/api_connexion/endpoints/test_dataset_endpoint.py`. ![Screenshot from 2024-04-06

Re: [PR] doc: add more instructions for `up_for_retry` [airflow]

2024-04-06 Thread via GitHub
RNHTTR commented on PR #38100: URL: https://github.com/apache/airflow/pull/38100#issuecomment-2041150749 I'm going to go ahead and close this out. Seems like the confusion has been sorted, and task instances in the `up_for_retry` state indeed do not consume a worker slot. -- This is an

Re: [PR] doc: add more instructions for `up_for_retry` [airflow]

2024-04-06 Thread via GitHub
RNHTTR closed pull request #38100: doc: add more instructions for `up_for_retry` URL: https://github.com/apache/airflow/pull/38100 -- 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

Re: [PR] Load providers configuration when gunicorn workers start [airflow]

2024-04-06 Thread via GitHub
potiuk commented on PR #38795: URL: https://github.com/apache/airflow/pull/38795#issuecomment-2041149688 Not easily. I will think about a test but I think we should rather merge it separately. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Fix BigQuery connection and add docs [airflow]

2024-04-06 Thread via GitHub
shahar1 commented on code in PR #38430: URL: https://github.com/apache/airflow/pull/38430#discussion_r1554640254 ## airflow/providers/google/cloud/hooks/bigquery.py: ## @@ -82,32 +83,61 @@ class BigQueryHook(GoogleBaseHook, DbApiHook): This hook uses the Google Cloud

Re: [PR] Fix BigQuery connection and add docs [airflow]

2024-04-06 Thread via GitHub
shahar1 commented on code in PR #38430: URL: https://github.com/apache/airflow/pull/38430#discussion_r1554640254 ## airflow/providers/google/cloud/hooks/bigquery.py: ## @@ -82,32 +83,61 @@ class BigQueryHook(GoogleBaseHook, DbApiHook): This hook uses the Google Cloud

Re: [PR] Fix teardown scheduling if indirect upstream is cleared [airflow]

2024-04-06 Thread via GitHub
jscheffl commented on code in PR #38788: URL: https://github.com/apache/airflow/pull/38788#discussion_r1554638601 ## tests/models/test_taskinstance.py: ## @@ -1249,16 +1255,16 @@ def test_depends_on_past(self, dag_maker): pytest.param(

[I] Default credentials cause warnings on Google Chrome after logging in to Airflow's UI [airflow]

2024-04-06 Thread via GitHub
shahar1 opened a new issue, #38797: URL: https://github.com/apache/airflow/issues/38797 ### Official Helm Chart version main (development) ### Apache Airflow version dev ### Kubernetes Version N/A ### Helm Chart configuration _No response_

Re: [PR] Fix Mark Instance state buttons stay disabled if user lacks permission (#37451) [airflow]

2024-04-06 Thread via GitHub
RodrigoGanancia commented on code in PR #38732: URL: https://github.com/apache/airflow/pull/38732#discussion_r1554632631 ## airflow/www/static/js/dag/details/taskInstance/taskActions/ClearInstance.tsx: ## @@ -242,21 +243,23 @@ const ClearInstance = ({

Re: [PR] Add metrics_allow_list and metrics_block_list to statsd deployments env [airflow]

2024-04-06 Thread via GitHub
romsharon98 commented on code in PR #38697: URL: https://github.com/apache/airflow/pull/38697#discussion_r1554628445 ## chart/templates/statsd/statsd-deployment.yaml: ## @@ -104,8 +104,17 @@ spec: - "--statsd.mapping-config=/etc/statsd-exporter/mappings.yml"

Re: [PR] Add metrics_allow_list and metrics_block_list to statsd deployments env [airflow]

2024-04-06 Thread via GitHub
Xiroo commented on code in PR #38697: URL: https://github.com/apache/airflow/pull/38697#discussion_r1554627220 ## chart/templates/statsd/statsd-deployment.yaml: ## @@ -104,8 +104,17 @@ spec: - "--statsd.mapping-config=/etc/statsd-exporter/mappings.yml"

Re: [PR] Add metrics_allow_list and metrics_block_list to statsd deployments env [airflow]

2024-04-06 Thread via GitHub
Xiroo commented on code in PR #38697: URL: https://github.com/apache/airflow/pull/38697#discussion_r1554627220 ## chart/templates/statsd/statsd-deployment.yaml: ## @@ -104,8 +104,17 @@ spec: - "--statsd.mapping-config=/etc/statsd-exporter/mappings.yml"

Re: [PR] Add metrics_allow_list and metrics_block_list to statsd deployments env [airflow]

2024-04-06 Thread via GitHub
Xiroo commented on code in PR #38697: URL: https://github.com/apache/airflow/pull/38697#discussion_r1554627220 ## chart/templates/statsd/statsd-deployment.yaml: ## @@ -104,8 +104,17 @@ spec: - "--statsd.mapping-config=/etc/statsd-exporter/mappings.yml"

[I] Publish Airflow mypy plugin [airflow]

2024-04-06 Thread via GitHub
RNHTTR opened a new issue, #38796: URL: https://github.com/apache/airflow/issues/38796 ### Description mypy can be a little finicky when creating an Airflow project; publishing the [plugin](https://github.com/apache/airflow/tree/main/dev/mypy/plugin) would make it easier for [users

Re: [PR] Fix loading configuration from providers for config endpoints [airflow]

2024-04-06 Thread via GitHub
potiuk commented on PR #38794: URL: https://github.com/apache/airflow/pull/38794#issuecomment-2041105593 Likely superseded by #38795 -- 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

[PR] Load providers configuration when gunicorn workers start [airflow]

2024-04-06 Thread via GitHub
potiuk opened a new pull request, #38795: URL: https://github.com/apache/airflow/pull/38795 When Gunicorn workers start, they do not fork, but start as new Python interpreters and they should also perform loadig of the configuration from installed providers.

Re: [I] Simplify metrics handling [airflow]

2024-04-06 Thread via GitHub
Ibrahim-Mukherjee commented on issue #38655: URL: https://github.com/apache/airflow/issues/38655#issuecomment-2041105131 Hi I would be willing to work on 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

(airflow) branch main updated: fix: try002 for provider apache cassandra (#38791)

2024-04-06 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 4d65f216de fix: try002 for provider apache

Re: [PR] fix: try002 for provider apache cassandra [airflow]

2024-04-06 Thread via GitHub
Taragolis merged PR #38791: URL: https://github.com/apache/airflow/pull/38791 -- 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:

(airflow) branch main updated: fix: try002 for provider alibaba (#38787)

2024-04-06 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new a747aecea2 fix: try002 for provider alibaba

Re: [PR] fix: try002 for provider alibaba [airflow]

2024-04-06 Thread via GitHub
Taragolis merged PR #38787: URL: https://github.com/apache/airflow/pull/38787 -- 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:

Re: [PR] Add `AirflowInternalRuntimeError` for raise "non catchable" errors [airflow]

2024-04-06 Thread via GitHub
Taragolis commented on PR #38778: URL: https://github.com/apache/airflow/pull/38778#issuecomment-2041100616 Final check with all python versions -- 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

Re: [PR] Adding Task ids in DAG details API endpoint #37564 [airflow]

2024-04-06 Thread via GitHub
msoni1369 commented on PR #37866: URL: https://github.com/apache/airflow/pull/37866#issuecomment-2041097727 Working on new Implementation -- 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

Re: [PR] Add metrics_allow_list and metrics_block_list to statsd deployments env [airflow]

2024-04-06 Thread via GitHub
romsharon98 commented on PR #38697: URL: https://github.com/apache/airflow/pull/38697#issuecomment-2041092273 Except this looks good -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] Fix loading configuration from providers for config endpoints [airflow]

2024-04-06 Thread via GitHub
potiuk commented on code in PR #38794: URL: https://github.com/apache/airflow/pull/38794#discussion_r1554585899 ## airflow/api_connexion/endpoints/config_endpoint.py: ## @@ -85,6 +85,9 @@ def get_config(*, section: str | None = None) -> Response: elif expose_config:

Re: [PR] Add metrics_allow_list and metrics_block_list to statsd deployments env [airflow]

2024-04-06 Thread via GitHub
romsharon98 commented on code in PR #38697: URL: https://github.com/apache/airflow/pull/38697#discussion_r1554584784 ## chart/templates/statsd/statsd-deployment.yaml: ## @@ -104,8 +104,17 @@ spec: - "--statsd.mapping-config=/etc/statsd-exporter/mappings.yml"

Re: [I] expand_kwargs does not preserve templates [airflow]

2024-04-06 Thread via GitHub
alexdrydew commented on issue #38745: URL: https://github.com/apache/airflow/issues/38745#issuecomment-2041073791 My concern was primarily about processing untrusted data from external sources in DAGs: it seems malicious data can be used to steal secrets in some cases: ```python

Re: [I] Automate links to configuration of providers [airflow]

2024-04-06 Thread via GitHub
poorvirohidekar commented on issue #34306: URL: https://github.com/apache/airflow/issues/34306#issuecomment-2041068244 @potiuk - I am interested in working on this. Can you please assign it to me? Could you help me understand this issue description? I built the docs in my local

Re: [PR] fix: try002 for provider apache cassandra [airflow]

2024-04-06 Thread via GitHub
dondaum commented on PR #38791: URL: https://github.com/apache/airflow/pull/38791#issuecomment-2041061428 > Could you merge/rebase main branch? Sure! Done. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] fix: try002 for provider airbyte [airflow]

2024-04-06 Thread via GitHub
dondaum commented on PR #38786: URL: https://github.com/apache/airflow/pull/38786#issuecomment-2041060171 Done. Static checks are green now. Am I correct that the failing postgres tests are not connected to my change, are they ? -- This is an automated message from the Apache Git

Re: [PR] fix: try002 for provider alibaba [airflow]

2024-04-06 Thread via GitHub
dondaum commented on code in PR #38787: URL: https://github.com/apache/airflow/pull/38787#discussion_r1554570741 ## airflow/providers/alibaba/cloud/hooks/oss.py: ## @@ -338,30 +338,30 @@ def get_credential(self) -> oss2.auth.Auth: extra_config =

Re: [PR] fix: try002 for provider apache cassandra [airflow]

2024-04-06 Thread via GitHub
hussein-awala commented on PR #38791: URL: https://github.com/apache/airflow/pull/38791#issuecomment-2041055507 Could you merge/rebase main branch? -- 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

[PR] Fix loading configuration from providers for config endpoints [airflow]

2024-04-06 Thread via GitHub
potiuk opened a new pull request, #38794: URL: https://github.com/apache/airflow/pull/38794 The configuration for providers is not loaded by default when airflow configuraiton is loaded, because loading configuration from providers is slow and configuration intialization is done by just

Re: [PR] Force loading provider configuration when using config.as_dict [airflow]

2024-04-06 Thread via GitHub
potiuk commented on code in PR #38793: URL: https://github.com/apache/airflow/pull/38793#discussion_r1554564372 ## airflow/configuration.py: ## @@ -1439,11 +1439,13 @@ def as_dict( ("default", self._default_values), ("airflow.cfg", self), ] +

Re: [PR] Force loading provider configuration when using config.as_dict [airflow]

2024-04-06 Thread via GitHub
potiuk commented on code in PR #38793: URL: https://github.com/apache/airflow/pull/38793#discussion_r1554564565 ## airflow/configuration.py: ## @@ -1439,11 +1439,13 @@ def as_dict( ("default", self._default_values), ("airflow.cfg", self), ] +

Re: [PR] Make `placeholder` of DbApiHook configurable in UI [airflow]

2024-04-06 Thread via GitHub
dabla commented on PR #38528: URL: https://github.com/apache/airflow/pull/38528#issuecomment-2041039654 Thank you for merging this pull request, could someone have a look to check this one also please: https://github.com/apache/airflow/pull/38707 -- This is an automated message from the

(airflow) branch main updated: Unit tests: Reorganize AWS custom waiter (#38785)

2024-04-06 Thread eladkal
This is an automated email from the ASF dual-hosted git repository. eladkal pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 6fd8f36893 Unit tests: Reorganize AWS custom

Re: [PR] Reorganize AWS custom waiter unit tests [airflow]

2024-04-06 Thread via GitHub
eladkal merged PR #38785: URL: https://github.com/apache/airflow/pull/38785 -- 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:

Re: [PR] Force loading provider configuration when using config.as_dict [airflow]

2024-04-06 Thread via GitHub
ashb commented on code in PR #38793: URL: https://github.com/apache/airflow/pull/38793#discussion_r1554547987 ## airflow/configuration.py: ## @@ -1439,11 +1439,13 @@ def as_dict( ("default", self._default_values), ("airflow.cfg", self), ] +

Re: [PR] Force loading provider configuration when using config.as_dict [airflow]

2024-04-06 Thread via GitHub
ashb commented on code in PR #38793: URL: https://github.com/apache/airflow/pull/38793#discussion_r1554548064 ## airflow/configuration.py: ## @@ -1439,11 +1439,13 @@ def as_dict( ("default", self._default_values), ("airflow.cfg", self), ] +

Re: [PR] fix: try002 for provider apache hdfs [airflow]

2024-04-06 Thread via GitHub
boring-cyborg[bot] commented on PR #38792: URL: https://github.com/apache/airflow/pull/38792#issuecomment-2041012703 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

(airflow) branch main updated: fix: try002 for provider apache hdfs (#38792)

2024-04-06 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new e706416cd5 fix: try002 for provider apache hdfs

(airflow) branch main updated (e706416cd5 -> a022d7b70c)

2024-04-06 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from e706416cd5 fix: try002 for provider apache hdfs (#38792) add a022d7b70c fix: try002 for provider apache beam

Re: [PR] fix: try002 for provider apache beam [airflow]

2024-04-06 Thread via GitHub
Taragolis merged PR #38790: URL: https://github.com/apache/airflow/pull/38790 -- 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:

Re: [PR] fix: try002 for provider apache hdfs [airflow]

2024-04-06 Thread via GitHub
Taragolis merged PR #38792: URL: https://github.com/apache/airflow/pull/38792 -- 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:

Re: [PR] fix: try002 for provider airbyte [airflow]

2024-04-06 Thread via GitHub
Taragolis commented on PR #38786: URL: https://github.com/apache/airflow/pull/38786#issuecomment-2041012638 Everything looks fine, just need to fix static checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

(airflow) branch main updated: Disable by default redact sensitive values in tests (#38765)

2024-04-06 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 6f16c9f5a2 Disable by default redact sensitive

Re: [PR] Disable by default redact sensitive values in tests [airflow]

2024-04-06 Thread via GitHub
Taragolis merged PR #38765: URL: https://github.com/apache/airflow/pull/38765 -- 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:

Re: [PR] Force loading provider configuration when using config.as_dict [airflow]

2024-04-06 Thread via GitHub
ephraimbuddy commented on code in PR #38793: URL: https://github.com/apache/airflow/pull/38793#discussion_r1554544576 ## airflow/configuration.py: ## @@ -1439,11 +1439,13 @@ def as_dict( ("default", self._default_values), ("airflow.cfg", self),

Re: [PR] fix: try002 for provider alibaba [airflow]

2024-04-06 Thread via GitHub
Taragolis commented on code in PR #38787: URL: https://github.com/apache/airflow/pull/38787#discussion_r1554540352 ## airflow/providers/alibaba/cloud/hooks/oss.py: ## @@ -338,30 +338,30 @@ def get_credential(self) -> oss2.auth.Auth: extra_config =

  1   2   >