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

✨ Improve logging in core components #3332

Draft
wants to merge 58 commits into
base: main
Choose a base branch
from
Draft
Changes from 1 commit
Commits
Show all changes
58 commits
Select commit Hold shift + click to select a range
815a0f3
:art: Refactor `register_plugin` steps for maintainability
ff137 Nov 6, 2024
ce9f8b7
:art: improve logging in register_package
ff137 Nov 6, 2024
3e5088e
:art: Rename method and remove redundant main call
ff137 Nov 6, 2024
59e4951
:art:
ff137 Nov 6, 2024
02011a8
:loud_sound: Add debug logging to Context Builder
ff137 Nov 7, 2024
1263191
:art: Deduplicate registering list of plugins
ff137 Nov 7, 2024
24c1426
:art:
ff137 Nov 7, 2024
8264688
:loud_sound: Add debug logging to init_context
ff137 Nov 7, 2024
b539ab0
:loud_sound: Add debug logging to load_protocol_version
ff137 Nov 7, 2024
595d391
:loud_sound: Add debug logging to load_protocols
ff137 Nov 7, 2024
b1a73a0
:loud_sound: Add debug logging to register_admin_routes
ff137 Nov 7, 2024
5e199d1
:loud_sound: Add debug logging to register_protocol_events
ff137 Nov 7, 2024
9cf03f6
:loud_sound: Add debug logging to post_process_routes
ff137 Nov 7, 2024
6330701
:art: Replace print statements with info logs
ff137 Nov 7, 2024
6a8febd
:art: Replace almost all print statements with info logs
ff137 Nov 7, 2024
8f46fdb
:loud_sound: Expand logging for Conductor setup
ff137 Nov 7, 2024
edde403
:art: Correct to uphold previous "Module doesn't exist" log behaviour
ff137 Nov 7, 2024
cc5ceab
:white_check_mark: Fix test by using assertLogs instead of capturing …
ff137 Nov 7, 2024
44e9807
Revert ":art: Rename method and remove redundant main call"
ff137 Nov 7, 2024
cfec6a6
:art:
ff137 Nov 7, 2024
500e4b2
:white_check_mark: Fix test by adding __name__ to mock object
ff137 Nov 7, 2024
5134d16
:art:
ff137 Nov 7, 2024
aa0765a
:sparkles: Add custom trace log level
ff137 Nov 7, 2024
190cc28
:art: Ensure trace level can only be added once
ff137 Nov 7, 2024
63ca170
:art: Add Logger
ff137 Nov 7, 2024
3641e2f
:art: Update newly added logs to be trace level instead of debug
ff137 Nov 7, 2024
bf1295f
:art: Update newly added logs to be trace level instead of debug
ff137 Nov 7, 2024
926eab1
:loud_sound: Expand logging for Conductor start and stop
ff137 Nov 7, 2024
32bb6b4
:white_check_mark: Add unit tests for adding trace level functionality
ff137 Nov 7, 2024
9d0275a
:white_check_mark: Reset the logging states before and after TestLogg…
ff137 Nov 7, 2024
a53b949
:art:
ff137 Nov 7, 2024
e77cf3f
:test_tube: Debug log assertion test failing sporadically
ff137 Nov 7, 2024
4f63240
:white_check_mark: Modify log assertion to be more robust
ff137 Nov 7, 2024
0ff9da4
:art:
ff137 Nov 7, 2024
9058784
:bug: Handle case that load_module return type is Optional
ff137 Nov 7, 2024
ebfbaaf
:loud_sound: Expand logging for ClassLoader
ff137 Nov 7, 2024
4312905
:loud_sound: Log a warning if resource not found, instead of passing
ff137 Nov 11, 2024
9d347bb
:art:
ff137 Nov 11, 2024
06ceea9
:art: Enable trace logs for remaining modules
ff137 Nov 11, 2024
838adcd
:bug: Fix registering and loading of modules
ff137 Nov 12, 2024
f342646
:loud_sound: Expand logging for Ledger Configuration
ff137 Nov 12, 2024
683c1c7
Merge branch 'main' into improve-logging
ff137 Nov 27, 2024
51ba3e7
:rewind: Revert `trace` log changes, and move to debug level
ff137 Nov 27, 2024
523d429
:sparkles: Add caching to ClassLoader.load_module
ff137 Nov 27, 2024
a2b32ce
:art: Improve log
ff137 Nov 27, 2024
a7e0a69
:art: Reduce log verbosity
ff137 Nov 27, 2024
35e6445
:sparkles: Add `lru_cache` to load_class and set maxsize
ff137 Nov 27, 2024
79c58f2
Merge branch 'main' into improve-logging
ff137 Nov 27, 2024
31782d4
:rewind: Revert addition of lru caching
ff137 Dec 2, 2024
15d4631
:art: Update logs / reduce verbosity
ff137 Dec 2, 2024
6f5634b
Merge branch 'main' into improve-logging
ff137 Dec 4, 2024
909c15a
:art: Reduce repetition
ff137 Dec 4, 2024
33ab9a0
:art: Log warning when StorageNotFoundError encountered, instead of e…
ff137 Dec 5, 2024
340ec9e
:art: Set Shutdown log level to fatal, with exc_info
ff137 Dec 5, 2024
be2f302
:art: Log warning for bad credential format, instead of always except…
ff137 Dec 5, 2024
ce6bf7b
:construction: Add exc_info to error log for debugging
ff137 Dec 5, 2024
3a5927d
Merge branch 'main' into improve-logging
ff137 Dec 5, 2024
f86406f
:art: Replace AskarError with ProfileError
ff137 Dec 5, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into improve-logging
  • Loading branch information
ff137 committed Dec 5, 2024
commit 3a5927da3daf71302c4cff93397e5cdf78731b2a

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.