[GitHub] [airflow] streetmapp commented on pull request #8777: Add Production Helm chart support

2020-06-02 Thread GitBox
streetmapp commented on pull request #8777: URL: https://github.com/apache/airflow/pull/8777#issuecomment-637931340 The ingress components were already capable of being on/off were they not? Seems odd to remove them if that was the case.

[GitHub] [airflow] casassg commented on pull request #8962: [AIRFLOW-8057] [AIP-31] Add @task decorator

2020-06-02 Thread GitBox
casassg commented on pull request #8962: URL: https://github.com/apache/airflow/pull/8962#issuecomment-637930476 Mostly adding more hours to my day. Sorry, been a bit busy this week. Will try to address most comments This

[jira] [Commented] (AIRFLOW-6535) add exception subclass to fail immediately without retrying

2020-06-02 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17124555#comment-17124555 ] ASF GitHub Bot commented on AIRFLOW-6535: - marshall7m commented on pull request #7133: URL:

[GitHub] [airflow] marshall7m commented on pull request #7133: [AIRFLOW-6535] add exception to fail without retry

2020-06-02 Thread GitBox
marshall7m commented on pull request #7133: URL: https://github.com/apache/airflow/pull/7133#issuecomment-637924361 Hi there, just wondering what version of airflow the AirflowFailException is available on. Thanks! This is

[jira] [Commented] (AIRFLOW-6290) Create guide for GKE service operator

2020-06-02 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6290?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17124537#comment-17124537 ] ASF GitHub Bot commented on AIRFLOW-6290: - tanjinP commented on pull request #8883: URL:

[GitHub] [airflow] tanjinP commented on pull request #8883: [AIRFLOW-6290] Create guide for GKE operators

2020-06-02 Thread GitBox
tanjinP commented on pull request #8883: URL: https://github.com/apache/airflow/pull/8883#issuecomment-637919016 @mik-laj - I'm good for the merge, thanks for checking in! :shipit: This is an automated message from the

[GitHub] [airflow] maverick12345678 commented on issue #8893: Displaying field templates and constructor parameters in scripts/list-integration.py

2020-06-02 Thread GitBox
maverick12345678 commented on issue #8893: URL: https://github.com/apache/airflow/issues/8893#issuecomment-637914952 > This file? What exactly code? > https://github.com/apache/airflow/blob/master/scripts/list-integrations.py > > I want to have the following output or equivalent:

[GitHub] [airflow] dimberman commented on pull request #8777: Add Production Helm chart support

2020-06-02 Thread GitBox
dimberman commented on pull request #8777: URL: https://github.com/apache/airflow/pull/8777#issuecomment-637908838 @schnie can you make the yamllint tests pass please? I think we can merge an initial PR just to get the code in and then start fixing it on master.

[GitHub] [airflow] jhtimmins commented on a change in pull request #9029: Remove hive/hadoop/java dependency from unit tests

2020-06-02 Thread GitBox
jhtimmins commented on a change in pull request #9029: URL: https://github.com/apache/airflow/pull/9029#discussion_r434267828 ## File path: tests/providers/apache/hive/hooks/test_hive.py ## @@ -454,34 +608,91 @@ def test_get_conn_with_password(self, mock_connect):

[GitHub] [airflow] dimberman commented on a change in pull request #8265: Kubernetes on host

2020-06-02 Thread GitBox
dimberman commented on a change in pull request #8265: URL: https://github.com/apache/airflow/pull/8265#discussion_r434263901 ## File path: kubernetes_tests/test_kubernetes_pod_operator.py ## @@ -141,98 +143,11 @@ def test_config_path_move(self):

[GitHub] [airflow] Acehaidrey commented on pull request #8598: make hive macros return string type vs bytes

2020-06-02 Thread GitBox
Acehaidrey commented on pull request #8598: URL: https://github.com/apache/airflow/pull/8598#issuecomment-637876351 Hey @ashb for 1.10 it actually can remain the same. So from a quick test I ran locally even if py2 it returns the value as a string type. Not sure why they encoded but maybe

[GitHub] [airflow] GMarkfjard commented on issue #8132: API Endpoints - Read-only - Task Instance

2020-06-02 Thread GitBox
GMarkfjard commented on issue #8132: URL: https://github.com/apache/airflow/issues/8132#issuecomment-637864599 @mik-laj I'd love to get my hands dirty with this! This is an automated message from the Apache Git Service.

