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 2022/10/20 08:15:48 UTC

[GitHub] [airflow] selvakumar-arumugapandian opened a new issue, #27157: JSON Linter in Variables

selvakumar-arumugapandian opened a new issue, #27157:
URL: https://github.com/apache/airflow/issues/27157

   ### Description
   
   # JSON Linter in Variables 
   
   > When user adds an object (JSON format) as variable, there should be option enabled to validate the object or validate automatically.
   
   ## For example:
   I have a variable named ***"my_var_group"***, with the following value
   
   ```json
   {
   "dag_prefix": "my-prefix",
   "dag_retry": 5
   }
   ```
   Imagine, I am using this variable "my_var_group" in 500+ DAGs and any mistake in the JSON fails to load all the 500+ DAGs. 
   
   So, A feature which does automatic linting of JSON object (By throwing error during variable creation/update) shall ensure that "error" prone JSON is not added.
   
   
   ### Use case/motivation
   
   Real-time experience in an environment where we are running more than 1000+ DAGs. 
   
   ### Related issues
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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-unsubscribe@airflow.apache.org.apache.org

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

[GitHub] [airflow] bbovenzi commented on issue #27157: JSON Linter in Variables

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

   Good idea. Feel free to copy over the JSON linter we use for a dag run config: https://github.com/apache/airflow/blob/main/airflow/www/templates/airflow/trigger.html


-- 
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-unsubscribe@airflow.apache.org

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

[GitHub] [airflow] boring-cyborg[bot] commented on issue #27157: JSON Linter in Variables

Posted by GitBox <gi...@apache.org>.
boring-cyborg[bot] commented on issue #27157:
URL: https://github.com/apache/airflow/issues/27157#issuecomment-1285122798

   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 message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

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

[GitHub] [airflow] selvakumar-arumugapandian commented on issue #27157: JSON Linter in Variables

Posted by GitBox <gi...@apache.org>.
selvakumar-arumugapandian commented on issue #27157:
URL: https://github.com/apache/airflow/issues/27157#issuecomment-1287797032

   @bbovenzi , Thank you. I shall check this out. 


-- 
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-unsubscribe@airflow.apache.org

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