Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

frame-support: Use logging for printing corrupted state #10612

Merged
merged 1 commit into from
Jan 8, 2022

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Jan 7, 2022

runtime_print! is printed by default using debug, aka not being visible. With log::error! it
will be printed directly to the user. Production networks like Polkadot disable logging, but for
them we run special nodes that have logging enabled.

`runtime_print!` is printed by default using `debug`, aka not being visible. With `log::error!` it
will be printed directly to the user. Production networks like Polkadot disable logging, but for
them we run special nodes that have logging enabled.
@bkchr bkchr added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Jan 7, 2022
@kianenigma
Copy link
Contributor

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 617c36e into master Jan 8, 2022
@paritytech-processbot paritytech-processbot bot deleted the bkchr-log-corrupted-state branch January 8, 2022 16:13
grishasobol pushed a commit to gear-tech/substrate that referenced this pull request Mar 28, 2022
…0612)

`runtime_print!` is printed by default using `debug`, aka not being visible. With `log::error!` it
will be printed directly to the user. Production networks like Polkadot disable logging, but for
them we run special nodes that have logging enabled.
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
…0612)

`runtime_print!` is printed by default using `debug`, aka not being visible. With `log::error!` it
will be printed directly to the user. Production networks like Polkadot disable logging, but for
them we run special nodes that have logging enabled.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants