apache / airflow

Apache Airflow - A platform to programmatically author, schedule, and monitor workflows
https://airflow.apache.org/
Apache License 2.0
37.29k stars 14.34k forks source link

Using a period for the DAG name (dag_id) prevents scheduling #310

Closed fdion closed 8 years ago

fdion commented 9 years ago

The dag gets registered, but the namespace is mismatched and the DAG never gets scheduled and shows no status.

As an example:

dag = DAG(dag_id='second_dag.py', default_args=args)

Will not get scheduled, while dag = DAG(dag_id='second_dag', default_args=args) will.

toryhaavik commented 9 years ago

I think this is also the reason why we see the status bubbles disappearing from some of our DAGs on the frontpage. Chrome's console tells me d3 says 'svg#dag.with.period' is not a valid selector

image

artwr commented 9 years ago

I think this was fixed in https://github.com/airbnb/airflow/pull/468

toryhaavik commented 9 years ago

ah, i think you're right! i'll look for it in the next release. thanks!