Skip to content
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

Feature/ongoing pp #354

Open
wants to merge 119 commits into
base: develop
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
119 commits
Select commit Hold shift + click to select a range
bbe1b35
ongoing
invocamanman Jul 3, 2024
81fe6c9
ongoing
invocamanman Jul 8, 2024
1f723c3
ongoing
invocamanman Jul 8, 2024
e6111ee
wip update polygonRollupManager
laisolizq Jul 11, 2024
45912c5
wip new polygon consensus contract
laisolizq Jul 11, 2024
3b3fcd7
review
laisolizq Jul 15, 2024
55029f8
review consensus contract
laisolizq Jul 15, 2024
5136a6c
review PR
laisolizq Jul 15, 2024
1e3c96c
small fixes to compile
invocamanman Jul 16, 2024
0520738
Implement PP at add new rollupType
ignasirv Jul 16, 2024
f0a993c
ongoign
invocamanman Jul 17, 2024
81a372a
now compiles
invocamanman Jul 17, 2024
3ee556c
hi
invocamanman Jul 17, 2024
9ad9c33
wip
krlosMata Jul 17, 2024
a267651
empty commit signed
invocamanman Jul 17, 2024
4c12976
verify PP
krlosMata Jul 18, 2024
591a6ca
rollup.verifier --> address type. Load interface depending on verify …
krlosMata Jul 18, 2024
116454f
fixes
invocamanman Jul 19, 2024
d80cd1e
mini updates
laisolizq Jul 18, 2024
87e91ac
review PR
laisolizq Jul 19, 2024
a3fc69f
fix compilation
invocamanman Jul 20, 2024
7473921
Update events for PP
ignasirv Jul 18, 2024
f9eca7b
sudo in docker script. Comments PR
krlosMata Jul 22, 2024
b19aafd
fix dockerv2 sudo
laisolizq Jul 22, 2024
77dc579
update scripts deploy-docker
laisolizq Jul 23, 2024
c70866c
update docs-compiledcontracts
krlosMata Jul 23, 2024
1ffa27a
ongoing
invocamanman Jul 3, 2024
089288a
ongoing
invocamanman Jul 8, 2024
70519e3
ongoing
invocamanman Jul 8, 2024
3fab639
wip update polygonRollupManager
laisolizq Jul 11, 2024
6be961b
wip new polygon consensus contract
laisolizq Jul 11, 2024
ff3220c
review
laisolizq Jul 15, 2024
c552eea
review consensus contract
laisolizq Jul 15, 2024
a268254
review PR
laisolizq Jul 15, 2024
1597d8f
small fixes to compile
invocamanman Jul 16, 2024
81d3c5c
Implement PP at add new rollupType
ignasirv Jul 16, 2024
b0ef71c
ongoign
invocamanman Jul 17, 2024
9c34d74
now compiles
invocamanman Jul 17, 2024
2324b1a
hi
invocamanman Jul 17, 2024
36ed20c
wip
krlosMata Jul 17, 2024
794e10d
empty commit signed
invocamanman Jul 17, 2024
60725a2
verify PP
krlosMata Jul 18, 2024
2cb2e20
rollup.verifier --> address type. Load interface depending on verify …
krlosMata Jul 18, 2024
942c65a
fixes
invocamanman Jul 19, 2024
f8c33cf
mini updates
laisolizq Jul 18, 2024
d48660c
review PR
laisolizq Jul 19, 2024
b15cacf
fix compilation
invocamanman Jul 20, 2024
77a9b46
Update events for PP
ignasirv Jul 18, 2024
e412d4b
sudo in docker script. Comments PR
krlosMata Jul 22, 2024
1f93373
fix dockerv2 sudo
laisolizq Jul 22, 2024
c4ee4ab
update scripts deploy-docker
laisolizq Jul 23, 2024
82f59ad
update docs + fox zeroAddress
krlosMata Jul 23, 2024
eb1e9af
update contracts tests
laisolizq Jul 24, 2024
1b328ce
ignore PolygonRollupManagerUpgrade test
laisolizq Jul 25, 2024
7a61a94
WIP
krlosMata Jul 25, 2024
f696419
test full pesismistic
krlosMata Jul 26, 2024
ae107da
new getters rollupData. All tests passing
krlosMata Jul 26, 2024
fdda2b3
rename _legacy... to _
krlosMata Jul 30, 2024
5225b1f
increase coverage
krlosMata Jul 30, 2024
da25752
add edge case
krlosMata Jul 30, 2024
3c7d6c8
all coverage
krlosMata Jul 31, 2024
fcf9142
fix eslint
krlosMata Jul 31, 2024
cebd5b8
add rollupID as a public input
krlosMata Aug 1, 2024
a1dd5a1
remove incorrect comment
krlosMata Aug 1, 2024
b3c3101
missing checkout on gha docker-build
krlosMata Aug 2, 2024
9735b85
returnData V2
krlosMata Aug 2, 2024
f508836
fix build docker
krlosMata Aug 2, 2024
0ae4063
mini thingds
invocamanman Aug 5, 2024
1a3fc82
move function
invocamanman Aug 5, 2024
e077054
mini fix
invocamanman Aug 18, 2024
65f9710
compiled and docs
invocamanman Aug 18, 2024
4fb1711
udpate PP
invocamanman Aug 18, 2024
00216b5
Spellcheck
ignasirv Aug 20, 2024
a2384a5
Add rollupManager initializer versioning
ignasirv Aug 22, 2024
cceaa7a
Add version in constant
ignasirv Aug 27, 2024
ad20201
github comments
ignasirv Aug 29, 2024
f8fcc1d
Move to constants
ignasirv Aug 29, 2024
c9cb821
selecterGER --> l1InfoTreeLeafCount
krlosMata Aug 29, 2024
0b9e04f
add _disableInitializers();
krlosMata Aug 30, 2024
9f7b676
hidden typo
krlosMata Aug 30, 2024
f0ba99b
Setup for sp1 e2e testing
ignasirv Sep 9, 2024
e52f54e
fix build dockerv2:contracts
laisolizq Sep 12, 2024
acf1d85
update forkID=11 docker build
laisolizq Sep 13, 2024
67ae71b
fix lint
laisolizq Sep 13, 2024
5c48e01
token wrapped bridge contract
invocamanman Sep 13, 2024
df75d51
get ABI from compiled-contracts
krlosMata Sep 15, 2024
713ef56
rename to TokenWrappedBridge
krlosMata Sep 15, 2024
5aaa318
fix test
krlosMata Sep 15, 2024
533f053
minor fixes on tools
krlosMata Sep 5, 2024
095fdf2
cherry-pick 4af640cfb84b4841450d335dc5a5c0cd4d800cea
krlosMata Sep 15, 2024
25d847d
disable initializer=false
krlosMata Sep 23, 2024
48b7bbc
Merge pull request #340 from 0xPolygonHermez/feature/fix-initilaize
invocamanman Sep 26, 2024
d05367e
audit remediations
krlosMata Oct 3, 2024
289b8ad
support deployment real sp1 verifier & update tools
krlosMata Oct 6, 2024
d5de496
Merge pull request #341 from 0xPolygonHermez/feature/audit-changes
invocamanman Oct 7, 2024
f57c655
PR review
krlosMata Oct 7, 2024
d0b44f2
Merge pull request #342 from 0xPolygonHermez/feature/real-sp1-verifier
invocamanman Oct 7, 2024
d947b94
Update e2e with l1InfoTree + upgrade verifier to v2
ignasirv Oct 7, 2024
82d3577
Merge pull request #343 from 0xPolygonHermez/feature/sp1-e2e-update
invocamanman Oct 7, 2024
2b044eb
Recover reference url comments
ignasirv Oct 7, 2024
f5a359b
Bridge implementation for sovereign chains + tests
ignasirv Sep 12, 2024
a612420
finish injected batch data for pessimisic consensus
krlosMata Oct 11, 2024
0440e6c
Internal audit fixes + new coverage
ignasirv Oct 21, 2024
8f5e753
Add weth at genesis + other review improvements
ignasirv Oct 22, 2024
2a3ebd7
Improve WETH comments at bridge contract
ignasirv Oct 24, 2024
29023ea
typos
ignasirv Oct 24, 2024
f1227e4
Review fixes
ignasirv Oct 29, 2024
ddfd159
Remove system address from final genesis state
ignasirv Nov 5, 2024
63a89fe
Merge pull request #330 from 0xPolygonHermez/feature/sovereign-bridge
ignasirv Nov 5, 2024
967376e
Upgrade sp1 verifier to v3 + improve comment at bridgev2
ignasirv Oct 29, 2024
3157872
Merge pull request #353 from 0xPolygonHermez/feature/missing-changes-sov
ignasirv Nov 5, 2024
1b569f5
Merge branch 'develop' into feature/ongoingPP
krlosMata Nov 5, 2024
44c089d
fix pp info upgrade
invocamanman Nov 5, 2024
db4c974
update gastTOkens
invocamanman Nov 5, 2024
f9c1d0d
update gastTOkens
invocamanman Nov 5, 2024
99a473b
Clean PolygonRollupManagerNotUpgraded
ignasirv Nov 5, 2024
c8659e6
Merge pull request #355 from 0xPolygonHermez/feature/fixUpgradePPInfo
krlosMata Nov 5, 2024
3395410
Remove mocked prev ler for sp1 e2e testing + updated input
ignasirv Nov 26, 2024
09369cd
Merge pull request #362 from 0xPolygonHermez/fefature/unmock-e2e
ignasirv Nov 26, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/build-docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:

strategy:
matrix:
node-version: [18.x]
node-version: [20.x]

steps:
- name: Use Node.js ${{ matrix.node-version }}
Expand All @@ -31,4 +31,4 @@ jobs:
- name: Build docker
run: npm run dockerv2:contracts
- name: Test docker
run: npm run docker:tests
run: npm run docker:tests
60 changes: 60 additions & 0 deletions .github/workflows/build-push-docker.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,60 @@
# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node
# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions

name: Build-Docker image and push it

on:
push:
branches: [main]
jobs:
build:
runs-on: ubuntu-latest

strategy:
matrix:
node-version: [20.x]

steps:
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v3
with:
node-version: ${{ matrix.node-version }}
- name: Set int-bot SSH key
run: |
touch /tmp/ssh-key
echo "${{ secrets.INT_BOT_SSH_KEY }}" > /tmp/ssh-key
chmod 400 /tmp/ssh-key
eval "$(ssh-agent -s)"
ssh-add /tmp/ssh-key
- name: Checkout code
uses: actions/checkout@v3
- name: setup
run: |
eval "$(ssh-agent -s)"
ssh-add /tmp/ssh-key
npm i
sudo curl -L "https://github.com/docker/compose/releases/download/1.26.0/docker-compose-$(uname -s)-$(uname -m)" -o /usr/local/bin/docker-compose
sudo chmod +x /usr/local/bin/docker-compose
- name: Login to DockerHub
uses: docker/login-action@v1
with:
username: ${{ secrets.DOCKERHUB_USERNAME }}
password: ${{ secrets.DOCKERHUB_TOKEN }}
- name: Build docker
run: npm run docker:contracts
- name: Push docker image
run: npm run push:docker:contracts
# Steps to push multi-platform image, it relies on the previous step:
# npm run docker:contracts
- name: Set up QEMU
uses: docker/setup-qemu-action@v1
- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v1
- name: Build and push
uses: docker/build-push-action@v2
with:
platforms: linux/amd64,linux/arm64
push: true
tags: hermeznetwork/geth-zkevm-contracts:1.5-integration
file: docker/Dockerfile
context: .
6 changes: 3 additions & 3 deletions .github/workflows/compile.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,17 +5,17 @@ name: Main CI

on:
push:
branches: [feature/ongoingPP]
branches: [main, develop, feature/ongoingPP]
pull_request:
branches: [feature/ongoingPP]
branches: [main, develop, feature/ongoingPP]

jobs:
lint-and-test:
runs-on: ubuntu-latest

strategy:
matrix:
node-version: [18.x]
node-version: [20.x]

steps:
- name: Use Node.js ${{ matrix.node-version }}
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,17 +5,17 @@ name: Main CI

on:
push:
branches: [main, develop]
branches: [main, develop, feature/ongoingPP]
pull_request:
branches: [main, develop]
branches: [main, develop, feature/ongoingPP]

jobs:
lint-and-test:
runs-on: ubuntu-latest

strategy:
matrix:
node-version: [18.x]
node-version: [20.x]

steps:
- name: Use Node.js ${{ matrix.node-version }}
Expand Down
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -22,3 +22,5 @@ typechain-types/
create_rollup_parameters.json

docker/deploymentOutput

.DS_Store
4 changes: 2 additions & 2 deletions compiled-contracts/FflonkVerifier.json

Large diffs are not rendered by default.

Loading
Loading