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

Disable CI test logs #4092

Merged
merged 2 commits into from
Feb 4, 2025
Merged

Disable CI test logs #4092

merged 2 commits into from
Feb 4, 2025

Conversation

ss-es
Copy link
Contributor

@ss-es ss-es commented Jan 28, 2025

Disables CI test logs; discussion in #hotshot on Zulip

@ss-es ss-es requested a review from bfish713 as a code owner January 28, 2025 16:33
@bfish713
Copy link
Collaborator

Can we instead go to error and downgrade the error logs that get spammed. I think it's just:

HotShot/crates/hotshot/src/tasks/mod.rs

Line 164 in 69bb9e4

tracing::error!("Failed to receive message: {:?}", e);

@ss-es ss-es merged commit 6ef49d1 into main Feb 4, 2025
24 checks passed
@ss-es ss-es deleted the ss/disable-ci-logs branch February 4, 2025 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lukaszrzasik lukaszrzasik lukaszrzasik approved these changes

@bfish713 bfish713 Awaiting requested review from bfish713 bfish713 is a code owner

Assignees

@jparr721 jparr721

@QuentinI QuentinI

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ss-es @bfish713 @lukaszrzasik @jparr721 @QuentinI