Skip to content

op-node: Add gratuitous logging#3100

Closed
trianglesphere wants to merge 1 commit intodevelopfrom
jg/too_verbose_logging
Closed

op-node: Add gratuitous logging#3100
trianglesphere wants to merge 1 commit intodevelopfrom
jg/too_verbose_logging

Conversation

@trianglesphere
Copy link
Contributor

@trianglesphere trianglesphere commented Jul 26, 2022

Description
This is logging is going to be extremely verbose, but should help
with tracking down a livelock bug.

@changeset-bot
Copy link

changeset-bot bot commented Jul 26, 2022 •
edited
Loading

⚠️ No Changeset found

Latest commit: 1948678

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@mergify
Copy link
Contributor

mergify bot commented Jul 26, 2022

This PR changes implementation code, but doesn't include a changeset. Did you forget to add one?

@trianglesphere trianglesphere force-pushed the jg/too_verbose_logging branch from cbf1503 to d387347 Compare July 26, 2022 18:54
@trianglesphere
op-node: Add gratuitous logging
1948678 
This is logging is going to be extremely verbose, but should help
with tracking down a livelock bug.
@trianglesphere trianglesphere force-pushed the jg/too_verbose_logging branch from d387347 to 1948678 Compare July 26, 2022 18:55
@trianglesphere
Copy link
Contributor Author

trianglesphere commented Aug 4, 2022

Closing b/c this is a lot of logging and did not help me track down the bug here.

@trianglesphere trianglesphere deleted the jg/too_verbose_logging branch August 4, 2022 18:27
theochap pushed a commit that referenced this pull request Dec 10, 2025
@mablr
feat(node): add admin_resetDerivationPipeline RPC endpoint (#3121)
58dbae6 
Close #3100 by adding `admin_resetDerivationPipeline` RPC endpoint.

Followed the testing patterns from #3101, whole diff is not covered but
the core logic is.
theochap pushed a commit that referenced this pull request Jan 14, 2026
@mablr
feat(node): add admin_resetDerivationPipeline RPC endpoint (op-rs/k…
d0c23a7 
…ona#3121)

Close #3100 by adding `admin_resetDerivationPipeline` RPC endpoint.

Followed the testing patterns from #3101, whole diff is not covered but
the core logic is.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mslipper mslipper mslipper approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@trianglesphere @mslipper