[GitHub] [airflow] kaxil commented on a change in pull request #9029: Remove hive/hadoop/java dependency from unit tests

2020-06-02 Thread GitBox
kaxil commented on a change in pull request #9029: URL: https://github.com/apache/airflow/pull/9029#discussion_r434222861 ## File path: tests/providers/apache/hive/hooks/test_hive.py ## @@ -454,34 +608,91 @@ def test_get_conn_with_password(self, mock_connect):

[GitHub] [airflow] schnie commented on pull request #8777: Add Production Helm chart support

2020-06-02 Thread GitBox
schnie commented on pull request #8777: URL: https://github.com/apache/airflow/pull/8777#issuecomment-637831190 > @schnie, not all K8S clusters support Ingress that are not on the public internet, therefore not all companies be able to use this chart. > > You should add support for

[GitHub] [airflow] schnie commented on a change in pull request #8777: Add Production Helm chart support

2020-06-02 Thread GitBox
schnie commented on a change in pull request #8777: URL: https://github.com/apache/airflow/pull/8777#discussion_r434196412 ## File path: chart/templates/workers/worker-horizontalpodautoscaler.yaml ## @@ -0,0 +1,34 @@ + +## Airflow Worker

[GitHub] [airflow] marclamberti commented on pull request #8777: Add Production Helm chart support

2020-06-02 Thread GitBox
marclamberti commented on pull request #8777: URL: https://github.com/apache/airflow/pull/8777#issuecomment-637818638 @dimberman Well indeed, I got some issues too, one at this moment actually. I agreed with @streetmapp my point was, there are three charts right now, having some very

[GitHub] [airflow] potiuk commented on a change in pull request #8265: Kubernetes on host

2020-06-02 Thread GitBox
potiuk commented on a change in pull request #8265: URL: https://github.com/apache/airflow/pull/8265#discussion_r434187900 ## File path: scripts/ci/kubernetes/app/templates/airflow.template.yaml ## @@ -42,33 +42,43 @@ spec: name: airflow spec:

[GitHub] [airflow] samuelkhtu commented on issue #9099: moving sql operators into sql.py

2020-06-02 Thread GitBox
samuelkhtu commented on issue #9099: URL: https://github.com/apache/airflow/issues/9099#issuecomment-637817437 Should we change the class name from BranchSqlOperator to SQLBranchOperator or use the Python organization? (BranchSQLOperator?)

[GitHub] [airflow] dimberman commented on pull request #8777: Add Production Helm chart support

2020-06-02 Thread GitBox
dimberman commented on pull request #8777: URL: https://github.com/apache/airflow/pull/8777#issuecomment-637806957 @marclamberti we get a lot of questions from users of the community chart who can't seem to get it to work/have all sorts of weird issues. Has this not been your experience?

[GitHub] [airflow] ashb commented on pull request #8227: Add run_type to DagRun

2020-06-02 Thread GitBox
ashb commented on pull request #8227: URL: https://github.com/apache/airflow/pull/8227#issuecomment-637799925 > > @turbaszek Is it much work to run your benchmark again: `DR.run_type != DagRunType.BACKFILL_JOB.value` is something we do in the "hot" path of the scheduler, and the `!=`

[GitHub] [airflow] ashb commented on pull request #9105: Cope with multiple processes get_remote_image_info in parallel

2020-06-02 Thread GitBox
ashb commented on pull request #9105: URL: https://github.com/apache/airflow/pull/9105#issuecomment-637798777 > `./breeze static-check-all-files flake8 --force-pull-images` will force-pull the image before you run it to save the time for local rebuilding. Takes 2 minutes to pull + build

[GitHub] [airflow] ashb commented on pull request #9105: Cope with multiple processes get_remote_image_info in parallel

2020-06-02 Thread GitBox
ashb commented on pull request #9105: URL: https://github.com/apache/airflow/pull/9105#issuecomment-637797720 > BTW. What was the error? I would love to understand what happened there. The error was (at first) not printed cos we were redirecting stderr to dev null. Once I'd

[GitHub] [airflow] houqp commented on issue #9115: API Endpoint - CRUD - Variable

2020-06-02 Thread GitBox
houqp commented on issue #9115: URL: https://github.com/apache/airflow/issues/9115#issuecomment-637790667 I will take this one as well :) This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] streetmapp commented on pull request #8777: Add Production Helm chart support

