[GitHub] [airflow] eladkal commented on pull request #24660: SnowflakeToSlackOperator: Use Slack API and send result as file

2022-06-25 Thread GitBox
eladkal commented on PR #24660: URL: https://github.com/apache/airflow/pull/24660#issuecomment-1166424561 @Taragolis note that we also have PrestoToSlackOperator https://github.com/apache/airflow/pull/23979 So this change should also applied there. @alexkruc FYI since you are

[GitHub] [airflow] github-actions[bot] commented on issue #22035: In 'list users', clicking 'show record' button returns an error page

2022-06-25 Thread GitBox
github-actions[bot] commented on issue #22035: URL: https://github.com/apache/airflow/issues/22035#issuecomment-1166383988 This issue has been automatically marked as stale because it has been open for 30 days with no response from the author. It will be closed in next 7 days if no further

[GitHub] [airflow] github-actions[bot] commented on issue #23520: Duplicate task instances starts if scheduler restarts in middle of execution of task

2022-06-25 Thread GitBox
github-actions[bot] commented on issue #23520: URL: https://github.com/apache/airflow/issues/23520#issuecomment-1166383981 This issue has been automatically marked as stale because it has been open for 30 days with no response from the author. It will be closed in next 7 days if no further

[GitHub] [airflow] github-actions[bot] commented on pull request #23568: Fix defaultAirflowRepository comment

2022-06-25 Thread GitBox
github-actions[bot] commented on PR #23568: URL: https://github.com/apache/airflow/pull/23568#issuecomment-1166383972 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

[airflow] branch constraints-main updated: Updating constraints. Build id:2562023364

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

[GitHub] [airflow] Taragolis opened a new pull request, #24662: Add test_connection method to AWS hook

2022-06-25 Thread GitBox
Taragolis opened a new pull request, #24662: URL: https://github.com/apache/airflow/pull/24662 For test AWS Credentials I move validation `client_type` and `resource_type` to conn property. General usage of AWS Hooks use `conn` property or `get_conn()` method. @ferruzzi @o-nikolas

