[GitHub] [airflow] NaveenGokavarapu19 opened a new pull request, #24724: Added Support in ts_nodash

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

[GitHub] [airflow] potiuk commented on pull request #24468: Add as_dict and custom query to BigQueryGetDataOperator

2022-06-28 Thread GitBox
potiuk commented on PR #24468: URL: https://github.com/apache/airflow/pull/24468#issuecomment-1169596566 Needs fixes. -- 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] branch main updated: `AirbyteHook` add cancel job option (#24593)

2022-06-28 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 c118b2836f `AirbyteHook` add cancel job option (#24

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #24593: `AirbyteHook` add cancel job option

2022-06-28 Thread GitBox
boring-cyborg[bot] commented on PR #24593: URL: https://github.com/apache/airflow/pull/24593#issuecomment-1169595539 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] potiuk merged pull request #24593: `AirbyteHook` add cancel job option

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

[GitHub] [airflow] potiuk closed issue #24574: `AirbyteHook` add cancel job option

2022-06-28 Thread GitBox
potiuk closed issue #24574: `AirbyteHook` add cancel job option URL: https://github.com/apache/airflow/issues/24574 -- 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

[GitHub] [airflow] potiuk commented on pull request #24593: `AirbyteHook` add cancel job option

2022-06-28 Thread GitBox
potiuk commented on PR #24593: URL: https://github.com/apache/airflow/pull/24593#issuecomment-1169595399 Timeout on sdist building has been fixed in main already. Merging. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [airflow] cswpy commented on pull request #24363: Fixed BigQueryCreateExternalTableOperator and its unit test (#24160)

2022-06-28 Thread GitBox
cswpy commented on PR #24363: URL: https://github.com/apache/airflow/pull/24363#issuecomment-1169578804 @eladkal @potiuk I believe the PR is ready to be merged😀 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] hterik opened a new issue, #24723: Grid view - Show vertical lines at rounded date intervals

2022-06-28 Thread GitBox
hterik opened a new issue, #24723: URL: https://github.com/apache/airflow/issues/24723 ### Description When looking at the DAG grid view, i want a quick overview of how many dags were run today. In grid view now (Airflow 2.3.2), there are some vertical lines inserted on the grid v

[GitHub] [airflow] cswpy commented on pull request #24363: Fixed BigQueryCreateExternalTableOperator and its unit test (#24160)

2022-06-28 Thread GitBox
cswpy commented on PR #24363: URL: https://github.com/apache/airflow/pull/24363#issuecomment-1169551841 @potiuk @eladkal I believe this PR is ready to be merged? Thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [airflow] potiuk commented on pull request #24588: add: container securityContext not available in podSecurityContext

2022-06-28 Thread GitBox
potiuk commented on PR #24588: URL: https://github.com/apache/airflow/pull/24588#issuecomment-1169549440 I'd say that better action would really be to have two separate contexts and maybe simply - rather than adding "containerSecurityContext", introduce `values/securityContexts/pod` and `va

[GitHub] [airflow] millin commented on pull request #21942: Improve ElasticsearchTaskHandler

2022-06-28 Thread GitBox
millin commented on PR #21942: URL: https://github.com/apache/airflow/pull/21942#issuecomment-1169542693 I need some time to see what changed in code, will try to resolve conflict in few days -- This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [airflow] potiuk commented on pull request #21942: Improve ElasticsearchTaskHandler

2022-06-28 Thread GitBox
potiuk commented on PR #21942: URL: https://github.com/apache/airflow/pull/21942#issuecomment-1169539557 There is a conflict now to solve. The change looks good with regards to it's "interfacing" with the airlfow core, but I do not know that many details about Elasticsearch to be able to v

[GitHub] [airflow] potiuk commented on pull request #24719: More typing and minor refactor for kubernetes

2022-06-28 Thread GitBox
potiuk commented on PR #24719: URL: https://github.com/apache/airflow/pull/24719#issuecomment-1169517889 Tests need fixing :) -- 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.

[airflow] branch main updated: Use our yaml util in all providers (#24720)

2022-06-28 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 45b11d4ed1 Use our yaml util in all providers (#247

[GitHub] [airflow] potiuk merged pull request #24720: Use our yaml util in all providers

2022-06-28 Thread GitBox
potiuk merged PR #24720: URL: https://github.com/apache/airflow/pull/24720 -- 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 (9f58e82332 -> c15cf6e10c)

2022-06-28 Thread bbovenzi
This is an automated email from the ASF dual-hosted git repository. bbovenzi pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from 9f58e82332 Fix migration 0080_2_0_2 - Replace null values before setting column not null (#24585) add c15cf6e10

[GitHub] [airflow] bbovenzi merged pull request #24722: Address all yarn test warnings

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

[GitHub] [airflow] bbovenzi opened a new pull request, #24722: Address all yarn test warnings

2022-06-28 Thread GitBox
bbovenzi opened a new pull request, #24722: URL: https://github.com/apache/airflow/pull/24722 - remove outdated react-hooks library - add missing mock --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/ma

[GitHub] [airflow] ming-zhang-SN opened a new issue, #24721: `default_wrap` setting for webserver is not working

2022-06-28 Thread GitBox
ming-zhang-SN opened a new issue, #24721: URL: https://github.com/apache/airflow/issues/24721 ### Apache Airflow version 2.3.2 (latest released) ### What happened The DAG logs are not wrapped even I set `default_wrap` in `airflow.cfg` to `True`. That may cause by this

[airflow] branch main updated: Fix migration 0080_2_0_2 - Replace null values before setting column not null (#24585)

2022-06-28 Thread uranusjr
This is an automated email from the ASF dual-hosted git repository. uranusjr 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 9f58e82332 Fix migration 0080_2_0_2 - Replace nul

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #24585: Fix migration 0080_2_0_2 - Replace null values before setting column not null

2022-06-28 Thread GitBox
boring-cyborg[bot] commented on PR #24585: URL: https://github.com/apache/airflow/pull/24585#issuecomment-1169449720 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] uranusjr closed issue #24566: Migration changes column to NOT NULL without updating NULL data first

2022-06-28 Thread GitBox
uranusjr closed issue #24566: Migration changes column to NOT NULL without updating NULL data first URL: https://github.com/apache/airflow/issues/24566 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go t

[GitHub] [airflow] uranusjr merged pull request #24585: Fix migration 0080_2_0_2 - Replace null values before setting column not null

2022-06-28 Thread GitBox
uranusjr merged PR #24585: URL: https://github.com/apache/airflow/pull/24585 -- 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 constraints-main updated: Updating constraints. Build id:2579621275

2022-06-28 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 5902fa7f53 Updating con

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #24156: Create new databases from the ORM

2022-06-28 Thread GitBox
ephraimbuddy commented on code in PR #24156: URL: https://github.com/apache/airflow/pull/24156#discussion_r909076701 ## airflow/models/renderedtifields.py: ## @@ -46,6 +46,14 @@ class RenderedTaskInstanceFields(Base): k8s_pod_yaml = Column(sqlalchemy_jsonfield.JSONField(jso

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #24156: Create new databases from the ORM

2022-06-28 Thread GitBox
ephraimbuddy commented on code in PR #24156: URL: https://github.com/apache/airflow/pull/24156#discussion_r909076701 ## airflow/models/renderedtifields.py: ## @@ -46,6 +46,14 @@ class RenderedTaskInstanceFields(Base): k8s_pod_yaml = Column(sqlalchemy_jsonfield.JSONField(jso

[GitHub] [airflow] jedcunningham opened a new pull request, #24720: Use our yaml util in all providers

2022-06-28 Thread GitBox
jedcunningham opened a new pull request, #24720: URL: https://github.com/apache/airflow/pull/24720 Our yaml util, which uses libyaml where possible, has been available in core since 2.0.2. Providers now require 2.2.0+, so we can safely use it now. -- This is an automated message from

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #24156: Create new databases from the ORM

2022-06-28 Thread GitBox
ephraimbuddy commented on code in PR #24156: URL: https://github.com/apache/airflow/pull/24156#discussion_r909070746 ## airflow/models/renderedtifields.py: ## @@ -46,6 +46,14 @@ class RenderedTaskInstanceFields(Base): k8s_pod_yaml = Column(sqlalchemy_jsonfield.JSONField(jso

[GitHub] [airflow] jedcunningham commented on a diff in pull request #24719: More typing and minor refactor for kubernetes

2022-06-28 Thread GitBox
jedcunningham commented on code in PR #24719: URL: https://github.com/apache/airflow/pull/24719#discussion_r909069294 ## airflow/providers/cncf/kubernetes/hooks/kubernetes.py: ## @@ -382,12 +383,11 @@ def get_pod_log_stream( :param container: container name :pa

[GitHub] [airflow] jedcunningham opened a new pull request, #24719: More typing and minor refactor for kubernetes

2022-06-28 Thread GitBox
jedcunningham opened a new pull request, #24719: URL: https://github.com/apache/airflow/pull/24719 Just some additional typing for the k8s provider and utils, and a refactor to use the cached client. -- This is an automated message from the Apache Git Service. To respond to the message, p

[airflow] branch main updated: Grid task log, multi select and ts files migration. (#24623)

2022-06-28 Thread bbovenzi
This is an automated email from the ASF dual-hosted git repository. bbovenzi pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 75db755f4b Grid task log, multi select and ts fil

[GitHub] [airflow] bbovenzi merged pull request #24623: Grid task log, multi select and ts files migration.

2022-06-28 Thread GitBox
bbovenzi merged PR #24623: URL: https://github.com/apache/airflow/pull/24623 -- 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: Do not fail requeued TIs (#23846)

2022-06-28 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham 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 66ffe39b0b Do not fail requeued TIs (#23846)

[GitHub] [airflow] jedcunningham closed issue #23824: Race condition between Triggerer and Scheduler

2022-06-28 Thread GitBox
jedcunningham closed issue #23824: Race condition between Triggerer and Scheduler URL: https://github.com/apache/airflow/issues/23824 -- 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 com

[GitHub] [airflow] jedcunningham merged pull request #23846: Do not fail requeued TIs

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

[airflow] branch move-setup-dependencies-to-providers updated (87f5671a60 -> dfda130ec4)

2022-06-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch move-setup-dependencies-to-providers in repository https://gitbox.apache.org/repos/asf/airflow.git discard 87f5671a60 Move provider dependencies to inside provider folders add dfda130ec4 Mo

[airflow] branch move-setup-dependencies-to-providers updated (8c75d701bf -> 87f5671a60)

2022-06-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch move-setup-dependencies-to-providers in repository https://gitbox.apache.org/repos/asf/airflow.git discard 8c75d701bf Move provider dependencies to inside provider folders add 0e1a6b9807 Sc

[airflow] branch main updated (0e1a6b9807 -> 94f4f81efb)

2022-06-28 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 0e1a6b9807 Script to filter candidates for PR of the month based on heuristics (#24654) add 94f4f81efb Merge-frie

[GitHub] [airflow] potiuk merged pull request #24711: Merge-friendly output of command hashes for breeze

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

[GitHub] [airflow] bbovenzi commented on pull request #24715: Upgrade more javascript files to typescript

2022-06-28 Thread GitBox
bbovenzi commented on PR #24715: URL: https://github.com/apache/airflow/pull/24715#issuecomment-1169319273 cc: @pierrejeambrun -- 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 commen

[GitHub] [airflow] potiuk commented on pull request #24663: Adding generic SqlToSlackOperator

2022-06-28 Thread GitBox
potiuk commented on PR #24663: URL: https://github.com/apache/airflow/pull/24663#issuecomment-1169271620 Needs rebase :( -- 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 pull request #24716: KubernetesPodOperator can get namespace from hook

2022-06-28 Thread GitBox
potiuk commented on PR #24716: URL: https://github.com/apache/airflow/pull/24716#issuecomment-1169267403 Needs rebase @dstandish :(. Sorry for that mishap -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above t

[GitHub] [airflow] potiuk commented on pull request #24715: Upgrade more javascript files to typescript

2022-06-28 Thread GitBox
potiuk commented on PR #24715: URL: https://github.com/apache/airflow/pull/24715#issuecomment-1169266606 Needs rebase I am afraid @bbovenzi -- 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

[airflow] branch main updated (5321577aee -> 0e1a6b9807)

2022-06-28 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 5321577aee Fix unnecessary check for ARM images (#24718) add 0e1a6b9807 Script to filter candidates for PR of the

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

2022-06-28 Thread GitBox
potiuk merged PR #24654: URL: https://github.com/apache/airflow/pull/24654 -- 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 move-setup-dependencies-to-providers updated (4a55d5a64c -> 8c75d701bf)

2022-06-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch move-setup-dependencies-to-providers in repository https://gitbox.apache.org/repos/asf/airflow.git discard 4a55d5a64c Move provider dependencies to inside provider folders add c7feb31786 De

[GitHub] [airflow] jedcunningham commented on a diff in pull request #24716: KubernetesPodOperator can get namespace from hook

2022-06-28 Thread GitBox
jedcunningham commented on code in PR #24716: URL: https://github.com/apache/airflow/pull/24716#discussion_r908964556 ## airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py: ## @@ -526,7 +526,7 @@ def build_pod_request_obj(self, context=None): api_version=

[airflow] branch main updated: Fix unnecessary check for ARM images (#24718)

2022-06-28 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 5321577aee Fix unnecessary check for ARM images (#2

[GitHub] [airflow] potiuk merged pull request #24718: Fix unnecessary check for ARM images

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

[GitHub] [airflow] o-nikolas commented on pull request #24643: Convert Step Functions Example DAG to System Test (AIP-47)

2022-06-28 Thread GitBox
o-nikolas commented on PR #24643: URL: https://github.com/apache/airflow/pull/24643#issuecomment-1169254637 @potiuk and @bhirsz do you have some spare cycles to look at this one? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] jedcunningham commented on pull request #24717: Fix CI image build

2022-06-28 Thread GitBox
jedcunningham commented on PR #24717: URL: https://github.com/apache/airflow/pull/24717#issuecomment-1169254423 Actual fix in #24718. -- 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] jedcunningham closed pull request #24717: Fix CI image build

2022-06-28 Thread GitBox
jedcunningham closed pull request #24717: Fix CI image build URL: https://github.com/apache/airflow/pull/24717 -- 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

[GitHub] [airflow] dstandish commented on a diff in pull request #24716: KubernetesPodOperator can get namespace from hook

2022-06-28 Thread GitBox
dstandish commented on code in PR #24716: URL: https://github.com/apache/airflow/pull/24716#discussion_r908960297 ## airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py: ## @@ -526,7 +526,7 @@ def build_pod_request_obj(self, context=None): api_version="v1"

[GitHub] [airflow] potiuk opened a new pull request, #24718: Fix unnecessary check for ARM images

2022-06-28 Thread GitBox
potiuk opened a new pull request, #24718: URL: https://github.com/apache/airflow/pull/24718 The ARM image build introduced in #24664 had problem with build image that was additionally checking for arm images which were moved out to a spearate step --- **^ Add meaningfu

[GitHub] [airflow] dstandish commented on a diff in pull request #24716: KubernetesPodOperator can get namespace from hook

2022-06-28 Thread GitBox
dstandish commented on code in PR #24716: URL: https://github.com/apache/airflow/pull/24716#discussion_r908958182 ## airflow/providers/cncf/kubernetes/hooks/kubernetes.py: ## @@ -152,16 +152,16 @@ def conn_extras(self): extras = {} return extras -def

[GitHub] [airflow] jedcunningham opened a new pull request, #24717: Fix CI image build

2022-06-28 Thread GitBox
jedcunningham opened a new pull request, #24717: URL: https://github.com/apache/airflow/pull/24717 We missed the matrix when adding ARM image builds in the CI process. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[airflow] branch fix_ci_image_build created (now 46dc817e18)

2022-06-28 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a change to branch fix_ci_image_build in repository https://gitbox.apache.org/repos/asf/airflow.git at 46dc817e18 Fix CI image build This branch includes the following new commits: new 46dc817e1

[airflow] 01/01: Fix CI image build

2022-06-28 Thread jedcunningham
This is an automated email from the ASF dual-hosted git repository. jedcunningham pushed a commit to branch fix_ci_image_build in repository https://gitbox.apache.org/repos/asf/airflow.git commit 46dc817e183e351c1251f3cb734876711dca12de Author: Jed Cunningham AuthorDate: Tue Jun 28 14:48:51 2022

[GitHub] [airflow] jedcunningham commented on a diff in pull request #24716: KubernetesPodOperator can get namespace from hook

2022-06-28 Thread GitBox
jedcunningham commented on code in PR #24716: URL: https://github.com/apache/airflow/pull/24716#discussion_r908925966 ## airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py: ## @@ -526,7 +526,7 @@ def build_pod_request_obj(self, context=None): api_version=

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

2022-06-28 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 edc62213da Updating con

[GitHub] [airflow] dstandish commented on a diff in pull request #24716: KubernetesPodOperator can get namespace from hook

2022-06-28 Thread GitBox
dstandish commented on code in PR #24716: URL: https://github.com/apache/airflow/pull/24716#discussion_r908918711 ## airflow/providers/cncf/kubernetes/hooks/kubernetes.py: ## @@ -363,11 +363,9 @@ def get_custom_object( def get_namespace(self) -> Optional[str]: """R

[GitHub] [airflow] dstandish opened a new pull request, #24716: KubernetesPodOperator can get namespace from hook

2022-06-28 Thread GitBox
dstandish opened a new pull request, #24716: URL: https://github.com/apache/airflow/pull/24716 Currently you have to set namespace in KPO init but we can get it from airflow connection if we're using one. -- This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [airflow-site] eladkal opened a new pull request, #619: Fixing `auth_mechanism` in `HiveMetastoreHook`/`HiveServer2Hook`

2022-06-28 Thread GitBox
eladkal opened a new pull request, #619: URL: https://github.com/apache/airflow-site/pull/619 Fix to the code applied in https://github.com/apache/airflow/pull/24713 for previous provider version we need to fix the docs to `authMechanism` (I don't know why the IDE force also edit of

[airflow-site] branch hive-provider created (now 05d645d1eb)

2022-06-28 Thread eladkal
This is an automated email from the ASF dual-hosted git repository. eladkal pushed a change to branch hive-provider in repository https://gitbox.apache.org/repos/asf/airflow-site.git at 05d645d1eb Fixing `auth_mechanism` in `HiveMetastoreHook`/`HiveServer2Hook` Fix to the code applied in

[airflow-site] branch hive created (now 73b5bff47a)

2022-06-28 Thread eladkal
This is an automated email from the ASF dual-hosted git repository. eladkal pushed a change to branch hive in repository https://gitbox.apache.org/repos/asf/airflow-site.git at 73b5bff47a Fixing `auth_mechanism` in `HiveMetastoreHook`/`HiveServer2Hook` This branch includes the following

[GitHub] [airflow] alexkruc commented on a diff in pull request #24663: Adding generic SqlToSlackOperator

2022-06-28 Thread GitBox
alexkruc commented on code in PR #24663: URL: https://github.com/apache/airflow/pull/24663#discussion_r908891277 ## airflow/providers/slack/transfers/sql_to_slack.py: ## @@ -0,0 +1,197 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor licens

[GitHub] [airflow] jedcunningham commented on pull request #20346: Fix inconsitencies in checking edit permissions for a DAG

2022-06-28 Thread GitBox
jedcunningham commented on PR #20346: URL: https://github.com/apache/airflow/pull/20346#issuecomment-1169172311 @kaxil, looks like we lost changes when this happened: e20e8f6 to ea7f928 Kicking this to 2.3.4, but we can pull it back if this gets done soon enough. -- This is an autom

[GitHub] [airflow] eladkal commented on a diff in pull request #24663: Adding generic SqlToSlackOperator

2022-06-28 Thread GitBox
eladkal commented on code in PR #24663: URL: https://github.com/apache/airflow/pull/24663#discussion_r908882238 ## airflow/providers/slack/transfers/sql_to_slack.py: ## @@ -0,0 +1,197 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

[GitHub] [airflow] eladkal commented on a diff in pull request #24599: updated documentation for databricks operator

2022-06-28 Thread GitBox
eladkal commented on code in PR #24599: URL: https://github.com/apache/airflow/pull/24599#discussion_r908874570 ## airflow/providers/databricks/operators/databricks.py: ## @@ -150,7 +150,7 @@ class DatabricksSubmitRunOperator(BaseOperator):

[GitHub] [airflow] bbovenzi commented on issue #24699: Clear one single mapped task instance

2022-06-28 Thread GitBox
bbovenzi commented on issue #24699: URL: https://github.com/apache/airflow/issues/24699#issuecomment-1169150371 I think that was the source of some confusion over at https://github.com/apache/airflow/pull/23516 -- This is an automated message from the Apache Git Service. To respond to the

[airflow] branch main updated: Add AWS operators to create and delete RDS Database (#24099)

2022-06-28 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 bf727525e1 Add AWS operators to create and delete

[GitHub] [airflow] eladkal merged pull request #24099: Add AWS operators to create and delete RDS Database

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

[GitHub] [airflow] eladkal closed issue #23822: Add an AWS operator for Create RDS Database

2022-06-28 Thread GitBox
eladkal closed issue #23822: Add an AWS operator for Create RDS Database URL: https://github.com/apache/airflow/issues/23822 -- 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] bbovenzi opened a new pull request, #24715: Upgrade more javascript files to typescript

2022-06-28 Thread GitBox
bbovenzi opened a new pull request, #24715: URL: https://github.com/apache/airflow/pull/24715 Improve type checking in the grid view to catch issues before runtime. --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airf

[GitHub] [airflow] eladkal opened a new pull request, #24713: fix connection extra parameter `auth_mechanism` in `HiveMetastoreHook` and `HiveServer2Hook`

2022-06-28 Thread GitBox
eladkal opened a new pull request, #24713: URL: https://github.com/apache/airflow/pull/24713 closes: https://github.com/apache/airflow/issues/24692 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airfl

[GitHub] [airflow] cromgit closed pull request #24712: [ENG-0000] add env values and tolerations

2022-06-28 Thread GitBox
cromgit closed pull request #24712: [ENG-] add env values and tolerations URL: https://github.com/apache/airflow/pull/24712 -- 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.

[GitHub] [airflow] cromgit opened a new pull request, #24712: [ENG-0000] add env values and tolerations

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

[airflow] branch main updated (2fbd750d35 -> c7feb31786)

2022-06-28 Thread bbovenzi
This is an automated email from the ASF dual-hosted git repository. bbovenzi pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git from 2fbd750d35 Add ARM image building for regular PRs (#24664) add c7feb31786 Debounce status highlighting in Grid v

[GitHub] [airflow] bbovenzi merged pull request #24710: Debounce status highlighting in Grid view

2022-06-28 Thread GitBox
bbovenzi merged PR #24710: URL: https://github.com/apache/airflow/pull/24710 -- 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 move-setup-dependencies-to-providers updated (122c809ce5 -> 4a55d5a64c)

2022-06-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch move-setup-dependencies-to-providers in repository https://gitbox.apache.org/repos/asf/airflow.git omit 122c809ce5 Move provider dependencies to inside provider folders add 2fbd750d35 Ad

[GitHub] [airflow] potiuk merged pull request #24664: Add ARM image building for regular PRs

2022-06-28 Thread GitBox
potiuk merged PR #24664: URL: https://github.com/apache/airflow/pull/24664 -- 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 move-setup-dependencies-to-providers updated (fc71606f87 -> 122c809ce5)

2022-06-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch move-setup-dependencies-to-providers in repository https://gitbox.apache.org/repos/asf/airflow.git omit fc71606f87 Move provider dependencies to inside provider folders add 122c809ce5 Mo

[GitHub] [airflow] jedcunningham commented on pull request #24489: Documentation on task mapping additions

2022-06-28 Thread GitBox
jedcunningham commented on PR #24489: URL: https://github.com/apache/airflow/pull/24489#issuecomment-1169067276 Removing from the milestone and setting to draft, as we should hold this until we are ready to get it in the next minor release. Feel free to adjust this as necessary @uranusjr.

[GitHub] [airflow] potiuk opened a new pull request, #24711: Merge-friendly output of command hashes for breeze

2022-06-28 Thread GitBox
potiuk opened a new pull request, #24711: URL: https://github.com/apache/airflow/pull/24711 We had just one hash generated from all commands in breeze and that basically meant that when there were two PRs on two different commands in Breeze, they resulted with merge conflict which sh

[GitHub] [airflow] jedcunningham commented on pull request #24530: Filter on XCOM key name when calculating map lengths in MappedOperator

2022-06-28 Thread GitBox
jedcunningham commented on PR #24530: URL: https://github.com/apache/airflow/pull/24530#issuecomment-1169063735 @hinnefe2, gentle ping. It'd be great to get this into 2.3.3. That test file is the right spot. If you have further questions/get stuck, feel free to ping us in #development on Sl

[GitHub] [airflow] bbovenzi commented on pull request #24710: Debounce status highlighting in Grid view

2022-06-28 Thread GitBox
bbovenzi commented on PR #24710: URL: https://github.com/apache/airflow/pull/24710#issuecomment-1169062195 Actually, looks like we weren't consistent with our tooltip delays. I made a `hoverDelay` util variable to fix this. -- This is an automated message from the Apache Git Service. To r

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

2022-06-28 Thread GitBox
jedcunningham commented on PR #24664: URL: https://github.com/apache/airflow/pull/24664#issuecomment-1169055918 Thanks for the detailed reply @potiuk. All sounds good and makes sense to me. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [airflow] pierrejeambrun commented on pull request #24710: Debounce status highlighting in Grid view

2022-06-28 Thread GitBox
pierrejeambrun commented on PR #24710: URL: https://github.com/apache/airflow/pull/24710#issuecomment-1169045293 Nice improvement :+1: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specifi

[airflow] branch move-setup-dependencies-to-providers updated (6ba9a8882a -> fc71606f87)

2022-06-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch move-setup-dependencies-to-providers in repository https://gitbox.apache.org/repos/asf/airflow.git discard 6ba9a8882a Move provider dependencies to inside provider folders add a5ef7a02e1 Cl

[GitHub] [airflow] bbovenzi commented on pull request #24710: Debounce status highlighting in Grid view

2022-06-28 Thread GitBox
bbovenzi commented on PR #24710: URL: https://github.com/apache/airflow/pull/24710#issuecomment-1169034410 > Code looks fine - only question is if 400ms is too big or too small It's the same delay we already have for Tooltips -- This is an automated message from the Apache Git Servi

[airflow] branch add-arm-image-building updated (79d3f00b4b -> f7b1b033dd)

2022-06-28 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch add-arm-image-building in repository https://gitbox.apache.org/repos/asf/airflow.git discard 79d3f00b4b Add ARM image building for regular PRs add a5ef7a02e1 Clear next method when clearing

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

2022-06-28 Thread GitBox
potiuk commented on code in PR #24654: URL: https://github.com/apache/airflow/pull/24654#discussion_r908766542 ## dev/stats/get_important_pr_candidates.py: ## @@ -0,0 +1,187 @@ +#!/usr/bin/env python3 +# Licensed to the Apache Software Foundation (ASF) under one +# or more contr

[GitHub] [airflow] ashb commented on pull request #24710: Debounce status highlighting in Grid view

2022-06-28 Thread GitBox
ashb commented on PR #24710: URL: https://github.com/apache/airflow/pull/24710#issuecomment-1169015129 How does it feel with a delay of 400ms? Did they number come from anywhere particular? -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] bbovenzi commented on pull request #24710: Debounce status highlighting in Grid view

2022-06-28 Thread GitBox
bbovenzi commented on PR #24710: URL: https://github.com/apache/airflow/pull/24710#issuecomment-1169005152 CC: @pierrejeambrun -- 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 commen

[GitHub] [airflow] bbovenzi opened a new pull request, #24710: Debounce status highlighting in Grid view

2022-06-28 Thread GitBox
bbovenzi opened a new pull request, #24710: URL: https://github.com/apache/airflow/pull/24710 Right now, any amount of hovering over a status in grid view will trigger highlighting. Instead, we should make sure the action is intentional by including a debounce with a 400ms delay. ---

[GitHub] [airflow] whitleykeith commented on issue #23361: Scheduler crashes with psycopg2.errors.DeadlockDetected exception

2022-06-28 Thread GitBox
whitleykeith commented on issue #23361: URL: https://github.com/apache/airflow/issues/23361#issuecomment-1168991129 Having this issue as well, we're only on 2.0.2. Looking through some of the Airflow configs I'm wondering if it could be related to the [file_parsing_sort_mode](https:/

[GitHub] [airflow] ArjunSharda commented on pull request #24705: (GitHub CI) update most CI workflows (stale, build-images, etc)

2022-06-28 Thread GitBox
ArjunSharda commented on PR #24705: URL: https://github.com/apache/airflow/pull/24705#issuecomment-1168984897 Thanks, am going to add for one more CI -- 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

[GitHub] [airflow-client-go] igr opened a new issue, #24: Release versions?

2022-06-28 Thread GitBox
igr opened a new issue, #24: URL: https://github.com/apache/airflow-client-go/issues/24 Could you please specify release versions? It would be great if they can match Airflow version -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [airflow] bbovenzi commented on pull request #24623: Grid task log, multi select and ts files migration.

2022-06-28 Thread GitBox
bbovenzi commented on PR #24623: URL: https://github.com/apache/airflow/pull/24623#issuecomment-1168940512 Let's update the `scrollIntoView()` effect to be based off of the `parsedLogs` and not data. The scroll position is lost when filtering. -- This is an automated message from the Apac

  1   2   3   >