The GitHub Actions job "Tests (AMD)" on 
airflow.git/fix/log-filename-none-logical-date-68075 has succeeded.
Run started by GitHub user Vamsi-klu (triggered by Vamsi-klu).

Head commit for run:
c4c54d4cd77cbc99da73a6253127c2d326a9fc53 / Ramachandra Nalam 
<[email protected]>
Render log_filename_template the same way the log reader resolves it

Address review feedback on PR #68149: drop the invented ``ts_nodash`` key
and the try/except fallback to the default template. The scheduler-side
renderer now mirrors ``FileTaskHandler._render_filename`` exactly -- it
resolves ``logical_date or run_after`` on both the Jinja and f-string paths,
exposes ``ts``, and passes ``run_id``/``data_interval_start``/
``data_interval_end`` -- so the path the worker writes to and the path the
UI reads from resolve to the same string for runs without a logical_date.

``{{ ti.logical_date }}`` still renders to "None" (ti is passed unchanged),
and a template that calls a method on a None logical_date
(``ti.logical_date.strftime(...)``) is left to raise as a user error rather
than silently falling back. Tests now assert write-path == read-path parity.

Report URL: https://github.com/apache/airflow/actions/runs/27668554387

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to