-
Notifications
You must be signed in to change notification settings - Fork 12
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
feat: e2e test Pesimisstic L2 to L2 claim #239
Changes from all commits
aa572c0
e8ba6b9
5ab4a04
5fe8671
c84df07
7f8dd00
61db2b1
c684718
2b1a61d
96b7f5d
4f1c47f
152167b
d1ade90
32af660
a80eba1
922b0f1
41594ae
6d1f19e
1a51719
0605d97
46523ea
4dac97c
41e0001
39582d3
cc232e4
94d5dda
ce1e5e5
759cabf
a36f31b
c2748ec
77d5fcf
c465e2d
51af863
adb73d1
98c2e9f
1fb05b8
78724d2
8113e91
0638b0b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||
---|---|---|---|---|
@@ -0,0 +1,85 @@ | ||||
# based on: https://github.com/0xPolygon/kurtosis-cdk/blob/jhilliard/multi-pp-testing/multi-pp-test.sh.md | ||||
name: Test e2e multi pp | ||||
on: | ||||
push: | ||||
branches: | ||||
- '**' | ||||
workflow_dispatch: {} | ||||
|
||||
|
||||
jobs: | ||||
test-e2e-multi_pp: | ||||
strategy: | ||||
fail-fast: false | ||||
matrix: | ||||
go-version: [ 1.22.x ] | ||||
goarch: [ "amd64" ] | ||||
e2e-group: | ||||
- "fork12-pessimistic" | ||||
runs-on: ubuntu-latest | ||||
steps: | ||||
- name: Checkout code | ||||
uses: actions/checkout@v4 | ||||
|
||||
- name: Install Go | ||||
uses: actions/setup-go@v5 | ||||
with: | ||||
go-version: ${{ matrix.go-version }} | ||||
env: | ||||
GOARCH: ${{ matrix.goarch }} | ||||
|
||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nitpick, remove extra blank line
Suggested change
|
||||
|
||||
- name: Build Docker | ||||
run: make build-docker | ||||
|
||||
- name: Build Tools | ||||
run: make build-tools | ||||
|
||||
- name: Checkout kurtosis-cdk | ||||
uses: actions/checkout@v4 | ||||
with: | ||||
repository: 0xPolygon/kurtosis-cdk | ||||
path: kurtosis-cdk | ||||
ref: jhilliard/multi-pp-testing | ||||
|
||||
- name: Install Kurtosis CDK tools | ||||
uses: ./kurtosis-cdk/.github/actions/setup-kurtosis-cdk | ||||
|
||||
- name: Install polycli | ||||
run: | | ||||
git clone https://github.com/0xPolygon/polygon-cli -b jhilliard/alonso | ||||
cd polygon-cli | ||||
make install | ||||
cp ~/go/bin/polycli /usr/local/bin/polycli | ||||
/usr/local/bin/polycli version | ||||
|
||||
- name: Setup Bats and bats libs | ||||
uses: bats-core/[email protected] | ||||
|
||||
- name: Test | ||||
run: make test-e2e-fork12-multi-pessimistic | ||||
|
||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||
working-directory: test | ||||
env: | ||||
KURTOSIS_FOLDER: ${{ github.workspace }}/kurtosis-cdk | ||||
BATS_LIB_PATH: /usr/lib/ | ||||
agglayer_prover_sp1_key: ${{ secrets.SP1_PRIVATE_KEY }} | ||||
|
||||
- name: Dump enclave logs | ||||
if: failure() | ||||
run: kurtosis dump ./dump | ||||
|
||||
- name: Generate archive name | ||||
if: failure() | ||||
run: | | ||||
archive_name="dump_run_with_args_${{matrix.e2e-group}}_${{ github.run_id }}" | ||||
echo "ARCHIVE_NAME=${archive_name}" >> "$GITHUB_ENV" | ||||
echo "Generated archive name: ${archive_name}" | ||||
kurtosis service exec cdk cdk-node-001 'cat /etc/cdk/cdk-node-config.toml' > ./dump/cdk-node-config.toml | ||||
|
||||
- name: Upload logs | ||||
if: failure() | ||||
uses: actions/upload-artifact@v4 | ||||
with: | ||||
name: ${{ env.ARCHIVE_NAME }} | ||||
path: ./dump |
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -56,7 +56,7 @@ install-linter: check-go check-curl | |||||
generate-code-from-proto: check-protoc | ||||||
|
||||||
.PHONY: build | ||||||
build: build-rust build-go ## Builds the binaries locally into ./target | ||||||
build: build-rust build-go build-tools## Builds the binaries locally into ./target | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
.PHONY: build-rust | ||||||
build-rust: | ||||||
|
@@ -66,6 +66,10 @@ build-rust: | |||||
build-go: | ||||||
$(GOENVVARS) go build -ldflags "all=$(LDFLAGS)" -o $(GOBIN)/$(GOBINARY) $(GOCMD) | ||||||
|
||||||
.PHONY: build-tools | ||||||
build-tools: ## Builds the tools | ||||||
$(GOENVVARS) go build -o $(GOBIN)/aggsender_find_imported_bridge ./tools/aggsender_find_imported_bridge | ||||||
|
||||||
.PHONY: build-docker | ||||||
build-docker: ## Builds a docker image with the cdk binary | ||||||
docker build -t cdk -f ./Dockerfile . | ||||||
|
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.
Nitpick: remove extra empty line