You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/01/18 14:42:21 UTC

[GitHub] [airflow] tandelDipak opened a new issue #13744: REST API Connection Endpoint doesn't return the extra field in response

tandelDipak opened a new issue #13744:
URL: https://github.com/apache/airflow/issues/13744


   <!--
   
   Welcome to Apache Airflow!  For a smooth issue process, try to answer the following questions.
   Don't worry if they're not all applicable; just try to include what you can :-)
   
   If you need to include code snippets or logs, please put them in fenced code
   blocks.  If they're super-long, please use the details tag like
   <details><summary>super-long log</summary> lots of stuff </details>
   
   Please delete these comment blocks before submitting the issue.
   
   -->
   
   <!--
   
   IMPORTANT!!!
   
   PLEASE CHECK "SIMILAR TO X EXISTING ISSUES" OPTION IF VISIBLE
   NEXT TO "SUBMIT NEW ISSUE" BUTTON!!!
   
   PLEASE CHECK IF THIS ISSUE HAS BEEN REPORTED PREVIOUSLY USING SEARCH!!!
   
   Please complete the next sections or the issue will be closed.
   These questions are the first thing we need to know to understand the context.
   
   -->
   
   **Apache Airflow version**:
   Apache Airflow: 2.0.0
   
   **Kubernetes version (if you are using kubernetes)** (use `kubectl version`):
   
   **Environment**:
   
   - **Cloud provider or hardware configuration**:
   - **OS** (e.g. from /etc/os-release): 
   Distributor ID: Ubuntu
   Description:    Ubuntu 18.04.5 LTS
   Release:        18.04
   Codename:       bionic
   - **Kernel** (e.g. `uname -a`):
   Linux Personal 5.4.0-62-generic #70~18.04.1-Ubuntu SMP Tue Jan 12 17:18:00 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
   - **Install tools**:
   - **Others**:
   
   **What happened**:
   
   <!-- (please include exact error messages if you can) -->
   REST API doesn't return the **extra** field of the connection in the response.
   
   **What you expected to happen**:
   
   <!-- What do you think went wrong? -->
   It should return all the fields as shown in the documentation.
   ![Screenshot from 2021-01-18 20-10-09](https://user-images.githubusercontent.com/15157792/104928902-38224280-59c9-11eb-814a-3f359c0796f2.png)
   
   
   **How to reproduce it**:
   <!---
   
   As minimally and precisely as possible. Keep in mind we do not have access to your cluster or dags.
   
   If you are using kubernetes, please attempt to recreate the issue using minikube or kind.
   
   ## Install minikube/kind
   
   - Minikube https://minikube.sigs.k8s.io/docs/start/
   - Kind https://kind.sigs.k8s.io/docs/user/quick-start/
   
   If this is a UI bug, please provide a screenshot of the bug or a link to a youtube video of the bug in action
   
   You can include images using the .md style of
   ![alt text](http://url/to/img.png)
   
   To record a screencast, mac users can use QuickTime and then create an unlisted youtube video with the resulting .mov file.
   
   --->
   
   Create one connection with id  **leads_ec2** and define values as shown in the screenshot. 
   
   ![Screenshot from 2021-01-18 19-49-48](https://user-images.githubusercontent.com/15157792/104927763-d44b4a00-59c7-11eb-9139-da83d7098b3c.png)
   
   Now call the below API endpoint to get the connection details. And as shown in the screenshot it doesn't include the extra field in the response.
   **API Endpoint** : `http://localhost:8000/api/v1/connections/leads_ec2`
    
   ![Screenshot from 2021-01-18 19-50-07](https://user-images.githubusercontent.com/15157792/104928126-491e8400-59c8-11eb-80d5-84b52e812d8e.png)
   
   **How often does this problem occur? Once? Every time etc?**:
   
   <!--
   
   How often does this problem occur? Once? Every time etc?
   
   Any relevant logs to include? Put them here in side a detail tag:
   <details><summary>x.log</summary> lots of stuff </details>
   
   -->
   Same for other connection_id. It doesn't return the extra field in the response.
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] tandelDipak edited a comment on issue #13744: REST API Connection Endpoint doesn't return the extra field in response

Posted by GitBox <gi...@apache.org>.
tandelDipak edited a comment on issue #13744:
URL: https://github.com/apache/airflow/issues/13744#issuecomment-762330087


   @potiuk Thank you for your support.
   I tried setting the below values but none of them worked.
   - '{ "test": "a"}'
   - '{}'
   - ' '
   - "{ "test": "a"}"
   - "{}"
   - " "
   - { "test": "a"}
   - Kept this field empty
   
   At least it should return empty value when field is kept empty.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] potiuk edited a comment on issue #13744: REST API Connection Endpoint doesn't return the extra field in response

Posted by GitBox <gi...@apache.org>.
potiuk edited a comment on issue #13744:
URL: https://github.com/apache/airflow/issues/13744#issuecomment-762301481


   The Extra field should be a dictionary. Add '{ "test": "a"}' for example. I am closing it for now. Please reopen if it does not solve your problem.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] mik-laj commented on issue #13744: REST API Connection Endpoint doesn't return the extra field in response

Posted by GitBox <gi...@apache.org>.
mik-laj commented on issue #13744:
URL: https://github.com/apache/airflow/issues/13744#issuecomment-762780931


   I confirm this bug. I am working on a fix now.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] mik-laj commented on issue #13744: REST API Connection Endpoint doesn't return the extra field in response

