# integration-dbt

Nicolas Parot Alvarez

08/01/2023, 9:19 AM
I think the change in
functions to use
dbt build
by default instead of
dbt run
is not very intuitive. The argument is that it will be more natural for someone following the tutorials, but I don't really understand it. Running seeds does make sense, otherwise models depending on seeds may not be able to materialize, but I don't see the logic behind running "tests" and "snapshots" from a function that is supposed to load DBT definitions into Dagster objects. Was the goal to run seeds at loading time and snapshots/tests are just a benign negative side effect? (Reflection triggered by laziness to edit our code to add
❤️ 2


08/03/2023, 3:57 PM
Our thought process behind this was that if you had seeds in the first place, you would have been running dbt build in
in the first place. Otherwise, your downstream models wouldn’t be able to run in the first place.

Nicolas Parot Alvarez

08/03/2023, 4:24 PM
We prefer to run dbt seed independently, because we may frequently refresh our Dagster definitions during testing, and we don't want to have it trigger an unnecessary seed run every time. Currently, we're running the seeds with
on a daily schedule, which is a bit excessive, but very practical to not have to bother with it during the deployment of a new version with a seed update. I think including seeds during the loading of assets makes sense as a step in a new deployment. But ideally there would be something in place to only run them if they were modified, maybe something like could be used ? Besides, the running of tests and snapshots doesn't seem relevant to loading assets into Dagster, at least not by default.