Conversation
Contributor
mslipper
commented
Aug 8, 2022
mslipper
commented
- op-node: More reset & pipeline logging (op-node: More reset & pipeline logging #3178)
- op-node: Fixed Integers in channel frame header (op-node: Fixed Integers in channel frame header #3126)
- actor-tests: Bedrock actor tests (actor-tests: Bedrock actor tests #3161)
- ci: only run changeset integrety on pushes (ci: only run changeset integrety on pushes #3177)
- op-node: fix invalid payload insertion error handling (op-node: fix invalid payload insertion error handling #3180)
- feat: Update wsteth address (feat: Update wsteth address #3181)
- pkg driver: use derive error types to optionally retry with backoff (pkg driver: use derive error types to optionally retry with backoff #3130)
- op-node: reorgs should be typed to force pipeline resets (op-node: reorgs should be typed to force pipeline resets [bedrock] #3185)
- op-node: separate sequencing and publishing errors. Old payloads fail to publish, but should not hold back catch-up work of sequencer (op-node: separate sequencing and publishing errors. #3186)
- ci: Upgrade ci-builder to Debian 11 (ci: Upgrade ci-builder to Debian 11 #3189)
* goals for fixed int * op-node: Fixed integer sizes in the channel header This mainly modifies the channel_frame, but has some knock on effects as the width of some of the fields have been reduced. The channel frame code is also changed more than I expected due to differences in API of working with fixed int vs uvarints in go. Otherwise the code reads very similarly with using Reader/Writer APIs. * op-node: fix frame unmarshal func to return correct err * specs: update frame format specs * Update op-node/rollup/derive/channel_bank_test.go Co-authored-by: protolambda <proto@protolambda.com> Co-authored-by: Matthew Slipper <me@matthewslipper.com>
* actor-tests: Bedrock actor tests - Moves actors into their own package - Minor updates to logging, etc. - Adds Cloudbuild config so that we can get Docker images on every commit to Develop - Updates Actors to support Bedrock Currently supported actors: - Deposits - Contract deployments - Simple sends * doctoc only look at specs
* ci: only run changeset integrety on PRs It will fail on merges to develop * ci: better fix
* op-node: fix invalid payload insertion error handling * fix goimports Co-authored-by: Matthew Slipper <me@matthewslipper.com>
* update wsteth bridge address * changeset Co-authored-by: Will Cory <willcory@Wills-MacBook-Pro.local> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
…3130) * pkg derive: wrap errors by severity * pkg derive: wrap errors by severity * pkg derive: fix call with 2 return values * ci: fix golangci-lint warnings * ci: fix attributes test * pkg derive: update tests, restore err types * driver: update to use err severity from derive * op-node: fix retry callers * pkg derive: reimplement the custom error type * pkg derive: error - nits * pkg driver: update Step in event loop to backoff * pkg derive: match epoch hash with l1 block hash * pkg driver: handle backoff error * pkg driver: handle temporary errs with backoff also include additional if / else statements to log errors * op-node: req next step, with backoff, without blocking other events * pkg derive: update error types * pkg derive: typo Co-authored-by: protolambda <proto@protolambda.com>
… to publish, but should not hold back catch-up work of sequencer (#3186)
🦋 Changeset detectedLatest commit: c666fed The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
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 |
github-actions
bot
added
2-reviewers
A-integration
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.