[airflow] branch main updated: Use target commit SHA for build image workflow (#24659)

2022-06-25 Thread kamilbregula
This is an automated email from the ASF dual-hosted git repository. kamilbregula 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 2ffaebee25 Use target commit SHA for build

[GitHub] [airflow] mik-laj merged pull request #24659: Use target commit SHA for build image workflow

2022-06-25 Thread GitBox
mik-laj merged PR #24659: URL: https://github.com/apache/airflow/pull/24659 -- 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:

[GitHub] [airflow] potiuk commented on pull request #24661: Add ARM image building for regular PRs

2022-06-25 Thread GitBox
potiuk commented on PR #24661: URL: https://github.com/apache/airflow/pull/24661#issuecomment-1166356156 cc: @gmsantos -- 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

[GitHub] [airflow] potiuk opened a new pull request, #24661: Add ARM image building for regular PRs

2022-06-25 Thread GitBox
potiuk opened a new pull request, #24661: URL: https://github.com/apache/airflow/pull/24661 The image building for ARM is currently only done in the main build only to refresh cache, however there are sometimes cases when new dependency (for example #24635) broke ARM image build and it

[GitHub] [airflow] sfc-gh-madkins commented on issue #24456: Add Snowpark Operator that allows execution of Snowflake Snowpark code

2022-06-25 Thread GitBox
sfc-gh-madkins commented on issue #24456: URL: https://github.com/apache/airflow/issues/24456#issuecomment-1166354162 I pulled the PR -- any advice on how to let me test locally? Doesnt appear to be as easy as pip install -e On Sat, Jun 25, 2022 at 8:24 AM Kamil BreguĊ‚a

[GitHub] [airflow] dstandish commented on a diff in pull request #24658: Add `airflow_kpo_in_cluster` label to KPO pods

2022-06-25 Thread GitBox
dstandish commented on code in PR #24658: URL: https://github.com/apache/airflow/pull/24658#discussion_r906718644 ## airflow/providers/cncf/kubernetes/hooks/kubernetes.py: ## @@ -127,6 +127,9 @@ def __init__( self.disable_verify_ssl = disable_verify_ssl

[GitHub] [airflow] Taragolis opened a new pull request, #24660: SnowflakeToSlackOperator: Use Slack API and send result as file

2022-06-25 Thread GitBox
Taragolis opened a new pull request, #24660: URL: https://github.com/apache/airflow/pull/24660 closes: #9145 -- 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

[GitHub] [airflow] potiuk commented on a diff in pull request #24591: Rewrite the 'version' command to use Click

2022-06-25 Thread GitBox
potiuk commented on code in PR #24591: URL: https://github.com/apache/airflow/pull/24591#discussion_r906717837 ## airflow/cli/__main__.py: ## @@ -18,6 +18,7 @@ # under the License. from airflow.cli import airflow_cmd +from airflow.cli.commands import version # noqa: F401

[GitHub] [airflow] potiuk commented on a diff in pull request #24591: Rewrite the 'version' command to use Click

2022-06-25 Thread GitBox
potiuk commented on code in PR #24591: URL: https://github.com/apache/airflow/pull/24591#discussion_r906717837 ## airflow/cli/__main__.py: ## @@ -18,6 +18,7 @@ # under the License. from airflow.cli import airflow_cmd +from airflow.cli.commands import version # noqa: F401

[GitHub] [airflow] potiuk commented on a diff in pull request #24591: Rewrite the 'version' command to use Click

2022-06-25 Thread GitBox
potiuk commented on code in PR #24591: URL: https://github.com/apache/airflow/pull/24591#discussion_r906716931 ## airflow/cli/commands/version.py: ## @@ -0,0 +1,28 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See

[airflow] branch main updated: Change postgres auth method (#24640)

2022-06-25 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 20ca9c21d3 Change postgres auth method (#24640)

[GitHub] [airflow] potiuk merged pull request #24640: Change Postgres auth method in docker-compose

2022-06-25 Thread GitBox
potiuk merged PR #24640: URL: https://github.com/apache/airflow/pull/24640 -- 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:

[GitHub] [airflow] potiuk commented on pull request #24657: Fix behaviour of build/pull after recent Breeze changes

2022-06-25 Thread GitBox
potiuk commented on PR #24657: URL: https://github.com/apache/airflow/pull/24657#issuecomment-1166346859 Nees #24659 to succeed. -- 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

[GitHub] [airflow] potiuk commented on pull request #24656: Handle "workflow_run" event properly in selective-check

2022-06-25 Thread GitBox
potiuk commented on PR #24656: URL: https://github.com/apache/airflow/pull/24656#issuecomment-1166346763 Needs #24659 merged to succeed -- 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

[GitHub] [airflow] potiuk opened a new pull request, #24659: Use target commit SHA for build image workflow

2022-06-25 Thread GitBox
potiuk opened a new pull request, #24659: URL: https://github.com/apache/airflow/pull/24659 The build-image workflow should use TARGET_COMMIT_SHA as the selective check COMMIT_REF otherwise it might not build image when needed --- **^ Add meaningful description

[GitHub] [airflow] jedcunningham opened a new pull request, #24658: Add `airflow_kpo_in_cluster` label to KPO pods

2022-06-25 Thread GitBox
jedcunningham opened a new pull request, #24658: URL: https://github.com/apache/airflow/pull/24658 This allows one to determine if the pod was created with in_cluster config or not, both on the k8s side and in pod_mutation_hooks. -- This is an automated message from the Apache Git

[GitHub] [airflow] jayarjo commented on issue #22328: bigquery provider's - BigQueryCursor missing implementation for description property.

2022-06-25 Thread GitBox
jayarjo commented on issue #22328: URL: https://github.com/apache/airflow/issues/22328#issuecomment-1166332391 Any update 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 above to go to the specific

[airflow] branch constraints-main updated: Updating constraints. Build id:2561161234

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

[GitHub] [airflow] potiuk commented on a diff in pull request #24657: Fix behaviour of build/pull after recent Breeze changes

2022-06-25 Thread GitBox
potiuk commented on code in PR #24657: URL: https://github.com/apache/airflow/pull/24657#discussion_r906702039 ## dev/breeze/src/airflow_breeze/commands/ci_image_commands.py: ## @@ -582,8 +589,9 @@ def rebuild_or_pull_ci_image_if_needed(

[GitHub] [airflow] potiuk commented on a diff in pull request #24657: Fix behaviour of build/pull after recent Breeze changes

2022-06-25 Thread GitBox
potiuk commented on code in PR #24657: URL: https://github.com/apache/airflow/pull/24657#discussion_r906701977 ## dev/breeze/src/airflow_breeze/commands/ci_image_commands.py: ## @@ -605,4 +613,5 @@ def rebuild_or_pull_ci_image_if_needed( 'Forcing build.[/]'

[GitHub] [airflow] potiuk opened a new pull request, #24657: Fix behaviour of build/pull after recent Breeze changes

2022-06-25 Thread GitBox
potiuk opened a new pull request, #24657: URL: https://github.com/apache/airflow/pull/24657 The behaviour of Breeze after some recent changes related to pulling and building images in parallel have been slightly broken. Nothing serious but slightly annoying behaviour: * when

[airflow] branch main updated: Migrate Google example DAG bigquery_transfer to new design AIP-47 (#24543)

2022-06-25 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 abb304c92f Migrate Google example DAG

[GitHub] [airflow] potiuk merged pull request #24543: Migrate Google example DAG bigquery_transfer to new design AIP-47

2022-06-25 Thread GitBox
potiuk merged PR #24543: URL: https://github.com/apache/airflow/pull/24543 -- 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:

[GitHub] [airflow] potiuk opened a new pull request, #24656: Handle "workflow_run" event properly in selective-check

2022-06-25 Thread GitBox
potiuk opened a new pull request, #24656: URL: https://github.com/apache/airflow/pull/24656 Unfortunately testing workflow_run is a bit difficult because the changes are only effective after merging them. Fixing (hopefully) yet another mistake in the workflow run where commit

[GitHub] [airflow] potiuk merged pull request #24655: Remove selective checks from the "release workflow"

2022-06-25 Thread GitBox
potiuk merged PR #24655: URL: https://github.com/apache/airflow/pull/24655 -- 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: Remove selective checks from the "release workflow" (#24655)

2022-06-25 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 47f54b6099 Remove selective checks from the

[GitHub] [airflow] potiuk commented on pull request #22910: Add impersonation_chain parameter for ComputeEngineSSHHook

2022-06-25 Thread GitBox
potiuk commented on PR #22910: URL: https://github.com/apache/airflow/pull/22910#issuecomment-1166312484 > Basically the problem is i cant use an Ubuntu configuration with enough performances to run the test environment. I'm stuck on a low hardware windows 10 laptop and i tried ubuntu in

[GitHub] [airflow] potiuk opened a new pull request, #24655: Remove selective checks from the "release workflow"

2022-06-25 Thread GitBox
potiuk opened a new pull request, #24655: URL: https://github.com/apache/airflow/pull/24655 Missed that one too :( --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] potiuk commented on pull request #24654: Script to filter candidates for PR of the month based on heuristics

2022-06-25 Thread GitBox
potiuk commented on PR #24654: URL: https://github.com/apache/airflow/pull/24654#issuecomment-1166298994 Top 10 PRs: * Score: 155.50: PR #24284 by @ashb: `Speed up grid_data endpoint by 10x. Merged at 2022-06-15 12:02:23: https://github.com/apache/airflow/pull/24284 * Score: 135.00:

[GitHub] [airflow] potiuk opened a new pull request, #24654: Script to filter candidates for PR of the month based on heuristics

2022-06-25 Thread GitBox
potiuk opened a new pull request, #24654: URL: https://github.com/apache/airflow/pull/24654 This scripts proposes top candidates for PR of the month based on simple heuristics as discussed in the document

[GitHub] [airflow] boring-cyborg[bot] commented on issue #24653: Mapped TriggerDagRunOperator causes SerializationError due to operator_extra_links 'property' object is not iterable

2022-06-25 Thread GitBox
boring-cyborg[bot] commented on issue #24653: URL: https://github.com/apache/airflow/issues/24653#issuecomment-1166296267 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] consciencee opened a new issue, #24653: Mapped TriggerDagRunOperator causes SerializationError due to operator_extra_links 'property' object is not iterable

2022-06-25 Thread GitBox
consciencee opened a new issue, #24653: URL: https://github.com/apache/airflow/issues/24653 ### Apache Airflow version 2.3.2 (latest released) ### What happened Hi, I have a kind of issue with launching several subDags via mapping TriggerDagRunOperator (mapping over

[GitHub] [airflow] mik-laj commented on issue #24456: Add Snowpark Operator that allows execution of Snowflake Snowpark code

2022-06-25 Thread GitBox
mik-laj commented on issue #24456: URL: https://github.com/apache/airflow/issues/24456#issuecomment-1166284621 I prepared a draft PR: https://github.com/apache/airflow/pull/24652 The implementation is complete, but I will want to add some documentation to improve the adoption of this

[GitHub] [airflow] mik-laj opened a new pull request, #24652: Add @task.snowpark decorator

2022-06-25 Thread GitBox
mik-laj opened a new pull request, #24652: URL: https://github.com/apache/airflow/pull/24652 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more

[airflow] branch constraints-main updated: Updating constraints. Build id:2560177236

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

[airflow] branch main updated: Remove misleading message from CI (#24650)

2022-06-25 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 ac104025f4 Remove misleading message from CI

[GitHub] [airflow] potiuk merged pull request #24650: Remove misleading message from CI

2022-06-25 Thread GitBox
potiuk merged PR #24650: URL: https://github.com/apache/airflow/pull/24650 -- 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 (2703874eb2 -> aa8cd30c46)

2022-06-25 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 2703874eb2 Switch to new selective-checks in label-when-reviewed workflow (#24651) add aa8cd30c46 Cleanup

[GitHub] [airflow] potiuk commented on pull request #24650: Remove misleading message from CI

2022-06-25 Thread GitBox
potiuk commented on PR #24650: URL: https://github.com/apache/airflow/pull/24650#issuecomment-1166239768 Same here "selective checks" failing is fixed. -- 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

[GitHub] [airflow] potiuk merged pull request #24649: Cleanup references to selective checks

2022-06-25 Thread GitBox
potiuk merged PR #24649: URL: https://github.com/apache/airflow/pull/24649 -- 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:

[GitHub] [airflow] potiuk commented on pull request #24649: Cleanup references to selective checks

2022-06-25 Thread GitBox
potiuk commented on PR #24649: URL: https://github.com/apache/airflow/pull/24649#issuecomment-1166239698 MErging. Failung "label when reviewed" 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

[airflow] branch main updated: Switch to new selective-checks in label-when-reviewed workflow (#24651)

2022-06-25 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 2703874eb2 Switch to new selective-checks in

[GitHub] [airflow] potiuk merged pull request #24651: Switch to new selective-checks in label-when-reviewed workflow

2022-06-25 Thread GitBox
potiuk merged PR #24651: URL: https://github.com/apache/airflow/pull/24651 -- 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:

[GitHub] [airflow] potiuk opened a new pull request, #24651: Switch to new selective-checks in label-when-reviewed workflow

2022-06-25 Thread GitBox
potiuk opened a new pull request, #24651: URL: https://github.com/apache/airflow/pull/24651 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more

[GitHub] [airflow] potiuk commented on pull request #24650: Remove misleading message from CI

2022-06-25 Thread GitBox
potiuk commented on PR #24650: URL: https://github.com/apache/airflow/pull/24650#issuecomment-1166228403 Gets rid of those: https://user-images.githubusercontent.com/595491/175765468-55e61c0e-e774-42a6-bb65-42a6c0bc3df4.png;> -- This is an automated message from the Apache

[GitHub] [airflow] potiuk opened a new pull request, #24650: Remove misleading message from CI

2022-06-25 Thread GitBox
potiuk opened a new pull request, #24650: URL: https://github.com/apache/airflow/pull/24650 The CI jobs do not usually have mypy_cache volume created and almost all jobs print misleading "mypy_cache_volume" error at the beginning. This is a noise - we are not interested in this

[GitHub] [airflow] potiuk commented on issue #12282: Rewrite Breeze in Python

2022-06-25 Thread GitBox
potiuk commented on issue #12282: URL: https://github.com/apache/airflow/issues/12282#issuecomment-1166226794 Some status with so-far bash remoal for Breeze: Before: ``` python: 288754 (96.35%) sh: 10950 (3.65%) ``` now: ``` python:

[GitHub] [airflow] potiuk opened a new pull request, #24649: Cleanup references to selective checks

2022-06-25 Thread GitBox
potiuk opened a new pull request, #24649: URL: https://github.com/apache/airflow/pull/24649 Selective checks docs have been moved to breeze as part of #24610 but some of the references were still left. This PR cleans it up. --- **^ Add meaningful description

[GitHub] [airflow] potiuk closed issue #19971: CI: Rewrite selective check script in Python

2022-06-25 Thread GitBox
potiuk closed issue #19971: CI: Rewrite selective check script in Python URL: https://github.com/apache/airflow/issues/19971 -- 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

[GitHub] [airflow] potiuk merged pull request #24610: Convert selective checks to Breeze Python

2022-06-25 Thread GitBox
potiuk merged PR #24610: URL: https://github.com/apache/airflow/pull/24610 -- 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:

[GitHub] [airflow] llilyy commented on pull request #24220: Add support for KEDA HPA config to Helm Chart

2022-06-25 Thread GitBox
llilyy commented on PR #24220: URL: https://github.com/apache/airflow/pull/24220#issuecomment-1166213892 Any updates on this PR? -- 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

[GitHub] [airflow] masterlittle commented on issue #20308: Multi Schedulers don't synchronize task instance state when using Triggerer

2022-06-25 Thread GitBox
masterlittle commented on issue #20308: URL: https://github.com/apache/airflow/issues/20308#issuecomment-1166201618 Is this planned to be fixed? This is a major issue and causing failures and operational nightmares -- This is an automated message from the Apache Git Service. To respond