-
Notifications
You must be signed in to change notification settings - Fork 9
Prefect dbt 0.2.6 coroutine error #104
Comments
Do you have this wrapped in a flow? Also are you working in notebook? Here's something you can try in notebook / script.
|
No the code is directly run within a kubernetes pod. Locally it seems to work |
Can you try to see if this branch potentially fixes the issue? I made the base class method also sync compatible, but unsure if it'll help. https://github.com/PrefectHQ/prefect-dbt/pull/105/files |
I have the same issue sadly :/
|
I suspect it's due to PrefectHQ/prefect-gcp#80 not yet released. Could you try pulling prefect-gcp main? e.g. in requirements.txt |
Same issue with the prefect-gcp version |
To clarify: "No the code is directly run within a kubernetes pod." Is your code wrapped in a flow? |
Sure all the code is wrapped in a flow and triggered with prefect |
Thanks for confirming and I appreciate your help testing! I made another commit on https://github.com/PrefectHQ/prefect-dbt/pull/105/files Can you help me test that again? If this doesn't work, I think we might have to make it sync again. |
@ahuang11 I'm so sorry It still doesn't work :( with same error |
Thanks for testing. Can you provide a minimal reproducible example that we can try on our end? |
hey @ahuang11 The 0.2.6 is broken for us because of the co routine error It can be hard to reproduce the error because it happens only in a Kubernetes context (it works well locally)
If it's too hard for you to replicate this, we can plan a call or even let you use my mac remotely |
Thanks for that; can you try #111 to see if this fixes the issue? I dropped the async. |
@ahuang11 yes now it works perfectly with the last fix |
It it possible to merge and release a 0.2.8 package ? :) |
Yes I'll try to get the PR ready soon! |
@lucienfregosibodyguard by any chance, are you loading a DbtCliProfile like shown in #114? |
Here is my DbtCliProfile Definition
let me know if it's not the up to date version to load DbtCliProfile |
Okay so you don't use |
No I can try with save and load |
It seems better actually i don't have the coroutine error, it starts the DBT command and I got this
Which is a "normal" error that should be solved in your PR |
Hi @ahuang11 could you release a new tag to easily test this and update my code ? |
Actually this isn't fixed yet. Will try to get a new release soon... |
@ahuang11 could we make a new tag with the last fix ? |
Done! |
Thaaaaaaaks it works like a charm awesome |
Hi @ahuang11
Following this issue Sadly it's still doesn't work, got a new error
I guess it's related to the async function definition but don't know how to fix it
The text was updated successfully, but these errors were encountered: