eladkal commented on code in PR #53176:
URL: https://github.com/apache/airflow/pull/53176#discussion_r2200973112


##########
airflow-core/docs/core-concepts/dags.rst:
##########
@@ -685,6 +685,22 @@ This is especially useful if your tasks are built 
dynamically from configuration
     Here's a [url](www.airbnb.com)
     """
 
+.. _concepts:dags-ownership:
+
+Ownership
+~~~~~~~~~
+
+In addition to these documentation and notes, you are also able to link a DAG 
with a set of owners.
+This way the DAG users may be able to reach out the right set of people when 
needed.
+
+This ownership is split in two parts:
+
+1. at the :ref:`Task level <concepts:task-owner>` through the ``owner`` 
definition;
+2. at the DAG level to customize the UI through the ``owner_links`` definition.
+
+Each DAG collects the set of owners from its task set, and display it on the 
DAGs view UI.
+Any owner matching an item defined in the ``owner_links`` dictionary is 
converted to a clickable link (that may be a instant messaging handle or a 
mailto link).

Review Comment:
   owner_links was not ported to 3.0
   which is why we don't have
   
https://airflow.apache.org/docs/apache-airflow/3.0.0/howto/add-owner-links.html
   
   but we do have
   
https://airflow.apache.org/docs/apache-airflow/2.11.0/howto/add-owner-links.html
   
   we should revive the doc as stand alone.
   
   I don't think owner links needs to be explained in core-concept. It is not a 
core concept.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to