Skip to content
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

Improve tracking error logging message #5211

Conversation

NicolasPA
Copy link
Contributor

@NicolasPA NicolasPA commented May 3, 2022

Resolves #5197

Description

The error message logged when tracking posting fails now makes it more
obvious that it is related to DBT's internal anonymous usage
statistics, and explains how to disable it.

As tracking failure doesn't real concern most users, we also reduce the
logging level to "DEBUG" so it doesn't appear during normal use.

The use case that triggered this change proposal is using DBT without
an internet connection and wondering what this error meant.

Checklist

The error message logged when tracking posting fails now makes it more
obvious that it is related to DBT's internal anonymous usage
statistics, and explains how to disable it.

As tracking failure doesn't real concern most users, we also reduce the
logging level to "DEBUG" so it doesn't appear during normal use.

Issue dbt-labs#5197

Signed-off-by: NicolasPA <[email protected]>
@NicolasPA NicolasPA requested a review from a team May 3, 2022 22:59
@NicolasPA NicolasPA requested a review from a team as a code owner May 3, 2022 22:59
@cla-bot cla-bot bot added the cla:yes label May 3, 2022
Signed-off-by: NicolasPA <[email protected]>
@NicolasPA NicolasPA requested a review from a team as a code owner May 3, 2022 23:02
@gshank gshank closed this May 4, 2022
@gshank gshank reopened this May 4, 2022
@gshank gshank merged commit fb5bb7f into dbt-labs:main May 4, 2022
@NicolasPA NicolasPA deleted the npa/feature/improve_tracking_error_loggin_message_CT-586 branch May 6, 2022 13:41
agoblet pushed a commit to BigDataRepublic/dbt-core that referenced this pull request May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CT-586] [Feature] Improve tracking error message
2 participants