sul-dlss / libsys-airflow

Airflow DAGS for migrating and managing ILS data into FOLIO along with other LibSys workflows
Apache License 2.0
5 stars 0 forks source link

poll for 979 dag failing #1441

Open shelleydoljack opened 3 hours ago

shelleydoljack commented 3 hours ago

See log for a failed dag run: https://sul-libsys-airflow-prod.stanford.edu/dags/poll_for_digital_bookplate_979s_email/grid?run_id=manual__2024-11-16T01%3A46%3A19.136056%2B00%3A00&execution_date=2024-11-16+01%3A46%3A19.136056%2B00%3A00&tab=logs&dag_run_id=manual__2024-11-16T01%3A46%3A19.136056%2B00%3A00&task_id=poll_979_dags

File "/opt/airflow/libsys_airflow/plugins/shared/utils.py", line 29, in dag_run_url
    return f"{airflow_url}dags/{dag_run.dag.dag_id}/grid?{params}"
                                ^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'dag_id'
shelleydoljack commented 2 hours ago

I think the problem is that for digital bookplates, the kwargs are passed in param dag_runs but the function dag_run_url is looking for kwargs dag_run. And OCLC reports use the utility and use the kwargs dag_run so a reconciliation needs to happen. I guess the init of DAG979Sensor needs updating?