ferruzzi commented on PR #30496:
URL: https://github.com/apache/airflow/pull/30496#issuecomment-1505698125

   In [this 
test](https://github.com/apache/airflow/blob/main/tests/models/test_dagrun.py#L891)
 we are specifically checking to make sure that there are no tags in the 
`dagrun.{dag.dag_id}.first_task_scheduling_delay` metric, but that they do 
exist in the `dagrun.first_task_scheduling_delay` metric.   Is that actually 
the desired behavior or is that a side effect of the test that wasn't intended? 
 
   
   I was planning to add tagging to every metric, but perhaps I overdid it and 
it's not wanted or needed on some?


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