Skip to content
This repository has been archived by the owner on Dec 26, 2022. It is now read-only.

Make our Rust logs a bit more readable by quieting noisy stuff #1984

Merged
merged 1 commit into from Sep 14, 2022

Conversation

wimax-grapl
Copy link
Contributor

Which issue does this PR correspond to?

Relates to launchbadge/sqlx#942 (comment)

What changes does this PR make to Grapl? Why?

Shush sqlx::query INFO logs
Downgrade noisy KafkaTopicScanner logs to TRACE

How were these changes tested?

CI

@codecov
Copy link

codecov bot commented Sep 14, 2022

Codecov Report

Base: 43.34% // Head: 43.35% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (4237297) compared to base (c4a1f77).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1984   +/-   ##
=======================================
  Coverage   43.34%   43.35%           
=======================================
  Files         427      427           
  Lines       10250    10250           
  Branches       23       23           
=======================================
+ Hits         4443     4444    +1     
+ Misses       5792     5791    -1     
  Partials       15       15           
Impacted Files Coverage Δ
pulumi/infra/log_levels.py 0.00% <ø> (ø)
src/rust/kafka/src/test_utils/topic_scanner.rs 0.00% <0.00%> (ø)
src/rust/sysmon-parser/src/util.rs 60.18% <0.00%> (+0.46%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@wimax-grapl wimax-grapl merged commit 7718480 into main Sep 14, 2022
@wimax-grapl wimax-grapl deleted the quieter branch September 14, 2022 20:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants