forked from ethereum-optimism/optimism
-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[pull] op-leader-election from ethereum-optimism:develop #97
Open
pull
wants to merge
4,099
commits into
EspressoSystems:op-leader-election
Choose a base branch
from
ethereum-optimism:develop
base: op-leader-election
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[pull] op-leader-election from ethereum-optimism:develop #97
pull
wants to merge
4,099
commits into
EspressoSystems:op-leader-election
from
ethereum-optimism:develop
+312,780
−293,168
Conversation
This file contains 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
* contracts-bedrock: create OptimismPortalInterop * contracts-bedrock: update to use ConfigType.GAS_PAYING_TOKEN * contracts-bedrock: update snapshots * contracts-bedrock: update semver-lock
* contracts-bedrock: create OptimismPortalInterop * contracts-bedrock: add conditional deployment of SystemConfigInterop * contracts-bedrock: drop initializeOptimismPortalInterop in Deploy script * contracts-bedrock: remove checkSystemConfigInterop from ChainAssertions * contracts-bedrock: drop initializeSystemConfigInterop in Deploy script * contracts-bedrock: drop deploySystemConfigInterop from Deploy script * contracts-bedrock: drop SystemConfigInterop from Types * contracts-bedrock: drop legacy import in ChainAssertions * contracts-bedrock: drop legacy fields in Deploy script * contracts-bedrock: drop unnecessary diffs * contracts-bedrock: add conditional deployment of SystemConfigInterop * contracts-bedrock: drop initializeOptimismPortalInterop in Deploy script * contracts-bedrock: create tests for SystemConfigInterop * contracts-bedrock: reorder dependencies in tests for SystemConfigInterop * contracts-bedrock: add SystemConfigInterop to tests Setup * contracts-bedrock: update tests for SystemConfigInterop * contracts-bedrock: restore Deploy script * contracts-bedrock: restore ChainAssertions * contracts-bedrock: restore Types * contracts-bedrock: drop SystemConfigInterop from Setup script * contracts-bedrock: update SystemConfigInterop * contracts-bedrock: add not vm check in SystemConfigInterop tests * contracts-bedrock: update tests for SystemConfigInterop * contracts-bedrock: define internal getter for systemConfigInterop in tests * contracts-bedrock: restrict visibility of function in tests for SystemConfig * contracts-bedrock: fix tests for SystemConfigInterop * contracts-bedrock: fix tests for SystemConfigInterop * contracts-bedrock: reinitialize for gas paying test for SystemConfigInterop * contracts-bedrock: reorder functions in tests for SystemConfigInterop * contracts-bedrock: use ConfigType.GAS_PAYING_TOKEN in tests for SystemConfigInterop * contracts-bedrock: use assumeNotForgeAddress in tests for SystemConfigInterop --------- Co-authored-by: Mark Tyneway <mark.tyneway@gmail.com>
…10658) Bumps [github.com/jackc/pgx/v5](https://github.com/jackc/pgx) from 5.5.5 to 5.6.0. - [Changelog](https://github.com/jackc/pgx/blob/master/CHANGELOG.md) - [Commits](jackc/pgx@v5.5.5...v5.6.0) --- updated-dependencies: - dependency-name: github.com/jackc/pgx/v5 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* contracts-bedrock: fix deployment Fixes #10634 The issue was that the salt was based on the name of the contract, meaning that it was deterministic. The salt is now created by mixing together the name and the `IMPL_SALT` env var. Anytime that `create2` is used in the deploy script, the salt MUST be mixed with `IMPL_SALT`, otherwise it will brick fresh deployments. * kontrol: update summary
* op-node: Better logging in commitment tracking * op-node: Properly handle differences in Keccak vs Generic Commitments * Use daCommitmentType in deploy config * da-server: Add option to use generic commitments * CI: Add generic commitment * op-chain-ops: Set daCommitmentType in testdata * Fix comm type check * Fix CI env var * Set proper default for daCommitmentType * Use string for daCommitmentType * Use bytes cast instead of abi.encodePacked
`endpoint-monitor` is not maintained anymore and is deprecated, so this commit removes it from the monorepo
…Messenger`) (#10670) * contracts-bedrock: reset semver of CrossL2Inbox * contracts-bedrock: reset semver of L2ToL2CrossDomainMessenger * contracts-bedrock: update semver-lock
* adding bridging to the faucet drip call * updating the logic based on suggesstions * adding changeset * fixing ci issue * update auth module tests * final fix * lint fix * update snapshots * update changeset * change to patch --------- Co-authored-by: Tarun Khasnavis <tarunkhasnavis@Taruns-MacBook-Pro.local> Co-authored-by: tre <tremarkley@gmail.com>
…mining if a claim is resolvable (#10653)
* ctb: Introduce Process library with run() method * ctb: Replace vm.ffi with Process.run
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* cannon: Add traceback debugger * review comments * fix go lint
* op-node: cleanup readme The readme hasn't been updated in some time, this updates the readme with the latest information as a lot of the information has been deprecated. This will make it more intuitive for how to use the `op-node` to generate the L2 genesis for a network. Also adds a simple `doctoc` recipe to the `Makefile` as other `README` files in the repo do the same thing. * readme: update
#10674) * Use the actual WETH delay when determining when bonds are withdrawable * Change metric label and logs to reflect using withdrawable timing, not game resolution
This removes running the external geth tests in CI since they are particularly flakey. These were added to enable running the e2e tests against external clients more easily. This does not remove that ability from the monorepo as the package is not touched. External clients can still run the e2e tests against their client, we just no longer run these tests in CI. There isn't a ton of value running these in CI as everything in process tests the same things and the value of removing them is the reduce in flakes. A change to docs or smart contracts should not need to rerun CI for unrelated tests flaking. Since there is no clear ownership for these tests, they should be removed. This is not reducing test coverage.
* op-node: Support multiple beacon archiver/fallbacks * op-node: rename flag to l1.beacon-fallbacks - retain old flag name and env var - fix some tests * Update op-node/node/client_test.go Co-authored-by: Adrian Sutton <adrian@oplabs.co> --------- Co-authored-by: Adrian Sutton <adrian@oplabs.co>
Bumps [github.com/rs/cors](https://github.com/rs/cors) from 1.9.0 to 1.11.0. - [Commits](rs/cors@v1.9.0...v1.11.0) --- updated-dependencies: - dependency-name: github.com/rs/cors dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [certifi](https://github.com/certifi/python-certifi) from 2023.7.22 to 2024.7.4. - [Commits](certifi/python-certifi@2023.07.22...2024.07.04) --- updated-dependencies: - dependency-name: certifi dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…1085) * dependabot(gomod): bump golang.org/x/crypto from 0.24.0 to 0.25.0 Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.24.0 to 0.25.0. - [Commits](golang/crypto@v0.24.0...v0.25.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * deps: update --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Mark Tyneway <mark.tyneway@gmail.com>
The specs are no longer managed in the `optimism` repo but instead in the `specs` repo. This commit removes the dead command in the `Makefile` Based on #11074
#11096) Bumps [github.com/holiman/uint256](https://github.com/holiman/uint256) from 1.2.4 to 1.3.0. - [Release notes](https://github.com/holiman/uint256/releases) - [Commits](holiman/uint256@v1.2.4...v1.3.0) --- updated-dependencies: - dependency-name: github.com/holiman/uint256 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* finish TODO(optimism#11023) * revert GetLogBlockNum() && use ClosestBlockInfo * catch error * rewind * gofmt -w -s . * op-supervisor: Remove unused import. --------- Co-authored-by: Adrian Sutton <adrian@oplabs.co>
Executing messages aren't extracted yet but basic log information is hashed and written.
Add tests for finding the block to start from.
* op-node: event-system with tracing * op-node: fix global synchronous events queue execution * op-node: bump rate limits on action tests * op-node: add missing read-lock to synchronous event executor * op-node: automatically attach emitters to derivers that need one
Bumps [mocha](https://github.com/mochajs/mocha) and [@types/mocha](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/mocha). These dependencies needed to be updated together. Updates `mocha` from 10.2.0 to 10.6.0 - [Release notes](https://github.com/mochajs/mocha/releases) - [Changelog](https://github.com/mochajs/mocha/blob/main/CHANGELOG.md) - [Commits](mochajs/mocha@v10.2.0...v10.6.0) Updates `@types/mocha` from 10.0.6 to 10.0.7 - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/mocha) --- updated-dependencies: - dependency-name: mocha dependency-type: direct:development update-type: version-update:semver-minor - dependency-name: "@types/mocha" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…for a block. (#11122) Ensures the database is rewound to the point where it can retry storing the logs for the block rather than having some logs already exist.
* op-supervisor: Move TruncatedHash and ExecutingMessage out of db package * op-supervisor: Introduce EntryIdx type Will make it clear when a value is a block number vs a db entry index when adding head tracking.
* op-node: sequencer / engine events refactor incl sequencer events fixes * op-node: distinguish block sealing error kinds * op-node: review fixes, stashed tweaks * op-node: events based sequencer chaos test * op-node: fix missing DerivedFrom data in attributes test * op-node: drop old wip debugging work log * op-node: sequencer move OnEvent function * op-node: update stale todo comment * op-node: detect derivation block-building as sequencer, and avoid conflict * op-node: clarify comments and rename PayloadSealTemporaryErrorEvent to PayloadSealExpiredErrorEvent to describe applicability better * op-node: prevent temporary engine error from influencing inactive sequencer
* op-supervisor: Introduce head tracker * op-supervisor: Move log db to subpackage of db. * op-supervisor: Route all updates through a common db that can track heads * op-supervisor: Remove unused error. * op-supervisor: Remove operations - it fits into a later PR. * op-supervisor: Fix semgrep * op-supervisor: Move resuming databases into ChainsDB so it can later update the chain heads too.
Bumps [@changesets/cli](https://github.com/changesets/changesets) from 2.27.1 to 2.27.7. - [Release notes](https://github.com/changesets/changesets/releases) - [Changelog](https://github.com/changesets/changesets/blob/main/docs/modifying-changelog-format.md) - [Commits](https://github.com/changesets/changesets/compare/@changesets/cli@2.27.1...@changesets/cli@2.27.7) --- updated-dependencies: - dependency-name: "@changesets/cli" dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [typescript](https://github.com/Microsoft/TypeScript) from 5.4.5 to 5.5.3. - [Release notes](https://github.com/Microsoft/TypeScript/releases) - [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml) - [Commits](microsoft/TypeScript@v5.4.5...v5.5.3) --- updated-dependencies: - dependency-name: typescript dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )