ashb commented on a change in pull request #19410:
URL: https://github.com/apache/airflow/pull/19410#discussion_r759254785
##########
File path: airflow/operators/trigger_dagrun.py
##########
@@ -156,6 +166,11 @@ def execute(self, context: Dict):
else:
raise e
+ # Store the execution date from the dag run (either created or found
above) to
+ # be used when creating the extra link on the webserver.
+ ti = context['task_instance']
+ ti.xcom_push(key=XCOM_EXECUTION_DATE_ISO,
value=dag_run.execution_date.isoformat())
Review comment:
Not _right_ now, but I am slowly working towards making Airflow be able
to have multiple runs for the _same_ date, so by adding this now it future
proofs us and means we can change the URL without having to worry about
backwards compat of "did we push run_id or not"
--
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]