Re: [PR] Support git-sync v4 in Helm Chart [airflow]

2023-12-23 Thread via GitHub
Swalloow commented on code in PR #34731: URL: https://github.com/apache/airflow/pull/34731#discussion_r1435525198 ## chart/templates/_helpers.yaml: ## @@ -202,50 +202,88 @@ If release name contains chart name it will be used as a full name. {{- if .Values.dags.gitSync.sshK

svn commit: r66281 - /release/airflow/clients/python/2.8.0/

2023-12-23 Thread ephraimanierobi
Author: ephraimanierobi Date: Sat Dec 23 08:08:03 2023 New Revision: 66281 Log: Release Apache Airflow Python Client 2.8.0 from 2.8.0rc1 Added: release/airflow/clients/python/2.8.0/ release/airflow/clients/python/2.8.0/apache-airflow-client-2.8.0-bin.tar.gz - copied unchanged from r

(airflow-client-python) tag 2.8.0 created (now 4775ec8)

2023-12-23 Thread ephraimanierobi
This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a change to tag 2.8.0 in repository https://gitbox.apache.org/repos/asf/airflow-client-python.git at 4775ec8 (commit) No new revisions were added by this update.

Re: [PR] Support git-sync v4 in Helm Chart [airflow]

2023-12-23 Thread via GitHub
Swalloow commented on code in PR #34731: URL: https://github.com/apache/airflow/pull/34731#discussion_r1435528015 ## chart/values.yaml: ## @@ -121,7 +121,7 @@ images: pullPolicy: IfNotPresent gitSync: repository: registry.k8s.io/git-sync/git-sync -tag: v3.6.9 +

