[go: up one dir, main page]

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

improved some logging in workflow activities #8278

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

filintod
Copy link
Contributor
@filintod filintod commented Nov 13, 2024

Description

Change a few Debug logs to Info logs in workflow activities to better understand workflows without a full debug logging.

Issue reference

Please reference the issue this PR will close: #8279

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

Signed-off-by: Filinto Duran <filinto@diagrid.io>
@filintod filintod requested review from a team as code owners November 13, 2024 23:10
Signed-off-by: Filinto Duran <filinto@diagrid.io>
@filintod filintod force-pushed the filinto/wfengine-enhanced-logging branch from ac6f141 to 36c5b28 Compare November 14, 2024 13:03
Signed-off-by: Filinto Duran <filinto@diagrid.io>
Signed-off-by: Filinto Duran <filinto@diagrid.io>
@filintod
Copy link
Contributor Author

@JoshVanL please take another look when you have time. thx

cmd/daprd/options/options.go Outdated Show resolved Hide resolved
Co-authored-by: Josh van Leeuwen <me@joshvanl.dev>
Signed-off-by: Filinto Duran <duranto@gmail.com>
Signed-off-by: Filinto Duran <filinto@diagrid.io>
@filintod filintod force-pushed the filinto/wfengine-enhanced-logging branch from c6b6362 to 86df23d Compare November 14, 2024 16:38
@cicoyle cicoyle added the autoupdate DaprBot will keep the Pull Request up to date with master branch label Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autoupdate DaprBot will keep the Pull Request up to date with master branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Workflow logging is too light to be useful without going with a full tracing solution
5 participants