-
Notifications
You must be signed in to change notification settings - Fork 427
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Added Several Datasources Part 2 #622
feat: Added Several Datasources Part 2 #622
Conversation
/ok-to-test sha=a0fc150 |
Integration tests failure for a0fc150 |
@alldoami Streams Acceptance Test fixed. Not sure why the warehouses, resource_monitors and storage_integrations are failing. Locally against my Snowflake account it works (see screenshot below), can you take a look if it fails again? |
/ok-to-test sha=0ac99db |
Integration tests failure for 0ac99db |
@alldoami Can you take a look why the warehouses, resource_monitors and storage_integrations are failing? Locally against my Snowflake account it works |
@alldoami Did some research and it seems that the depends_on in the acceptance tests is triggering it, maybe it works locally on my side because I'm using TF 1.02? I found that there is a Test parameter for this |
/ok-to-test sha=94a8884 |
Integration tests failure for 94a8884 |
@alldoami I was able to reproduce this locally. These tests are failing indeed due to the |
I'll upgrade tf for testing! Could you get rid of your workaround 😅 |
@alldoami Done! Removed workaround |
/ok-to-test sha=6887b89 |
Integration tests failure for 6887b89 |
@alldoami Can you run it again? Seems it failing for no apparent reason? |
/ok-to-test sha=6887b89 |
Integration tests success for 6887b89 |
Part 2 of #618 (Sorry in advanced for the big PR)
Implemented following DataSources (which can be later on used to iterate over other resources):
Test Plan
References
https://docs.snowflake.com/en/sql-reference/sql/show.html
*