2020-06-02 Thread GitBox
streetmapp commented on pull request #8777: URL: https://github.com/apache/airflow/pull/8777#issuecomment-637790261 I would definitely agree in terms of basing it from the chart that is in the stable Helm charts repo. There are some interesting components in the Astronomer and Bitnami

[GitHub] [airflow] mik-laj edited a comment on pull request #9029: Remove hive/hadoop/java dependency from unit tests

2020-06-02 Thread GitBox
mik-laj edited a comment on pull request #9029: URL: https://github.com/apache/airflow/pull/9029#issuecomment-637779636 @jhtimmins My change has caused us to cross a certain threshold that causes instability. However. the exact cause of this instability is unknown. I will want to track

[GitHub] [airflow] jhtimmins commented on pull request #9029: Remove hive/hadoop/java dependency from unit tests

2020-06-02 Thread GitBox
jhtimmins commented on pull request #9029: URL: https://github.com/apache/airflow/pull/9029#issuecomment-637780230 @mik-laj Awesome, that's good to know. Thanks for the info! This is an automated message from the Apache Git

[GitHub] [airflow] mik-laj commented on pull request #9029: Remove hive/hadoop/java dependency from unit tests

2020-06-02 Thread GitBox
mik-laj commented on pull request #9029: URL: https://github.com/apache/airflow/pull/9029#issuecomment-637779636 @jhtimmins My change has caused us to cross a certain threshold that causes the exact cause of this instability. I will want to track her cause soon. I restarted Github Action

[GitHub] [airflow] jhtimmins edited a comment on pull request #9029: Remove hive/hadoop/java dependency from unit tests

2020-06-02 Thread GitBox
jhtimmins edited a comment on pull request #9029: URL: https://github.com/apache/airflow/pull/9029#issuecomment-637776998 @mik-laj This is now failing tests that seem related to this change you made to `test_scheduler_job.py`

[GitHub] [airflow] jhtimmins commented on pull request #9029: Remove hive/hadoop/java dependency from unit tests

2020-06-02 Thread GitBox
jhtimmins commented on pull request #9029: URL: https://github.com/apache/airflow/pull/9029#issuecomment-637776998 @mik-laj This is now failing tests that seem related to this change you made to `test_scheduler_job.py`

[GitHub] [airflow] randr97 commented on issue #8134: API Endpoints - Read-only - XCOM

2020-06-02 Thread GitBox
randr97 commented on issue #8134: URL: https://github.com/apache/airflow/issues/8134#issuecomment-637771167 @mik-laj will be working on it thanks :) This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] bertosantamaria commented on issue #9109: API Endpoint - CRUD - DAG

2020-06-02 Thread GitBox
bertosantamaria commented on issue #9109: URL: https://github.com/apache/airflow/issues/9109#issuecomment-637765085 Yes hi and thank you. > On Jun 2, 2020, at 3:36 PM, Kamil BreguĊ‚a wrote: > > > @bertosantamaria Can you reply to

[GitHub] [airflow] bertosantamaria commented on issue #8128: API Endpoints - Read-only - DAG

2020-06-02 Thread GitBox
bertosantamaria commented on issue #8128: URL: https://github.com/apache/airflow/issues/8128#issuecomment-637764429 Ah, I see. Ok, sounds great. thanks! This is an automated message from the Apache Git Service. To respond

[GitHub] [airflow] mik-laj commented on issue #9109: API Endpoint - CRUD - DAG

2020-06-02 Thread GitBox
mik-laj commented on issue #9109: URL: https://github.com/apache/airflow/issues/9109#issuecomment-637762656 @bertosantamaria Can you reply to this? This allows me to assign you to this ticket. This is an automated

[GitHub] [airflow] pegerto opened a new issue #9117: Apache beam issue extracting job from monitoring page

2020-06-02 Thread GitBox
pegerto opened a new issue #9117: URL: https://github.com/apache/airflow/issues/9117 **Apache Airflow version**: 1.10.10 and 1.10.9 has been verify **Kubernetes version (if you are using kubernetes)** (use `kubectl version`): no kube **Environment**: -

[GitHub] [airflow] boring-cyborg[bot] commented on issue #9117: Apache beam issue extracting job from monitoring page

2020-06-02 Thread GitBox
boring-cyborg[bot] commented on issue #9117: URL: https://github.com/apache/airflow/issues/9117#issuecomment-637762162 Thanks for opening your first issue here! Be sure to follow the issue template! This is an automated

[GitHub] [airflow] mik-laj commented on issue #8128: API Endpoints - Read-only - DAG

