Skip to content

chore: demote some logs from INFO to more appropriate TRACE/DEBUG/WARN#533

Merged
dvdplm merged 12 commits into
mainfrom
dvdplm/chore/tweak-a-few-log-levels
May 20, 2026
Merged

chore: demote some logs from INFO to more appropriate TRACE/DEBUG/WARN#533
dvdplm merged 12 commits into
mainfrom
dvdplm/chore/tweak-a-few-log-levels

Conversation

@dvdplm
Copy link
Copy Markdown
Contributor

@dvdplm dvdplm commented Apr 20, 2026

Salvaged from PR #459.

Less INFO-level logs in the hope of making logs more useful while debugging.

@dvdplm dvdplm requested a review from a team as a code owner April 20, 2026 09:15
@cla-bot cla-bot Bot added the cla-signed The CLA has been signed. label Apr 20, 2026
Comment thread core/service/src/engine/threshold/service/epoch_manager.rs Outdated
eudelins-zama
eudelins-zama previously approved these changes Apr 20, 2026
Copy link
Copy Markdown
Contributor

@eudelins-zama eudelins-zama left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 20, 2026

Consolidated Tests Results 2026-05-18 - 18:02:38

Test Results

passed 7 passed

Details

tests 7 tests
clock not captured
tool junit-to-ctrf
build build-and-test arrow-right test-reporter link #2194
pull-request chore: demote some logs from INFO to more appropriate TRACE/DEBUG/WARN link #533

test-reporter: Run #2194

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
7 7 0 0 0 0 0 not captured

🎉 All tests passed!

Tests

View All Tests
Test Name Status Flaky Duration
k8s_test_crs_uniqueness 40.6s
k8s_test_insecure_keygen_encrypt_and_public_decrypt 1m 54s
k8s_test_insecure_keygen_encrypt_multiple_types 2m 10s
k8s_test_keygen_and_crs 1m 55s
k8s_test_keygen_uniqueness 4m 45s
k8s_test_centralized_insecure 58.5s
nightly_full_gen_tests_default_k8s_centralized_sequential_crs 1.8s

🍂 No flaky tests in this run.

Github Test Reporter by CTRF 💚

🔄 This comment has been updated

Comment thread core-client/src/lib.rs Outdated
Comment thread core/service/src/vault/storage/file.rs Outdated
Comment thread core/service/src/vault/storage/file.rs Outdated
Comment thread core/service/src/vault/storage/mod.rs Outdated
Comment thread core/service/src/vault/storage/mod.rs Outdated
Comment thread observability/src/conf.rs Outdated
Comment thread observability/src/conf.rs Outdated
Copy link
Copy Markdown
Member

@dd23 dd23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@dvdplm dvdplm merged commit 02205dc into main May 20, 2026
60 checks passed
@dvdplm dvdplm deleted the dvdplm/chore/tweak-a-few-log-levels branch May 20, 2026 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The CLA has been signed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants