dungdm93 opened a new pull request #11009:
URL: https://github.com/apache/airflow/pull/11009


   In k8s, it's highly recommended to prefix labels/annotations by domains 
name. like [recommended app 
labels](https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/)
   ```
   apiVersion: apps/v1
   kind: StatefulSet
   metadata:
     labels:
       app.kubernetes.io/name: mysql
       app.kubernetes.io/instance: mysql-abcxzy
       app.kubernetes.io/version: "5.7.21"
       app.kubernetes.io/component: database
       app.kubernetes.io/part-of: wordpress
       app.kubernetes.io/managed-by: helm
   ```
   Or in helm, we have `helm.sh/chart: NAME-VERSION`.
   
   So, in Airflow's `KubernetesPodOperator`, I also suggest that prefix all 
airflow managed labels with `airflow.apache.org/`:
   ```
   dag_id          -> airflow.apache.org/dag_id
   task_id         -> airflow.apache.org/task_id
   execution_date  -> airflow.apache.org/execution_date
   try_number      -> airflow.apache.org/try_number
   parent_dag_id   -> airflow.apache.org/parent_dag_id
   ```


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


Reply via email to