Skip to content

feat: add LOG_TERMINAL flag, default to JSON#2176

Merged
mslipper merged 1 commit intoethereum-optimism:developfrom
cfromknecht:bss-json-logs
Feb 9, 2022
Merged

feat: add LOG_TERMINAL flag, default to JSON#2176
mslipper merged 1 commit intoethereum-optimism:developfrom
cfromknecht:bss-json-logs

Conversation

@cfromknecht
Copy link
Contributor

Description
This commit modifies the BSS to default to JSON logs, which are more
useful in production. This can be overridden by setting
LOG_TERMINAL to true for debugging locally, as is kept in our itests.

cc @optimisticben

Metadata

  • Fixes ENG-1952

This commit modifies the BSS to default to JSON logs, which are more
useful in production. This can be overridden by setting
LOG_TERMINAL to true for debugging locally.
@changeset-bot
Copy link

changeset-bot bot commented Feb 8, 2022

🦋 Changeset detected

Latest commit: 3ec0630

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/batch-submitter-service Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

@optimisticben optimisticben left a comment

Choose a reason for hiding this comment

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

LGTM

@mslipper mslipper merged commit ee8499c into ethereum-optimism:develop Feb 9, 2022
@cfromknecht cfromknecht deleted the bss-json-logs branch February 9, 2022 17:13
@mslipper mslipper mentioned this pull request Feb 10, 2022
2 tasks
theochap added a commit that referenced this pull request Dec 10, 2025
## Description

This PR removes the `process` method from `NodeActor` trait to simplify
the structure of the `NodeActor`

Progress to #2176
theochap added a commit that referenced this pull request Jan 14, 2026
…s/kona#2266)

## Description

This PR removes the `process` method from `NodeActor` trait to simplify
the structure of the `NodeActor`

Progress to #2176
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ops Area: ops

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants