Skip to content

Comments

specs: Add deposit gas metering spec #2679

Merged
mergify[bot] merged 9 commits intodevelopfrom
jg/deposit_fee_spec
Jun 9, 2022
Merged

specs: Add deposit gas metering spec #2679
mergify[bot] merged 9 commits intodevelopfrom
jg/deposit_fee_spec

Conversation

@trianglesphere
Copy link
Contributor

Description
Add specs for the new deposit fee model.

Implementation is done in the following PRs:

Metadata

  • Fixes ENG-2109

@changeset-bot
Copy link

changeset-bot bot commented Jun 3, 2022

⚠️ No Changeset found

Latest commit: 8b07004

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 mergify bot requested review from Inphi and mslipper June 3, 2022 18:59
@trianglesphere trianglesphere changed the title Deposit Fee Spec specs: Add deposit gas metering spec Jun 3, 2022
@trianglesphere trianglesphere requested review from norswap and protolambda and removed request for Inphi June 3, 2022 19:52
@mergify mergify bot requested a review from Inphi June 3, 2022 20:02
@mergify mergify bot requested review from Inphi and mslipper June 7, 2022 15:56
@trianglesphere trianglesphere requested review from tynes and removed request for Inphi and mslipper June 7, 2022 17:01
@mergify mergify bot requested review from Inphi and mslipper June 7, 2022 17:02
@mslipper
Copy link
Collaborator

mslipper commented Jun 7, 2022

@trianglesphere it'll just keep putting those reviewers back. I can remove that in Mergify.

@trianglesphere
Copy link
Contributor Author

@trianglesphere it'll just keep putting those reviewers back. I can remove that in Mergify.

I didn't realize it was mergify that kept adding them. If you use GH code owners it doesn't re-add them if you manually remove them (also code owners lints for people to make sure they are in the project).

Copy link
Contributor

@norswap norswap left a comment

Choose a reason for hiding this comment

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

It's in pretty good shape!

@trianglesphere trianglesphere force-pushed the jg/deposit_fee_spec branch 2 times, most recently from 1803af6 to 50c5367 Compare June 7, 2022 22:28
@trianglesphere trianglesphere requested a review from norswap June 7, 2022 22:42
Copy link
Contributor

@norswap norswap left a comment

Choose a reason for hiding this comment

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

Made some followup comments, fix to you liking, and after that gtg!

@mergify
Copy link
Contributor

mergify bot commented Jun 9, 2022

This PR has been added to the merge queue, and will be merged soon.

@mergify
Copy link
Contributor

mergify bot commented Jun 9, 2022

This PR is next in line to be merged, and will be merged as soon as checks pass.

@mergify mergify bot merged commit 6648fc9 into develop Jun 9, 2022
@mergify mergify bot deleted the jg/deposit_fee_spec branch June 9, 2022 18:15
@mergify mergify bot removed the on-merge-train label Jun 9, 2022
trianglesphere added a commit that referenced this pull request Jun 9, 2022
* Deposit Fee Spec

* Update guaranteed-gas-market.md

* Update guaranteed-gas-market.md

* Update guaranteed-gas-market.md

* PR Fixes

* PR Changes

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
roninjin10 pushed a commit that referenced this pull request Jun 9, 2022
* Deposit Fee Spec

* Update guaranteed-gas-market.md

* Update guaranteed-gas-market.md

* Update guaranteed-gas-market.md

* PR Fixes

* PR Changes

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
This was referenced Jun 9, 2022
theochap added a commit that referenced this pull request Dec 10, 2025
## Description

Bootstraps the integration tests crate for kona services. For now we
only have a very basic p2p test that spins off two networks and perform
a check that those two are properly connected to each other

Progress on #2623
theochap added a commit that referenced this pull request Dec 10, 2025
…uce test flakiness (#2693)

## Description

This PR achieves the following goals:
- Extend the simple p2p connection tests of #2679 with more
comprehensive tests involving block signature and propagation (a follow
PR will add the cases for higher block versions, it requires updating
the rollup config with hardfork times)
- Fixes the network bootstrap logic to remove flakiness. In particular,
it uses kernel allocated ports for discovery/gossip to avoid static
modification races.
- It adds random valid block building logic
theochap added a commit that referenced this pull request Jan 14, 2026
…uce test flakiness (op-rs/kona#2693)

## Description

This PR achieves the following goals:
- Extend the simple p2p connection tests of #2679 with more
comprehensive tests involving block signature and propagation (a follow
PR will add the cases for higher block versions, it requires updating
the rollup config with hardfork times)
- Fixes the network bootstrap logic to remove flakiness. In particular,
it uses kernel allocated ports for discovery/gossip to avoid static
modification races.
- It adds random valid block building logic
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.

5 participants