-
Notifications
You must be signed in to change notification settings - Fork 7
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
pbss implemention #161
Open
ryanmorphl2
wants to merge
46
commits into
main
Choose a base branch
from
ryan/pbss
base: main
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
pbss implemention #161
Changes from 41 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
1485d89
feat: pbss implemention
ryanmorphl2 44392b7
WIP
ryanmorphl2 cb759c6
WIP
ryanmorphl2 fade32f
WIP
ryanmorphl2 4814a62
WIP
ryanmorphl2 0a527a0
WIP
ryanmorphl2 01f4ce7
WIP
ryanmorphl2 811ebab
WIP
ryanmorphl2 bcf0b6c
WIP
ryanmorphl2 6c25201
compact key of storage trie node
ryanmorphl2 62f89aa
rawdb is not found err
ryanmorphl2 5bf5bcb
remove unused
ryanmorphl2 bafb670
Copy nodes opt
ryanmorphl2 7a2557b
WIP
ryanmorphl2 5be65e2
WIP
ryanmorphl2 84eaac3
copy nodes opt
ryanmorphl2 0beacac
WIP
ryanmorphl2 feb48bb
buffer: get node opt
ryanmorphl2 a0ed014
rename trie
ryanmorphl2 bba1e75
rm unused
ryanmorphl2 d92fee3
WIP
ryanmorphl2 b1210db
upgrade path zktrie
ryanmorphl2 c72970d
path trie proof
ryanmorphl2 b37cd5f
add flag --state.scheme
ryanmorphl2 ce557df
WIP
ryanmorphl2 ccd7590
WIP
ryanmorphl2 207825a
WIP
ryanmorphl2 bb12ec6
WIP
ryanmorphl2 6e33fa4
WIP
ryanmorphl2 570a566
merge main
ryanmorphl2 b4f0316
upgrage new trie
ryanmorphl2 9109d84
merge main
ryanmorphl2 d554540
statedb: copy with originRoot
ryanmorphl2 49904bf
difflayer: remove unused cache
ryanmorphl2 9c0f993
trie proof: mark todo
ryanmorphl2 6d9401f
fix typo
ryanmorphl2 0f62c7a
offline prune only in hash schema
ryanmorphl2 dea4aa8
merge main
ryanmorphl2 d8f08de
preimages commit
ryanmorphl2 6d086c5
update zktrie version
ryanmorphl2 eb305cb
return state not available
ryanmorphl2 e844e22
fix comment
ryanmorphl2 804cb32
flush with time interval
ryanmorphl2 1e734d8
flush intercal default time
ryanmorphl2 284315c
decrease flush intercal default time
ryanmorphl2 c78b876
set flush intercal default time
ryanmorphl2 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
name: Push Docker Image | ||
|
||
on: | ||
push: | ||
tags: | ||
- morph-v* | ||
|
||
env: | ||
IMAGE_NAME: go-ethereum | ||
|
||
jobs: | ||
# Push image to GitHub Packages. | ||
push: | ||
runs-on: ubuntu-latest | ||
if: github.event_name == 'push' | ||
|
||
steps: | ||
- uses: actions/checkout@v4 | ||
|
||
- name: Build the Docker image | ||
run: docker build . --file Dockerfile -t "${IMAGE_NAME}" | ||
|
||
- name: Log into registry | ||
run: echo "${{ secrets.PACKAGE_TOKEN }}" | docker login ghcr.io -u ${{ github.actor }} --password-stdin | ||
|
||
- name: Push image | ||
run: | | ||
IMAGE_ID=ghcr.io/${{ github.repository }} | ||
|
||
# Change all uppercase to lowercase | ||
IMAGE_ID=$(echo $IMAGE_ID | tr '[A-Z]' '[a-z]') | ||
# Strip git ref prefix from version | ||
VERSION=$(echo "${{ github.ref }}" | sed -e 's,.*/\(.*\),\1,') | ||
# Strip "morph-v" prefix from tag name | ||
[[ "${{ github.ref }}" == "refs/tags/"* ]] && VERSION=$(echo $VERSION | sed -e 's/^morph-v//') | ||
echo IMAGE_ID=$IMAGE_ID | ||
echo VERSION=$VERSION | ||
docker tag $IMAGE_NAME $IMAGE_ID:$VERSION | ||
docker tag $IMAGE_NAME $IMAGE_ID:latest | ||
docker push $IMAGE_ID:$VERSION | ||
docker push $IMAGE_ID:latest |
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
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
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
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
Changed source of genesis path - possible regression
The change from using
ctx.Args().First()
toctx.GlobalString(utils.DataDirFlag.Name)
for retrieving the genesis path is significant. This changes the expected input from a direct argument to a global flag value, which might cause confusion or backward compatibility issues.This change appears to be incorrect. The
DataDirFlag
typically points to the Ethereum data directory, not the genesis JSON file path. Check whether this change was intentional or if it should continue to use the first argument as the genesis file path:🏁 Script executed:
Length of output: 872
Action Required: Revert Genesis Path Retrieval to Use Command Argument
The current change uses the data directory flag (
ctx.GlobalString(utils.DataDirFlag.Name)
) to obtain the genesis file path, which contradicts the command’s documented usage of<genesisPath>
. Other parts of the codebase consistently usectx.Args().First()
for file path retrieval, confirming that the intended behavior is to use the first argument.cmd/geth/chaincmd.go
at line 194DataDirFlag
(intended for specifying the data directory) as the source for the genesis JSON file path is misleading and may break backward compatibility.Please review whether this change was intentional. If not, revert to using
ctx.Args().First()
to align with both the command’s usage documentation and similar implementations elsewhere in the code.📝 Committable suggestion