-
Notifications
You must be signed in to change notification settings - Fork 144
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
feat: add new features to hiro-system-kit #1396
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1396 +/- ##
==========================================
- Coverage 40.91% 40.85% -0.07%
==========================================
Files 87 87
Lines 32519 32567 +48
==========================================
Hits 13305 13305
- Misses 19214 19262 +48 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
77263f7
to
51f484c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM @MicaiahReid
Chainhook needs a debug log level that still uses slog_json when built in release mode. This PR adds a new feature to accomplish this.
Additionally, Grafana has set log levels that it uses for parsing logs. Currently the devops team has custom patches to support parsing logs from hiro system kit, because it abbreviates log prefixes in weird ways (
DEBUG
=>DEBG
,ERROR
=>ERRO
). This PR also adds a feature,full_log_level_prefix
, which just has slog log the full prefix, which will allow devops to remove this custom patches.