2020-06-02 Thread GitBox
mik-laj commented on issue #8128: URL: https://github.com/apache/airflow/issues/8128#issuecomment-637761887 @bertosantamaria These tickets depend on each other. We first need to have endpoints that allow data to be read so that we can implement further endpoints. Both endpoints must

[GitHub] [airflow] mik-laj commented on issue #9115: API Endpoint - CRUD - Variable

2020-06-02 Thread GitBox
mik-laj commented on issue #9115: URL: https://github.com/apache/airflow/issues/9115#issuecomment-637759645 @houqp Can you write a comment here? This will allow me to assign you to this ticket. This is an automated message

[GitHub] [airflow] bertosantamaria removed a comment on issue #8128: API Endpoints - Read-only - DAG

2020-06-02 Thread GitBox
bertosantamaria removed a comment on issue #8128: URL: https://github.com/apache/airflow/issues/8128#issuecomment-637752682 @mik-laj to be honest the change I'd like to implement is this one https://github.com/apache/airflow/issues/9109. It seems this issue (8128) has been rescoped to

[GitHub] [airflow] bertosantamaria edited a comment on issue #8128: API Endpoints - Read-only - DAG

2020-06-02 Thread GitBox
bertosantamaria edited a comment on issue #8128: URL: https://github.com/apache/airflow/issues/8128#issuecomment-637752682 @mik-laj to be honest the change I'd like to implement is this one https://github.com/apache/airflow/issues/9109. It seems this issue (8128) has been rescoped to

[GitHub] [airflow] bertosantamaria commented on issue #8128: API Endpoints - Read-only - DAG

2020-06-02 Thread GitBox
bertosantamaria commented on issue #8128: URL: https://github.com/apache/airflow/issues/8128#issuecomment-637759134 @mik-laj to be honest the change I'd like to implement is this one #9109. It seems this issue (8128) has been rescoped to implement read-only endpoints. Are there any

[GitHub] [airflow] houqp commented on issue #8133: API Endpoints - Read-only - Variable

2020-06-02 Thread GitBox
houqp commented on issue #8133: URL: https://github.com/apache/airflow/issues/8133#issuecomment-637755972 @mik-laj looks good to me! This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow] mik-laj commented on issue #8128: API Endpoints - Read-only - DAG

2020-06-02 Thread GitBox
mik-laj commented on issue #8128: URL: https://github.com/apache/airflow/issues/8128#issuecomment-637755684 @bertosantamaria I think that a better idea would be to take care of another ticket that has no assigned person. You can track the current tickets on the board.

[airflow] branch master updated (67379d1 -> 5f038d0)

