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.

Sign up for GitHub

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

Jump to bottom

Split fetched dbt Cloud logs to ensure they are not aggregated #60

Closed
anna-geller opened this issue Sep 25, 2023 · 2 comments
Closed

Split fetched dbt Cloud logs to ensure they are not aggregated #60

anna-geller opened this issue Sep 25, 2023 · 2 comments
Assignees
@Ben8t @mgabelle
Labels
area/backend Needs backend code changes area/plugin Plugin-related issue or feature request enhancement New feature or request

Comments

@anna-geller
Copy link
Member

anna-geller commented Sep 25, 2023 •
edited
Loading

Feature description

dbt Cloud logs are currently aggregated:

image

We should ensure that this ERROR message gets streamed as a separate error-level log

Related issue: kestra-io/plugin-airbyte#87
@anna-geller anna-geller added the enhancement New feature or request label Sep 25, 2023
@anna-geller anna-geller added this to the v0.15.0 milestone Dec 5, 2023
@anna-geller anna-geller modified the milestones: v0.15.0, v0.16.0 Jan 15, 2024
@anna-geller anna-geller modified the milestones: v0.16.0, v0.17.0 Mar 25, 2024
@anna-geller anna-geller modified the milestones: v0.17.0, v0.18.0 Apr 17, 2024
@anna-geller anna-geller removed this from the v0.18.0 milestone Jun 16, 2024
@anna-geller anna-geller added area/plugin Plugin-related issue or feature request area/backend Needs backend code changes labels Sep 4, 2024
@anna-geller
Copy link
Member Author

same here @Ben8t -- let's try to reproduce + coordinate a fix with Matt 🙏

@Ben8t
Copy link
Member

Ben8t commented Sep 17, 2024

I'm able to reproduce - syncing with Mat this afternoon 👍

mgabelle added a commit that referenced this issue Sep 23, 2024
@mgabelle
fix: aggregated logs
572c0b9 
enforce json log for DbtCli plugin commands

fix log level "warning" to "warn"

#60
@mgabelle mgabelle mentioned this issue Sep 23, 2024
Merged
mgabelle added a commit that referenced this issue Sep 23, 2024
@mgabelle
fix: aggregated logs
7accefa 
enforce json log for DbtCli plugin commands

fix log level "warning" to "warn"

#60
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ben8t Ben8t

@mgabelle mgabelle

Labels
area/backend Needs backend code changes area/plugin Plugin-related issue or feature request enhancement New feature or request
Projects
Issues
Status: Done
Plugins and Documentation
Status: Planned
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Ben8t @mgabelle @anna-geller