[GitHub] [airflow] potiuk commented on issue #20053: Add Prometheus Provider

[GitBox <gi...@apache.org>]

potiuk commented on issue #20053:
URL: https://github.com/apache/airflow/issues/20053#issuecomment-986274931


   Just to comment for the discussion related - we are looking into open telemetry support during our Outreachy internship[ (https://github.com/apache/airflow/projects/14). This might take quite a while to explore the options, but is likely to turn into a long-term solution, provide that we produce an AIP and it gets approved. 
   
   I understand @subkanthi  that you want to work on it? 
   
   I thought about it and it might be benefitial for everyone to see how firect Prometeus integratiom might work (experimentally  - to do in parallel. If somone is willing to do it, possibly yes if wills to spend the time and focus on that, we should treat it (same as Open Telemetry) as "Experimental" feature or POC possibly also to build a knowledge on the integration and see what challenges it brings and seeing if the OT approch might bring the same level of integration.
   
   I m not sure however if we want to merge it, if by the time this one is complete (including likely Helm chart integration etc.) we will see that we can do similar integration level with Open Telemetry.
   
   See the comments here: https://github.com/apache/airflow/issues/19972#issuecomment-986090147 - also @Melodie97  - you might be interested to take a look at the progress of this if there is a PR by @subkanthi so that it coudl be taken as inspiration (both ways most likely).
   
   @subkanthi - does it sound like a good approach?


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