(airflow) branch main updated: Add 'uuid_column', 'tenant' params to WeaviateIngestOperator (#36387)

2023-12-23 Thread utkarsharma
This is an automated email from the ASF dual-hosted git repository. utkarsharma 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 ff3b8daac0 Add 'uuid_column', 'tenant' params

Re: [PR] Add 'uuid_column', 'tenant' params to WeaviateIngestOperator [airflow]

2023-12-23 Thread via GitHub
utkarsharma2 merged PR #36387: URL: https://github.com/apache/airflow/pull/36387 -- 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...@air

Re: [I] Impersonation logic missing in BigQuery Async operators [airflow]

2023-12-23 Thread via GitHub
shahar1 commented on issue #34727: URL: https://github.com/apache/airflow/issues/34727#issuecomment-1868242009 > @melugoyal I think it is more complicated than that. As mentioned above, if you follow the trigger through it leads to `GoogleBaseHook.provide_gcp_credential_file_as_context()` w

[PR] Add libxml2 install command for macOS in contributor quick start doc [airflow]

2023-12-23 Thread via GitHub
yo1956 opened a new pull request, #36388: URL: https://github.com/apache/airflow/pull/36388 The both documents below states that libxml2 should be installed as a required package for the pre-commit hook. While the first document includes installation commands for both Linux and Mac, the

Re: [PR] Add libxml2 install command for macOS in contributor quick start doc [airflow]

2023-12-23 Thread via GitHub
boring-cyborg[bot] commented on PR #36388: URL: https://github.com/apache/airflow/pull/36388#issuecomment-1868245332 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 Contribution

Re: [PR] Add libxml2 install command for macOS in contributor quick start doc [airflow]

2023-12-23 Thread via GitHub
Taragolis commented on code in PR #36388: URL: https://github.com/apache/airflow/pull/36388#discussion_r1435568594 ## CONTRIBUTORS_QUICK_START.rst: ## @@ -543,10 +543,18 @@ To avoid burden on CI infrastructure and to save time, Pre-commit hooks can be r 1. Installing requir

Re: [PR] Add libxml2 install command for macOS in contributor quick start doc [airflow]

2023-12-23 Thread via GitHub
yo1956 commented on code in PR #36388: URL: https://github.com/apache/airflow/pull/36388#discussion_r1435576775 ## CONTRIBUTORS_QUICK_START.rst: ## @@ -543,10 +543,18 @@ To avoid burden on CI infrastructure and to save time, Pre-commit hooks can be r 1. Installing required

[I] Identify duplicate kubernetes section airflow configuration and mark them as deprecated [airflow]

2023-12-23 Thread via GitHub
dirrao opened a new issue, #36389: URL: https://github.com/apache/airflow/issues/36389 ### Apache Airflow Provider(s) cncf-kubernetes ### Versions of Apache Airflow Providers apache-airflow-providers-cncf-kubernetes: 7.11.0 ### Apache Airflow version 2.8.0

Re: [PR] add impersonation_chain support when calling Bigquery Operators in deferrable mode [airflow]

2023-12-23 Thread via GitHub
m1racoli commented on PR #36341: URL: https://github.com/apache/airflow/pull/36341#issuecomment-1868290626 FYI We might get better impersonation support in gcloud-aio soon. :) https://github.com/talkiq/gcloud-aio/pull/665 -- This is an automated message from the Apache Git Service. To r

Re: [I] Status of testing Providers that were prepared on December 23, 2023 [airflow]

2023-12-23 Thread via GitHub
Joffreybvn commented on issue #36384: URL: https://github.com/apache/airflow/issues/36384#issuecomment-1868292197 Databricks and ODBC providers work fine. Tested #36205 and #36000 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

Re: [PR] Fix --use-airflow-version constraints [airflow]

2023-12-23 Thread via GitHub
potiuk merged PR #36378: URL: https://github.com/apache/airflow/pull/36378 -- 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: [I] Status of testing Providers that were prepared on December 23, 2023 [airflow]

2023-12-23 Thread via GitHub
potiuk commented on issue #36384: URL: https://github.com/apache/airflow/issues/36384#issuecomment-1868298706 > Would it be possible to have an RC2 to include this commits <[75d74b1](https://github.com/apache/airflow/commit/75d74b1f3a535fdc3624077bde3a34d1abcf641e)> and <[ff3b8da](https://

Re: [I] Status of testing Providers that were prepared on December 23, 2023 [airflow]

2023-12-23 Thread via GitHub
potiuk commented on issue #36384: URL: https://github.com/apache/airflow/issues/36384#issuecomment-1868299418 As you can see - for example - in the past wave we had to withdraw google and databricks, so while we are releasing new changes in other providers, in this wave the google and datab

Re: [PR] Add `on_skipped_callback` in to `BaseOperator` [airflow]

2023-12-23 Thread via GitHub
eladkal commented on PR #36374: URL: https://github.com/apache/airflow/pull/36374#issuecomment-1868301645 > I have seen a scenario where the skipped state is set when branching doesn't follow some of the downstream tasks. Looks like that's not covered. I'm not sure if we can/should ha

Re: [PR] Prevent templated field logic checks in __init__ of operators automatically [airflow]

2023-12-23 Thread via GitHub
eladkal commented on PR #33786: URL: https://github.com/apache/airflow/pull/33786#issuecomment-1868306737 @shahar1 I see that the pre-commit found some invalid usages which is awesome. I suggest to handle them in separated PR(s) then once all merged you can rebase this one and it will be

Re: [PR] Prevent templated field logic checks in __init__ of operators automatically [airflow]

2023-12-23 Thread via GitHub
shahar1 commented on PR #33786: URL: https://github.com/apache/airflow/pull/33786#issuecomment-1868306978 > @shahar1 I see that the pre-commit found some invalid usages which is awesome. > I suggest to handle them in separated PR(s) then once all merged you can rebase this one and it wil

Re: [PR] Prevent templated field logic checks in __init__ of operators automatically [airflow]

2023-12-23 Thread via GitHub
potiuk commented on PR #33786: URL: https://github.com/apache/airflow/pull/33786#issuecomment-1868308640 > @shahar1 I see that the pre-commit found some invalid usages which is awesome. I suggest to handle them in separated PR(s) then once all merged you can rebase this one and it wi

Re: [PR] Add `on_skipped_callback` in to `BaseOperator` [airflow]

2023-12-23 Thread via GitHub
romsharon98 commented on PR #36374: URL: https://github.com/apache/airflow/pull/36374#issuecomment-1868309042 > The trigger rule dep could change the TI state to skipped without raising this exception, I wonder if we should handle that case too 🤔 > > https://github.com/apache/airflow

(airflow) branch pendulum-3 updated (edd2d32d36 -> ccb2e4e76e)

2023-12-23 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis pushed a change to branch pendulum-3 in repository https://gitbox.apache.org/repos/asf/airflow.git omit edd2d32d36 Add pendulum 3 test desiarilisation pendulum 2 omit 5681d12af9 Add support of Pendulum 3 a

Re: [I] AWS Provder Operators and Sensors Duplicate Code [airflow]

2023-12-23 Thread via GitHub
Taragolis commented on issue #36386: URL: https://github.com/apache/airflow/issues/36386#issuecomment-1868311278 Is it some how related to the https://github.com/apache/airflow/issues/35278 ? -- This is an automated message from the Apache Git Service. To respond to the message, please lo

Re: [PR] feat(EcsRunTaskOperator): add `count` param [airflow]

2023-12-23 Thread via GitHub
Taragolis commented on code in PR #36383: URL: https://github.com/apache/airflow/pull/36383#discussion_r1435628529 ## airflow/providers/amazon/aws/operators/ecs.py: ## @@ -635,6 +639,8 @@ def _start_task(self): run_opts["tags"] = [{"key": k, "value": v} for (k, v) i

(airflow) branch postgres-default-quarantined created (now 0fced6ba60)

2023-12-23 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis pushed a change to branch postgres-default-quarantined in repository https://gitbox.apache.org/repos/asf/airflow.git at 0fced6ba60 Use default Postgres version for Quarantined tests This branch includes the foll

(airflow) 01/01: Use default Postgres version for Quarantined tests

2023-12-23 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis pushed a commit to branch postgres-default-quarantined in repository https://gitbox.apache.org/repos/asf/airflow.git commit 0fced6ba602a73e5cc78b190069cb78dc855ee84 Author: Andrey Anshin AuthorDate: Sat Dec 23 19:27:49

Re: [PR] Add parameter confirm to GCSToSFTPOperator [airflow]

2023-12-23 Thread via GitHub
pankajastro commented on code in PR #35704: URL: https://github.com/apache/airflow/pull/35704#discussion_r1435631781 ## airflow/providers/google/cloud/transfers/gcs_to_sftp.py: ## @@ -84,6 +84,7 @@ class GCSToSFTPOperator(BaseOperator): :param gcp_conn_id: (Optional) The co

[PR] Use default Postgres version for Quarantined tests [airflow]

2023-12-23 Thread via GitHub
Taragolis opened a new pull request, #36390: URL: https://github.com/apache/airflow/pull/36390 Instead of hardcoded `11` use default version --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob

Re: [PR] feat(EcsRunTaskOperator): add `count` param [airflow]

2023-12-23 Thread via GitHub
jayceslesar commented on PR #36383: URL: https://github.com/apache/airflow/pull/36383#issuecomment-1868316538 @Taragolis thanks for the feedback! I see what you mean that it will need to be reworked. A specific use case I can relate to this feature is long-running ECS tasks that are running

[PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk opened a new pull request, #36391: URL: https://github.com/apache/airflow/pull/36391 This PR adds possibility of marking the provider as "not ready" in the provider.yaml (by setting optional field as "not-ready" to `true". Setting provider as "not-ready", removes it by default

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on PR #36391: URL: https://github.com/apache/airflow/pull/36391#issuecomment-1868317653 CC: @bolkedebruin - respnding to some of the needs we had in the past. In the future things like "common.io" where the API is not ready to be released yet could be marked as "not-ready

(airflow) branch pendulum-3 updated (ccb2e4e76e -> 59e997338f)

2023-12-23 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis pushed a change to branch pendulum-3 in repository https://gitbox.apache.org/repos/asf/airflow.git from ccb2e4e76e Add backcompat to pendulum 2 add 59e997338f Fix _run_test_command input No new revisions were

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on PR #36391: URL: https://github.com/apache/airflow/pull/36391#issuecomment-1868320927 I also added a chapter in the Provider release documentation about all possible states of provider packages. Now we have 4: * default * not-ready * suspended * removed

Re: [I] Status of testing Providers that were prepared on December 23, 2023 [airflow]

2023-12-23 Thread via GitHub
guillaumeblaquiere commented on issue #36384: URL: https://github.com/apache/airflow/issues/36384#issuecomment-1868322265 Ok for #36133 -- 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 specifi

Re: [PR] Update release process for providers to include mixed RC versions [airflow]

2023-12-23 Thread via GitHub
potiuk commented on PR #36385: URL: https://github.com/apache/airflow/pull/36385#issuecomment-1868322365 BTW. This is just intermediate step. After #36391 and this one, I will add one more change where I will automatically bump RC versions for providers that need RCN. This will make our pro

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
Taragolis commented on PR #36391: URL: https://github.com/apache/airflow/pull/36391#issuecomment-1868322734 > And I thought it might be worthwile to capture the differences between those 4 states in a single place. Release documentation seems like a good place for it. I think it also

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on PR #36391: URL: https://github.com/apache/airflow/pull/36391#issuecomment-1868323608 > I think it also would be nice to split into different sections list of providers as follow up in https://airflow.apache.org/docs/#providers-packagesdocsapache-airflow-providersindexhtm

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
Taragolis commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435642835 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false

[PR] Add note for release manager about updating the states of providers [airflow]

2023-12-23 Thread via GitHub
potiuk opened a new pull request, #36392: URL: https://github.com/apache/airflow/pull/36392 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)*

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435643382 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false +

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435643528 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false +

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
hussein-awala commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435643507 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: f

Re: [PR] Use pre-release version of pendulum in CI and breeze [airflow]

2023-12-23 Thread via GitHub
Taragolis commented on PR #35798: URL: https://github.com/apache/airflow/pull/35798#issuecomment-1868326529 closed in favor of: - https://github.com/apache/airflow/pull/36281 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Use pre-release version of pendulum in CI and breeze [airflow]

2023-12-23 Thread via GitHub
Taragolis closed pull request #35798: Use pre-release version of pendulum in CI and breeze URL: https://github.com/apache/airflow/pull/35798 -- 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 speci

(airflow) branch pendulum-prerelease deleted (was f106e4e4d0)

2023-12-23 Thread taragolis
This is an automated email from the ASF dual-hosted git repository. taragolis pushed a change to branch pendulum-prerelease in repository https://gitbox.apache.org/repos/asf/airflow.git was f106e4e4d0 Revert internal utc timzone to FixedTimezone type The revisions that were on this branch

(airflow-site) branch providers-can-be-in-different-states created (now cb91ff1cd1)

2023-12-23 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch providers-can-be-in-different-states in repository https://gitbox.apache.org/repos/asf/airflow-site.git at cb91ff1cd1 Update provider state information to airflow site This branch includes

(airflow-site) 01/01: Update provider state information to airflow site

2023-12-23 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch providers-can-be-in-different-states in repository https://gitbox.apache.org/repos/asf/airflow-site.git commit cb91ff1cd17dcfdcde8b744a52b36d0fb125963c Author: Jarek Potiuk AuthorDate: Sat Dec 23

[PR] Update provider state information to airflow site [airflow-site]

2023-12-23 Thread via GitHub
potiuk opened a new pull request, #918: URL: https://github.com/apache/airflow-site/pull/918 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-

Re: [PR] Add note for release manager about updating the states of providers [airflow]

2023-12-23 Thread via GitHub
potiuk commented on PR #36392: URL: https://github.com/apache/airflow/pull/36392#issuecomment-1868327187 And update to our airflow-site to add the split: https://github.com/apache/airflow-site/pull/918 I will merge it when we remove plexus and apache.sqoop. -- This is an automated

Re: [I] Status of testing Providers that were prepared on December 23, 2023 [airflow]

2023-12-23 Thread via GitHub
shohamy7 commented on issue #36384: URL: https://github.com/apache/airflow/issues/36384#issuecomment-1868327063 I've tested #36241, all 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

(airflow-site) branch providers-can-be-in-different-states updated (cb91ff1cd1 -> bddbeaced7)

2023-12-23 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch providers-can-be-in-different-states in repository https://gitbox.apache.org/repos/asf/airflow-site.git discard cb91ff1cd1 Update provider state information to airflow site add bddbeaced7 U

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
Taragolis commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435642835 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
Taragolis commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435642835 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false

Re: [PR] Support git-sync v4 in Helm Chart [airflow]

2023-12-23 Thread via GitHub
shohamy7 commented on code in PR #34731: URL: https://github.com/apache/airflow/pull/34731#discussion_r1435648802 ## chart/templates/_helpers.yaml: ## @@ -202,50 +202,88 @@ If release name contains chart name it will be used as a full name. {{- if .Values.dags.gitSync.sshK

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435648862 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false +

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435649438 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false +

Re: [PR] Add note for release manager about updating the states of providers [airflow]

2023-12-23 Thread via GitHub
potiuk merged PR #36392: URL: https://github.com/apache/airflow/pull/36392 -- 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

(airflow) branch main updated (5ddd67a9a6 -> d172ca933a)

2023-12-23 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from 5ddd67a9a6 Fix --use-airflow-version constraints (#36378) add d172ca933a Add note for release manager about updati

Re: [PR] Support git-sync v4 in Helm Chart [airflow]

2023-12-23 Thread via GitHub
shohamy7 commented on code in PR #34731: URL: https://github.com/apache/airflow/pull/34731#discussion_r1435649775 ## chart/templates/_helpers.yaml: ## @@ -202,50 +202,88 @@ If release name contains chart name it will be used as a full name. {{- if .Values.dags.gitSync.sshK

Re: [PR] Update release process for providers to include mixed RC versions [airflow]

2023-12-23 Thread via GitHub
potiuk merged PR #36385: URL: https://github.com/apache/airflow/pull/36385 -- 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

(airflow) branch main updated: Update release process for providers to include mixed RC versions (#36385)

2023-12-23 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 4deed64132 Update release process for providers to

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435650685 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false +

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435653026 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false +

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435653981 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false +

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435653981 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false +

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435654459 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false +

[PR] Use base aws classes in Amazon ECS Operators/Sensors/Triggers [airflow]

2023-12-23 Thread via GitHub
jayceslesar opened a new pull request, #36393: URL: https://github.com/apache/airflow/pull/36393 Similar to https://github.com/apache/airflow/pull/35133 (relates to https://github.com/apache/airflow/issues/35278) Unsure how to get tests passing for `test_hook_and_client` in `test

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk commented on code in PR #36391: URL: https://github.com/apache/airflow/pull/36391#discussion_r1435654902 ## airflow/providers/fab/provider.yaml: ## @@ -25,6 +25,11 @@ description: | `Flask App Builder `__ suspended: false +

Re: [I] Removal of project specific privacy page [airflow-site]

2023-12-23 Thread via GitHub
potiuk commented on issue #917: URL: https://github.com/apache/airflow-site/issues/917#issuecomment-1868333951 Yep. It is indeed. Thanks for notifying us. -- 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

[PR] Link to Apache Software Foundation Privacy Policy. [airflow]

2023-12-23 Thread via GitHub
potiuk opened a new pull request, #36394: URL: https://github.com/apache/airflow/pull/36394 Since we are following the privacy Policy of the ASF, we can link to the policyu (and remove out-dated information about Google Analytics). Fixes: https://github.com/apache/airflow-site/issues/

(airflow-site) 01/01: Remove the privacy policy page with link to the ASF one

2023-12-23 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch update-link-to-privacy-policy in repository https://gitbox.apache.org/repos/asf/airflow-site.git commit 78f360ec81cd9ab75b8c93e1b7e71853cae84ee4 Author: Jarek Potiuk AuthorDate: Sat Dec 23 18:24:

(airflow-site) branch update-link-to-privacy-policy created (now 78f360ec81)

2023-12-23 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch update-link-to-privacy-policy in repository https://gitbox.apache.org/repos/asf/airflow-site.git at 78f360ec81 Remove the privacy policy page with link to the ASF one This branch includes

[PR] Remove the privacy policy page with link to the ASF one [airflow-site]

2023-12-23 Thread via GitHub
potiuk opened a new pull request, #919: URL: https://github.com/apache/airflow-site/pull/919 Since we are using Matomo and following the ASF policy we should remove the out-dated information about the Google Analytics and replace it with link to the ASF policy. Fixes: #917 -- This

Re: [I] Removal of project specific privacy page [airflow-site]

2023-12-23 Thread via GitHub
potiuk commented on issue #917: URL: https://github.com/apache/airflow-site/issues/917#issuecomment-1868336321 Will be fixed in https://github.com/apache/airflow-site/pull/919 and in the airflow content https://github.com/apache/airflow/pull/36394 when we release Airflow next time (it's lin

Re: [PR] Update provider state information to airflow site [airflow-site]

2023-12-23 Thread via GitHub
potiuk merged PR #918: URL: https://github.com/apache/airflow-site/pull/918 -- 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.

(airflow-site) branch main updated: Update provider state information to airflow site (#918)

2023-12-23 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow-site.git The following commit(s) were added to refs/heads/main by this push: new 35c5253b23 Update provider state information t

Re: [PR] Remove the privacy policy page with link to the ASF one [airflow-site]

2023-12-23 Thread via GitHub
potiuk merged PR #919: URL: https://github.com/apache/airflow-site/pull/919 -- 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] Removal of project specific privacy page [airflow-site]

2023-12-23 Thread via GitHub
potiuk closed issue #917: Removal of project specific privacy page URL: https://github.com/apache/airflow-site/issues/917 -- 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

(airflow-site) branch main updated: Remove the privacy policy page with link to the ASF one (#919)

2023-12-23 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow-site.git The following commit(s) were added to refs/heads/main by this push: new f8a3fb8760 Remove the privacy policy page with

Re: [PR] Make databricks sql hook return a namedtuple [airflow]

2023-12-23 Thread via GitHub
bolkedebruin commented on PR #36205: URL: https://github.com/apache/airflow/pull/36205#issuecomment-1868339037 Sorry @potiuk i did actually squash and merge but adjusted the commit message to cover what was done. That removed the ref. I'll be more careful -- This is an automated message f

(airflow-site) branch gh-pages updated (ccfde9f054 -> b71db33283)

2023-12-23 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch gh-pages in repository https://gitbox.apache.org/repos/asf/airflow-site.git discard ccfde9f054 Rewritten history to remove past gh-pages deployments new b71db33283 Rewritten history to

Re: [PR] Use default Postgres version for Quarantined tests [airflow]

2023-12-23 Thread via GitHub
eladkal merged PR #36390: URL: https://github.com/apache/airflow/pull/36390 -- 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.

(airflow) branch main updated: Use default Postgres version for Quarantined tests (#36390)

2023-12-23 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 8e70d56024 Use default Postgres version for Quaran

(airflow) branch postgres-default-quarantined deleted (was 0fced6ba60)

2023-12-23 Thread eladkal
This is an automated email from the ASF dual-hosted git repository. eladkal pushed a change to branch postgres-default-quarantined in repository https://gitbox.apache.org/repos/asf/airflow.git was 0fced6ba60 Use default Postgres version for Quarantined tests The revisions that were on this

[I] When accessing the DAG page through the K3S proxy, an error of "404 page not found" is displayed [airflow]

2023-12-23 Thread via GitHub
microhuang opened a new issue, #36395: URL: https://github.com/apache/airflow/issues/36395 ### Official Helm Chart version 1.11.0 (latest released) ### Apache Airflow version 2.7.1 ### Kubernetes Version v1.26.4 +k3s1 ### Helm Chart configuration

(airflow-site) branch gh-pages updated (b71db33283 -> ae6c8c1ef7)

2023-12-23 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch gh-pages in repository https://gitbox.apache.org/repos/asf/airflow-site.git discard b71db33283 Rewritten history to remove past gh-pages deployments new ae6c8c1ef7 Rewritten history to

[PR] Fix the required access for get_variable endpoint [airflow]

2023-12-23 Thread via GitHub
hussein-awala opened a new pull request, #36396: URL: https://github.com/apache/airflow/pull/36396 (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 unsubscri

Re: [PR] Add feture of "not-ready" provider. [airflow]

2023-12-23 Thread via GitHub
potiuk merged PR #36391: URL: https://github.com/apache/airflow/pull/36391 -- 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

(airflow) branch main updated: Add feture of "not-ready" provider. (#36391)

2023-12-23 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 341d5b747d Add feture of "not-ready" provider. (#36

Re: [PR] Use base aws classes in Amazon ECS Operators/Sensors/Triggers [airflow]

2023-12-23 Thread via GitHub
Taragolis commented on code in PR #36393: URL: https://github.com/apache/airflow/pull/36393#discussion_r1435669740 ## airflow/providers/amazon/aws/operators/ecs.py: ## @@ -45,21 +46,18 @@ from airflow.models import TaskInstance from airflow.utils.context import Context

(airflow) branch main updated (341d5b747d -> b30a1b854c)

2023-12-23 Thread husseinawala
This is an automated email from the ASF dual-hosted git repository. husseinawala pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from 341d5b747d Add feture of "not-ready" provider. (#36391) add b30a1b854c Link to Apache Software Foundation Pr

Re: [PR] Add support of Pendulum 3 [airflow]

2023-12-23 Thread via GitHub
Taragolis commented on PR #36281: URL: https://github.com/apache/airflow/pull/36281#issuecomment-1868351118 Seems like it close to backcompat with 2, just need to resolve some static checks -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Link to Apache Software Foundation Privacy Policy. [airflow]

2023-12-23 Thread via GitHub
hussein-awala merged PR #36394: URL: https://github.com/apache/airflow/pull/36394 -- 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] Add libxml2 install command for macOS in contributor quick start doc [airflow]

2023-12-23 Thread via GitHub
hussein-awala merged PR #36388: URL: https://github.com/apache/airflow/pull/36388 -- 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] Add libxml2 install command for macOS in contributor quick start doc [airflow]

2023-12-23 Thread via GitHub
boring-cyborg[bot] commented on PR #36388: URL: https://github.com/apache/airflow/pull/36388#issuecomment-1868351371 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.

(airflow) branch main updated: Add libxml2 install command for macOS in contributor quick start doc (#36388)

2023-12-23 Thread husseinawala
This is an automated email from the ASF dual-hosted git repository. husseinawala 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 e6b4f7d8c0 Add libxml2 install command for ma

Re: [PR] Add libxml2 install command for macOS in contributor quick start doc [airflow]

2023-12-23 Thread via GitHub
hussein-awala commented on PR #36388: URL: https://github.com/apache/airflow/pull/36388#issuecomment-1868351431 Congrats on your first commit 🎉 -- 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 th

Re: [PR] Make databricks sql hook return a namedtuple [airflow]

2023-12-23 Thread via GitHub
potiuk commented on PR #36205: URL: https://github.com/apache/airflow/pull/36205#issuecomment-1868353278 No worries :). I caught it luckily -- 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 sp

Re: [I] Status of testing Providers that were prepared on December 23, 2023 [airflow]

2023-12-23 Thread via GitHub
spencertollefson commented on issue #36384: URL: https://github.com/apache/airflow/issues/36384#issuecomment-1868353719 I tested #36273. All 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

  1   2   >