Posted by GitBox <gi...@apache.org>.
mik-laj commented on issue #13744:
URL: https://github.com/apache/airflow/issues/13744#issuecomment-762789676


   @tandelDipak Feel free. Here is a similar PR: https://github.com/apache/airflow/pull/13684


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] potiuk closed issue #13744: REST API Connection Endpoint doesn't return the extra field in response

Posted by GitBox <gi...@apache.org>.
potiuk closed issue #13744:
URL: https://github.com/apache/airflow/issues/13744


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] tandelDipak commented on issue #13744: REST API Connection Endpoint doesn't return the extra field in response

Posted by GitBox <gi...@apache.org>.
tandelDipak commented on issue #13744:
URL: https://github.com/apache/airflow/issues/13744#issuecomment-762330087


   @potiuk Thank you for your support.
   I tried setting the below values but none of them worked.
   - '{ "test": "a"}'
   - '{}'
   - ' '
   - "{ "test": "a"}"
   - "{}"
   - " "
   - { "test": "a"}
   - Kept this field empty
   
   At least it should return empty value when filed is kept empty.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] potiuk commented on issue #13744: REST API Connection Endpoint doesn't return the extra field in response

Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #13744:
URL: https://github.com/apache/airflow/issues/13744#issuecomment-762301481


   The Extra field should be a dictionary. 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] mik-laj closed issue #13744: REST API Connection Endpoint doesn't return the extra field in response

Posted by GitBox <gi...@apache.org>.
mik-laj closed issue #13744:
URL: https://github.com/apache/airflow/issues/13744


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] mik-laj closed issue #13744: REST API Connection Endpoint doesn't return the extra field in response

Posted by GitBox <gi...@apache.org>.
mik-laj closed issue #13744:
URL: https://github.com/apache/airflow/issues/13744


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] tandelDipak commented on issue #13744: REST API Connection Endpoint doesn't return the extra field in response

Posted by GitBox <gi...@apache.org>.
tandelDipak commented on issue #13744:
URL: https://github.com/apache/airflow/issues/13744#issuecomment-762787003


   @mik-laj  can I work on this? I can not fix it quickly as you but if it's not urgent I would like to contribute.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] potiuk edited a comment on issue #13744: REST API Connection Endpoint doesn't return the extra field in response

Posted by GitBox <gi...@apache.org>.
potiuk edited a comment on issue #13744:
URL: https://github.com/apache/airflow/issues/13744#issuecomment-762301481


   The Extra field should be a dictionary. Add '{ "test": "a"}' for example. I am closing it for now. Please comment if it does not solve your problem.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org