2020-06-02 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 67379d1 Add fudament for API based on connexion (#8149) add 5f038d0 Add filepaths for API label in

[GitHub] [airflow] kaxil merged pull request #9116: Add filepaths for API label in BoringCyborg Bot

2020-06-02 Thread GitBox
kaxil merged pull request #9116: URL: https://github.com/apache/airflow/pull/9116 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

[GitHub] [airflow] bertosantamaria commented on issue #8128: API Endpoints - Read-only - DAG

2020-06-02 Thread GitBox
bertosantamaria commented on issue #8128: URL: https://github.com/apache/airflow/issues/8128#issuecomment-637752682 @mik-laj Just confirming you are suggesting picking up where you left off on https://github.com/apache/airflow/pull/9045? I'd be happy to work on that. thanks

[GitHub] [airflow] waleedsamy commented on issue #9015: Update to Pendulum 2.0.2

2020-06-02 Thread GitBox
waleedsamy commented on issue #9015: URL: https://github.com/apache/airflow/issues/9015#issuecomment-637749723 +1 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] kaxil opened a new pull request #9116: Add filepaths for API label in BoringCyborg Bot

2020-06-02 Thread GitBox
kaxil opened a new pull request #9116: URL: https://github.com/apache/airflow/pull/9116 Automatically add labels on PRs related to API --- Make sure to mark the boxes below before creating PR: [x] - [x] Description above provides context of the change - [x] Unit tests

[GitHub] [airflow] mik-laj commented on issue #8133: API Endpoints - Read-only - Variable

2020-06-02 Thread GitBox
mik-laj commented on issue #8133: URL: https://github.com/apache/airflow/issues/8133#issuecomment-637735716 A single PR can close multiple tickets. I divided the variable endpoints because I want to involve many new people in these tasks. When the ticket is small it is easier to finish

[GitHub] [airflow] klsnreddy commented on issue #9046: TimeDeltaSensor and schedule_interval="@once" leads to crash

2020-06-02 Thread GitBox
klsnreddy commented on issue #9046: URL: https://github.com/apache/airflow/issues/9046#issuecomment-637734773 This is kind of expected behavior, as TimeDeltaSensor adds the delta time to the schedule end time. So in case of manual or once executions there is not specified end time.

[GitHub] [airflow] houqp commented on issue #8133: API Endpoints - Read-only - Variable

2020-06-02 Thread GitBox
houqp commented on issue #8133: URL: https://github.com/apache/airflow/issues/8133#issuecomment-637723779 @mik-laj yes, I can take a stab at it this week. do you mind if I implement write endpoints as well or do you prefer me to only fill out the read endpoints?

[GitHub] [airflow] marclamberti edited a comment on pull request #8777: Add Production Helm chart support

2020-06-02 Thread GitBox
marclamberti edited a comment on pull request #8777: URL: https://github.com/apache/airflow/pull/8777#issuecomment-637609564 Wouldn't be easier to start with the chart of @thesuperzapper ? In the sense that it already include a lot of features ? I'm pretty sure it is widely used too as it

[GitHub] [airflow] mik-laj closed issue #8114: (de)serialization common objects

2020-06-02 Thread GitBox
mik-laj closed issue #8114: URL: https://github.com/apache/airflow/issues/8114 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] mik-laj commented on issue #8115: (de)serialization for Python objects

2020-06-02 Thread GitBox
mik-laj commented on issue #8115: URL: https://github.com/apache/airflow/issues/8115#issuecomment-637710254 We will do this when implementing read-only operations for each endpoint. This will allow us to check schema compliance with OpenAPI.

[GitHub] [airflow] mik-laj commented on issue #8114: (de)serialization common objects

2020-06-02 Thread GitBox
mik-laj commented on issue #8114: URL: https://github.com/apache/airflow/issues/8114#issuecomment-637710355 We will do this when implementing read-only operations for each endpoint. This will allow us to check schema compliance with OpenAPI.

[GitHub] [airflow] mik-laj closed issue #8115: (de)serialization for Python objects

2020-06-02 Thread GitBox
mik-laj closed issue #8115: URL: https://github.com/apache/airflow/issues/8115 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] mik-laj commented on issue #8110: (de)serialziation for SQLAlchemy objects

2020-06-02 Thread GitBox
mik-laj commented on issue #8110: URL: https://github.com/apache/airflow/issues/8110#issuecomment-637710058 We will do this when implementing read-only operations for each endpoint. This will allow us to check schema compliance with OpenAPI.

[GitHub] [airflow] mik-laj closed issue #8110: (de)serialziation for SQLAlchemy objects

2020-06-02 Thread GitBox
mik-laj closed issue #8110: URL: https://github.com/apache/airflow/issues/8110 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] mik-laj closed issue #8108: Basic OpenAPI spec

2020-06-02 Thread GitBox
mik-laj closed issue #8108: URL: https://github.com/apache/airflow/issues/8108 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] mik-laj opened a new issue #9115: API Endpoint - CRUD.- Variable

2020-06-02 Thread GitBox
mik-laj opened a new issue #9115: URL: https://github.com/apache/airflow/issues/9115 Hello We need to create several endpoints that perform basic CRUD operations on **Variable**. We need the following endpoints: POST /variables DELETE /variables/{variable_key} PATCH

[GitHub] [airflow] mik-laj opened a new issue #9114: API Endpoint - CRUD - Pool

2020-06-02 Thread GitBox
mik-laj opened a new issue #9114: URL: https://github.com/apache/airflow/issues/9114 Hello We need to create several endpoints that perform basic CRUD operations on **Pools**. We need the following endpoints: POST /pools DELETE /pools/{pool_name} PATCH

[GitHub] [airflow] mik-laj opened a new issue #9113: API Endpoint - DELETE Import Error

2020-06-02 Thread GitBox
mik-laj opened a new issue #9113: URL: https://github.com/apache/airflow/issues/9113 Hello We need to create several endpoints that perform one ono read-only operation on **Import Error**. We need the following endpoint: DELETE /importErrors/{import_error_id} Detailed

[GitHub] [airflow] mik-laj commented on issue #8133: API Endpoints - Read-only - Variable

2020-06-02 Thread GitBox
mik-laj commented on issue #8133: URL: https://github.com/apache/airflow/issues/8133#issuecomment-637704864 @houqp Do you want to work on it? We are ready. You can start working from this file

[GitHub] [airflow] ephraimbuddy commented on issue #8127: API Endpoints - Read-only - Connection

2020-06-02 Thread GitBox
ephraimbuddy commented on issue #8127: URL: https://github.com/apache/airflow/issues/8127#issuecomment-637703654 Yes. I am working on it This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] mik-laj opened a new issue #9112: API Endpoint - Batch

2020-06-02 Thread GitBox
mik-laj opened a new issue #9112: URL: https://github.com/apache/airflow/issues/9112 Hello We need to create several endpoints that perform batch operations on **XCOM Entries*. We need the following endpoints: POST /dags/~/dagRuns/list POST

[GitHub] [airflow] mik-laj opened a new issue #9111: API Endpoints - CRUD - XCOM

2020-06-02 Thread GitBox
mik-laj opened a new issue #9111: URL: https://github.com/apache/airflow/issues/9111 Hello We need to create several endpoints that perform basic CRUD operations on **XCOM Entries*. We need the following endpoints: POST

[GitHub] [airflow] mik-laj opened a new issue #9110: API Endpoint - CRUD Dag Runs

2020-06-02 Thread GitBox
mik-laj opened a new issue #9110: URL: https://github.com/apache/airflow/issues/9110 Hello We need to create several endpoints that perform basic CRUD operations on **DAG Run**. We need the following endpoints: DELETE /dags/{dag_id}/dagRuns/{dag_run_id} PATCH

[GitHub] [airflow] mik-laj opened a new issue #9109: API Endpoint - CRUD - DAG

2020-06-02 Thread GitBox
mik-laj opened a new issue #9109: URL: https://github.com/apache/airflow/issues/9109 Hello We need to create several endpoints that perform basic CRUD operations on **DAG**. We need the following endpoints: - PATCH /dags/{dag_id} It depends on

[GitHub] [airflow] mik-laj opened a new issue #9108: API Endpoint - CRUD - Connection

2020-06-02 Thread GitBox
mik-laj opened a new issue #9108: URL: https://github.com/apache/airflow/issues/9108 Hello We need to create several endpoints that perform basic CRUD operations on **Connection** . We need the following endpoints: - POST /connections - DELETE /connections/{connection_id}

[GitHub] [airflow] mik-laj commented on issue #8127: API Endpoints - Read-only - Connection

2020-06-02 Thread GitBox
mik-laj commented on issue #8127: URL: https://github.com/apache/airflow/issues/8127#issuecomment-637694114 @ephraimbuddy Are you working on this change? Can you comment on this ticket so that I can assign you? https://github.com/apache/airflow/pull/9095

[GitHub] [airflow] mik-laj commented on issue #8136: API Endpoint - Config

2020-06-02 Thread GitBox
mik-laj commented on issue #8136: URL: https://github.com/apache/airflow/issues/8136#issuecomment-637692135 Hi. We have already started to implement. Here is config_endpoint.py file. You should start working here.

[GitHub] [airflow] OmairK edited a comment on issue #8131: API Endpoints - Read-only - Pools

2020-06-02 Thread GitBox
OmairK edited a comment on issue #8131: URL: https://github.com/apache/airflow/issues/8131#issuecomment-637689130 Yes I am working on it can you please assign it to me, thanks. >For now, we focus only on read-only operations, but the others will also be implemented as the next step.

[GitHub] [airflow] OmairK edited a comment on issue #8131: API Endpoints - Read-only - Pools

2020-06-02 Thread GitBox
OmairK edited a comment on issue #8131: URL: https://github.com/apache/airflow/issues/8131#issuecomment-637689130 Yes I am working on it can you please assign it to me, thanks. >For now, we focus only on read-only operations, but the others will also be implemented as the next step.

[GitHub] [airflow] OmairK commented on issue #8131: API Endpoints - Read-only - Pools

2020-06-02 Thread GitBox
OmairK commented on issue #8131: URL: https://github.com/apache/airflow/issues/8131#issuecomment-637689130 Yes I am working on it can you please assign it to me, thanks. This is an automated message from the Apache Git

[GitHub] [airflow] seelmann commented on a change in pull request #9087: Query TaskReschedule only if task is UP_FOR_RESCHEDULE

2020-06-02 Thread GitBox
seelmann commented on a change in pull request #9087: URL: https://github.com/apache/airflow/pull/9087#discussion_r434039646 ## File path: airflow/models/taskreschedule.py ## @@ -63,24 +63,41 @@ def __init__(self, task, execution_date, try_number, start_date, end_date,

[GitHub] [airflow] mik-laj edited a comment on issue #8131: API Endpoints - Read-only - Pools

2020-06-02 Thread GitBox
mik-laj edited a comment on issue #8131: URL: https://github.com/apache/airflow/issues/8131#issuecomment-637688065 @OmairK Are you working on this change? Can you comment on this ticket so that I can assign you? https://github.com/apache/airflow/pull/9097 For now, we focus only

[GitHub] [airflow] mik-laj commented on issue #8131: API Endpoints - CRUD - Pools

2020-06-02 Thread GitBox
mik-laj commented on issue #8131: URL: https://github.com/apache/airflow/issues/8131#issuecomment-637688065 @OmairK Are you working on this change? Can you comment on this ticket so that I can assign you? https://github.com/apache/airflow/pull/9097

[GitHub] [airflow] mik-laj commented on issue #8129: API Endpoints - CRUD - DAG Runs

2020-06-02 Thread GitBox
mik-laj commented on issue #8129: URL: https://github.com/apache/airflow/issues/8129#issuecomment-637686812 @davinchia or @rtstock Hi. We have already started to implement. Do you still want to work on this ticket? For now, we focus only on read-only operations, but others will also be

[GitHub] [airflow] mik-laj commented on issue #8128: API Endpoints - CRUD - DAG Model

2020-06-02 Thread GitBox
mik-laj commented on issue #8128: URL: https://github.com/apache/airflow/issues/8128#issuecomment-637682206 Hey, I found a PR who is already starting to implement this ticket. Do you want to take it over? https://github.com/apache/airflow/pull/9045 Would you not be interested in

[GitHub] [airflow] kaxil commented on issue #8135: API Endpoint - Logs

2020-06-02 Thread GitBox
kaxil commented on issue #8135: URL: https://github.com/apache/airflow/issues/8135#issuecomment-637681838 Yeah sorry I won't be able to work on it for at least a month or two, so please feel free to reassign This is an

[GitHub] [airflow] mik-laj commented on issue #8134: API Endpoints - CRUD - XCOM

2020-06-02 Thread GitBox
mik-laj commented on issue #8134: URL: https://github.com/apache/airflow/issues/8134#issuecomment-637679871 We are ready. You can start working from this file https://github.com/apache/airflow/blob/master/airflow/api_connexion/endpoints/xcom_endpoint.py You can be inspired by

[GitHub] [airflow] mik-laj commented on issue #8135: API Endpoint - Logs

2020-06-02 Thread GitBox
mik-laj commented on issue #8135: URL: https://github.com/apache/airflow/issues/8135#issuecomment-637677894 @kaxil Hi. We have already started to implement. Do you still want to work on this ticket? Here is log_endpoint.py file. You should start working here.

[GitHub] [airflow] mik-laj commented on issue #8137: API Endpoint - Dag source

2020-06-02 Thread GitBox
mik-laj commented on issue #8137: URL: https://github.com/apache/airflow/issues/8137#issuecomment-637676872 @TobKed Hi. We have already started to implement. Do you still want to work on this ticket? Here is config_endpoint.py file. You should start working here.

[airflow] branch master updated: Add fudament for API based on connexion (#8149)

2020-06-02 Thread kamilbregula
This is an automated email from the ASF dual-hosted git repository. kamilbregula pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/master by this push: new 67379d1 Add fudament for API based on

[GitHub] [airflow] mik-laj merged pull request #8149: Add fudament for API based on connexion

2020-06-02 Thread GitBox
mik-laj merged pull request #8149: URL: https://github.com/apache/airflow/pull/8149 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

[GitHub] [airflow] mik-laj closed issue #8109: Basic integration Airflow and connexion

2020-06-02 Thread GitBox
mik-laj closed issue #8109: URL: https://github.com/apache/airflow/issues/8109 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] feluelle commented on issue #9071: Airflow scheduler works normally, fails with -D

2020-06-02 Thread GitBox
feluelle commented on issue #9071: URL: https://github.com/apache/airflow/issues/9071#issuecomment-637672611 Can't reproduce this issue either. @yyz940922 can you post your sql alchemy conn uri - at least protocol (e.g.`mysql://`) and params (e.g. `charset`), please?

[GitHub] [airflow] vbmade2000 removed a comment on issue #8136: API Endpoint - Config

2020-06-02 Thread GitBox
vbmade2000 removed a comment on issue #8136: URL: https://github.com/apache/airflow/issues/8136#issuecomment-637657583 Will it fall under /api/v1 only or it is a common one between all the api versions ? This is an

[GitHub] [airflow] jhtimmins commented on a change in pull request #9029: WIP: Remove hive dependency from unit tests

2020-06-02 Thread GitBox
jhtimmins commented on a change in pull request #9029: URL: https://github.com/apache/airflow/pull/9029#discussion_r434017381 ## File path: Dockerfile.ci ## @@ -121,14 +121,14 @@ RUN adduser airflow \ # https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=863199 RUN mkdir -pv

[GitHub] [airflow] blackw1ng closed issue #9104: s3_file_transform (and gcs) do not check exit code of subprocess correctly

2020-06-02 Thread GitBox
blackw1ng closed issue #9104: URL: https://github.com/apache/airflow/issues/9104 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

[GitHub] [airflow] blackw1ng commented on issue #9104: s3_file_transform (and gcs) do not check exit code of subprocess correctly

2020-06-02 Thread GitBox
blackw1ng commented on issue #9104: URL: https://github.com/apache/airflow/issues/9104#issuecomment-637667881 Sorry - i wasnt entirely sure, if this project accepts PRs without an issue - so i basically simultaneously did both. Thanks for the quick review (of this rather trivial fix) - i

[GitHub] [airflow] blackw1ng commented on pull request #9106: Fix handling of subprocess error handling in s3_file_transform and gcs

2020-06-02 Thread GitBox
blackw1ng commented on pull request #9106: URL: https://github.com/apache/airflow/pull/9106#issuecomment-637666961 Glad to help. It took us a while to figure it out - hopefully saves someone else some pains. This is an

[GitHub] [airflow] vbmade2000 commented on issue #8136: API Endpoint - Config

2020-06-02 Thread GitBox
vbmade2000 commented on issue #8136: URL: https://github.com/apache/airflow/issues/8136#issuecomment-637657583 Will it fall under /api/v1 only or it is a common one between all the api versions ? This is an automated

[GitHub] [airflow] samuelkhtu commented on issue #8525: SQLBranchOperator

2020-06-02 Thread GitBox
samuelkhtu commented on issue #8525: URL: https://github.com/apache/airflow/issues/8525#issuecomment-637654316 Thank you for the tips! This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] potiuk commented on pull request #9106: Fix handling of subprocess error handling in s3_file_transform and gcs

2020-06-02 Thread GitBox
potiuk commented on pull request #9106: URL: https://github.com/apache/airflow/pull/9106#issuecomment-637645346 THanks! @blackw1ng ! This is an automated message from the Apache Git Service. To respond to the message, please

[airflow] branch master updated (9e0ccde -> 17adcea)

2020-06-02 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 9e0ccde Produce less verbose output when building docker mount options (#9103) add 17adcea Fix handling of

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #9106: Fix handling of subprocess error handling in s3_file_transform and gcs

2020-06-02 Thread GitBox
boring-cyborg[bot] commented on pull request #9106: URL: https://github.com/apache/airflow/pull/9106#issuecomment-637645112 Awesome work, congrats on your first merged pull request! This is an automated message from the

[GitHub] [airflow] potiuk merged pull request #9106: Fix handling of subprocess error handling in s3_file_transform and gcs

2020-06-02 Thread GitBox
potiuk merged pull request #9106: URL: https://github.com/apache/airflow/pull/9106 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

[GitHub] [airflow] potiuk commented on issue #8525: SQLBranchOperator

2020-06-02 Thread GitBox
potiuk commented on issue #8525: URL: https://github.com/apache/airflow/issues/8525#issuecomment-637641497 Closed. For the future - it's enough to add Closes #ISSUE in the commit message so that the issue is closed automatically on merge :)

[GitHub] [airflow] potiuk edited a comment on issue #8525: SQLBranchOperator

2020-06-02 Thread GitBox
potiuk edited a comment on issue #8525: URL: https://github.com/apache/airflow/issues/8525#issuecomment-637641497 Closed. For the future - it's enough to add `Closes #ISSUE` in the commit message so that the issue is closed automatically on merge :)

[GitHub] [airflow] potiuk closed issue #8525: SQLBranchOperator

2020-06-02 Thread GitBox
potiuk closed issue #8525: URL: https://github.com/apache/airflow/issues/8525 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

  1   2   >