-
Notifications
You must be signed in to change notification settings - Fork 976
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
[CT-3568] print
/ no_print
in profiles.yml
has no effect
#4776
Comments
Hey @dbeatty10, I came across a possibly related Slack convo. It maybe needs to be wrapped in a macro until it's fixed in Core? |
Oooh, good call about that Slack convo 🧠 Digging into that convo deeper, it seems it is mainly related to other issues like these:
Looking at this more closely, I think this might be a bug in dbt-core, so I'm going to transfer this issue to that repo. |
print
/ no_print
in profiles.yml
has no effect in dbt v1.7print
/ no_print
in profiles.yml
has no effect in dbt v1.7
See also: dbt-labs/dbt-core#7036 (which also mentions |
After further research, I think we just need an update to the docs. Opened #4775 to address this. |
print
/ no_print
in profiles.yml
has no effect in dbt v1.7print
/ no_print
in profiles.yml
has no effect
[Preview](https://docs-getdbt-com-git-dbeatty-print-no-print-dbt-labs.vercel.app/reference/global-configs/print-output#suppress-print-messages-in-stdout) resolves #4776 ## What are you changing in this pull request and why? I checked from v1.7 back to v1.1, and I couldn't get either of these to work in any version: Up to 1.5: ```yaml config: no_print: true ``` 1.5 and after: ```yaml config: print: false ``` However, the `DBT_NO_PRINT` / `DBT_PRINT` environment variables and `--no-print` and `--print` CLI flags _do_ work. So I think the code example for `profiles.yml` was just accidentally introduced in the following PRs, and we should remove it because it's not actually an option for any versions. - #1319 - #3134 For additional context, see: - [Upgrading to v1.1](https://docs.getdbt.com/docs/dbt-versions/core-upgrade/upgrading-to-v1.1#advanced-and-experimental-functionality) - [Upgrading to v1.5](https://docs.getdbt.com/docs/dbt-versions/core-upgrade/upgrading-to-v1.5#behavior-changes) - #1102 - dbt-labs/dbt-core#4701 - #3122 - #3332 ## Checklist - [x] Review the [Content style guide](https://github.com/dbt-labs/docs.getdbt.com/blob/current/contributing/content-style-guide.md) so my content adheres to these guidelines. - [x] For [docs versioning](https://github.com/dbt-labs/docs.getdbt.com/blob/current/contributing/single-sourcing-content.md#about-versioning), review how to [version a whole page](https://github.com/dbt-labs/docs.getdbt.com/blob/current/contributing/single-sourcing-content.md#adding-a-new-version) and [version a block of content](https://github.com/dbt-labs/docs.getdbt.com/blob/current/contributing/single-sourcing-content.md#versioning-blocks-of-content).
Here's the relevant documentation about the
print
global config:I tried specifying
print: False
withinprofiles.yml
as described in the docs, but it didn't have any effect.So maybe it was fully deprecated sometime between dbt v1.5 and v1.7 but we didn't yet update the docs? Or maybe it was accidentally stopped working during the
click
migration (in dbt-labs/dbt-core#7086)?Originally posted by @dbeatty10 in #4740 (comment)
To do
Try out the following within each of v1.5, v1.6, and v1.7. Figure out if it was fully deprecated or not and update the docs accordingly.
models/my_model.sql
profiles.yml
:*** Parsing ...
shows up in the logs or not:The text was updated successfully, but these errors were encountered: