Skip to content

Conversation

m30m
Copy link
Contributor

@m30m m30m commented Jun 24, 2025

Summary

Uses another log format when not running in shell

Rationale

Some alerts are not working properly because the error log is multiline and we can't capture the chain_id of the error log

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

This is exactly how hermes sets up the logging.

Copy link

vercel bot commented Jun 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api-reference ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2025 11:27pm
component-library ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2025 11:27pm
developer-hub ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2025 11:27pm
entropy-debugger ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2025 11:27pm
entropy-explorer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2025 11:27pm
insights ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2025 11:27pm
proposals ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2025 11:27pm
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2025 11:27pm

Copy link
Contributor

@tejasbadadare tejasbadadare left a comment

Choose a reason for hiding this comment

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

thx! beats having to regexp it in grafana haha

@m30m m30m merged commit d5d48bc into main Jun 24, 2025
11 checks passed
@m30m m30m deleted the branch-BFA66E13 branch June 24, 2025 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants