forked from ethereum-optimism/optimism
-
Notifications
You must be signed in to change notification settings - Fork 34
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
ci: add e2e test #121
Open
tuxcanfly
wants to merge
54
commits into
celestia-develop
Choose a base branch
from
tux/add-ci-e2e-test
base: celestia-develop
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
ci: add e2e test #121
Changes from all commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
f4998ee
da: use celestia as da
tuxcanfly 54c1052
batcher: fix duplicate cli flag name
tuxcanfly d7cf0bb
docker: use latest devnet image
tuxcanfly 146d21c
bump local celestia devnet to 0.9.1
tuxcanfly 80977ae
docker: wait for da to boot
tuxcanfly 82d4fee
op-ndoe: decodeeETHData - use uint32 for height
tuxcanfly 884fd58
proposer: fix da rpc, estimate gas
tuxcanfly 29c04d1
op-node: fix failed to parse frame error
tuxcanfly 83e4fb9
proposer: fix revert
tuxcanfly 249d312
proposer: restore gas limit; add explanation
tuxcanfly bcf0845
proposer: nit
tuxcanfly c4080a0
add celestia logo, update readme
jcstein 73a4651
Update README.md
jcstein a536860
style size
jcstein 544bb81
plus not x
jcstein 1a3ed81
docs: edit intro
jcstein 7550915
Update README.md
jcstein 99b2a9b
Update README.md
jcstein 3591a20
Merge pull request #33 from celestiaorg/jcs/update-readme
jcstein f073f20
nit
jcstein c33c5a0
Merge pull request #34 from celestiaorg/jcs/update-readme
jcstein 4f73953
docs: add section on deposit bridge
jcstein d4f66d0
add workaround from @Ferret-san
jcstein 6b45c90
close code block
jcstein e24f0b1
fix: typo
jcstein 6dd4495
Update README.md
jcstein 320b34a
docs: add bridging section from guidance from @Ferret-san
jcstein ccee67b
Merge pull request #35 from celestiaorg/jcs/bridge
jcstein 1cdb0b8
deps: bump local-devnet to 0.9.2
jcstein 9837663
Merge pull request #36 from celestiaorg/jcs/bump-local-devnet
jcstein b0519b4
docker: fix da container name
tuxcanfly d0280e1
op-node: gracefully retry da connection err
tuxcanfly 8f48706
test: update e2e tests for celestia da
tuxcanfly 7f3cca5
docs: update readme
jcstein 102c5e2
deps: update local-celestia-devnet to v0.9.5
jcstein f05b485
Merge pull request #46 from celestiaorg/jcs/devnet-ver-bump
jcstein cb82872
Add DALC support (#47)
jcstein e71c751
fix: update path to node store (#52)
jcstein 5b663de
devnet: increase block times (#50)
tuxcanfly 1fa2312
.github: add dependabot
MSevey cebd51e
Merge pull request #61 from celestiaorg/sevey/dependabot
MSevey 8f53660
deps: bump celestia-node & local-celestia-devnet image version to v0.…
jcstein 5b97ddc
op-node: log namespaced data details (#94)
tuxcanfly 521cd32
Dependabot and Fork Sync (#93)
MSevey b261ed1
ci: add e2e test
tuxcanfly 049b71e
ci: run devnet step before running e2e test
tuxcanfly 98e7e96
test: workaround to running docker in parallel
tuxcanfly 816d5e2
test: pushd / popd -> cd
tuxcanfly b692daf
ci: try run multiple commands
tuxcanfly 6dcc8b8
nit
tuxcanfly b2678ef
ci: disable release / lint
tuxcanfly 9178519
test: skip TestCrossLayerUser
tuxcanfly 1b82e6d
ci: rm release
tuxcanfly a212778
ci: unskip test
tuxcanfly 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
Validating CODEOWNERS rules …
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 |
---|---|---|
@@ -1,48 +1,10 @@ | ||
# Legacy codebases | ||
/batch-submitter @ethereum-optimism/legacy-reviewers | ||
/bss-core @ethereum-optimism/legacy-reviewers | ||
/gas-oracle @ethereum-optimism/legacy-reviewers | ||
/integration-tests @ethereum-optimism/legacy-reviewers | ||
/l2geth @ethereum-optimism/legacy-reviewers | ||
/l2geth-exporter @ethereum-optimism/legacy-reviewers | ||
/packages/actor-tests @ethereum-optimism/legacy-reviewers | ||
/packages/common-ts @ethereum-optimism/typescript-reviewers | ||
/packages/contracts @ethereum-optimism/contract-reviewers | ||
/packages/contracts-bedrock @ethereum-optimism/contract-reviewers | ||
/packages/contracts-periphery @ethereum-optimism/contract-reviewers | ||
/packages/core-utils @ethereum-optimism/legacy-reviewers | ||
/packages/data-transport-layer @ethereum-optimism/legacy-reviewers | ||
/packages/chain-mon @smartcontracts | ||
/packages/fault-detector @ethereum-optimism/legacy-reviewers | ||
/packages/hardhat-deploy-config @ethereum-optimism/legacy-reviewers | ||
/packages/message-relayer @ethereum-optimism/legacy-reviewers | ||
/packages/migration-data @ethereum-optimism/legacy-reviewers | ||
/packages/replica-healthcheck @ethereum-optimism/legacy-reviewers | ||
/packages/sdk @ethereum-optimism/ecopod | ||
/packages/atst @ethereum-optimism/ecopod | ||
|
||
# Bedrock codebases | ||
/bedrock-devnet @ethereum-optimism/go-reviewers | ||
/op-batcher @ethereum-optimism/go-reviewers | ||
/op-chain-ops @ethereum-optimism/go-reviewers | ||
/op-e2e @ethereum-optimism/go-reviewers | ||
/op-node @ethereum-optimism/go-reviewers | ||
/op-node/rollup @protolambda @trianglesphere | ||
/op-proposer @ethereum-optimism/go-reviewers | ||
/op-program @ethereum-optimism/go-reviewers | ||
/op-service @ethereum-optimism/go-reviewers | ||
/ops-bedrock @ethereum-optimism/go-reviewers | ||
|
||
|
||
# Ops | ||
/.circleci @ethereum-optimism/infra-reviewers | ||
/.github @ethereum-optimism/infra-reviewers | ||
/ops @ethereum-optimism/infra-reviewers | ||
/op-signer @ethereum-optimism/infra-reviewers | ||
|
||
# Misc | ||
/proxyd @ethereum-optimism/infra-reviewers | ||
/indexer @ethereum-optimism/infra-reviewers | ||
/infra @ethereum-optimism/infra-reviewers | ||
/specs @ethereum-optimism/contract-reviewers @ethereum-optimism/go-reviewers | ||
/endpoint-monitor @ethereum-optimism/infra-reviewers | ||
# CODEOWNERS: https://help.github.com/articles/about-codeowners/ | ||
|
||
# Everything goes through the following "global owners" by default. | ||
# Unless a later match takes precedence, these three will be | ||
# requested for review when someone opens a PR. | ||
# Note that the last matching pattern takes precedence, so | ||
# global owners are only requested if there isn't a more specific | ||
# codeowner specified below. For this reason, the global codeowners | ||
# are often repeated in package-level definitions. | ||
* @tuxcanfly @jcstein @Ferret-san |
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,18 @@ | ||
reviewers: | ||
defaults: | ||
- code-owners | ||
groups: | ||
code-owners: | ||
- tuxcanfly | ||
- jcstein | ||
- Ferret-san | ||
celestia: | ||
- team:celestia | ||
files: | ||
'**': | ||
- code-owners | ||
options: | ||
ignore_draft: true | ||
ignored_keywords: | ||
- WIP | ||
number_of_reviewers: 3 |
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,25 @@ | ||
version: 2 | ||
updates: | ||
# Check for updates on our docker image and for celestia-node | ||
- package-ecosystem: docker | ||
directory: "/ops-bedrock" | ||
schedule: | ||
interval: daily | ||
open-pull-requests-limit: 10 | ||
labels: | ||
- dependencies | ||
- package-ecosystem: github-actions | ||
directory: "/" | ||
schedule: | ||
interval: daily | ||
open-pull-requests-limit: 10 | ||
labels: | ||
- dependencies | ||
# Check for updates on celestia packages | ||
- package-ecosystem: gomod | ||
directory: "/" | ||
schedule: | ||
interval: daily | ||
open-pull-requests-limit: 10 | ||
labels: | ||
- dependencies |
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
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,29 @@ | ||
name: CI | ||
on: | ||
push: | ||
branches: | ||
- main | ||
# Trigger on version tags | ||
tags: | ||
- 'v[0-9]+\.[0-9]+\.[0-9]+-OP_v[0-9]+\.[0-9]+\.[0-9]+-CN_v[0-9]+\.[0-9]+\.[0-9]+' | ||
- 'v[0-9]+\.[0-9]+\.[0-9]+-OP_v[0-9]+\.[0-9]+\.[0-9]+-CN_v[0-9]+\.[0-9]+\.[0-9]+-rc(?:[0-9]+|\.[0-9]+)' | ||
pull_request: | ||
workflow_dispatch: | ||
# Inputs the workflow accepts. | ||
inputs: | ||
version: | ||
# Friendly description to be shown in the UI instead of 'name' | ||
description: "Semver type of new version (major / minor / patch)" | ||
# Input has to be provided for the workflow to run | ||
required: true | ||
type: choice | ||
options: | ||
- patch | ||
- minor | ||
- major | ||
|
||
jobs: | ||
test: | ||
uses: ./.github/workflows/test.yml | ||
with: | ||
GO_VERSION: '1.20' |
This file was deleted.
Oops, something went wrong.
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,10 @@ | ||
# devnet.yml | ||
name: 'Celestia Devnet' | ||
description: 'Launch a local celestia devnet' | ||
runs: | ||
using: 'docker' | ||
image: 'ghcr.io/rollkit/local-celestia-devnet:v0.9.5' | ||
ports: | ||
- 26650:26657 | ||
- 26659:26659 | ||
options: --platform linux/amd64 |
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,75 @@ | ||
name: Housekeeping | ||
|
||
on: | ||
issues: | ||
types: [opened] | ||
pull_request_target: | ||
types: [opened] | ||
|
||
jobs: | ||
project: | ||
# ignore dependabot PRs | ||
if: ${{ github.actor != 'dependabot[bot]' }} | ||
name: Add issues and PRs to project and add grooming label | ||
uses: celestiaorg/.github/.github/workflows/[email protected] | ||
secrets: inherit | ||
permissions: | ||
issues: write | ||
pull-requests: write | ||
with: | ||
run-labels: true | ||
labels-to-add: "needs-grooming" | ||
run-projects: true | ||
project-url: https://github.com/orgs/celestiaorg/projects/32 | ||
|
||
auto-add-reviewer: | ||
name: Auto add reviewer to PR | ||
if: github.event.pull_request | ||
uses: celestiaorg/.github/.github/workflows/[email protected] | ||
secrets: inherit | ||
permissions: | ||
issues: write | ||
pull-requests: write | ||
with: | ||
run-auto-request-review: true | ||
|
||
auto-add-assignee: | ||
# ignore dependabot PRs | ||
if: ${{ github.actor != 'dependabot[bot]' }} | ||
name: Assign issue and PR to creator | ||
runs-on: ubuntu-latest | ||
permissions: | ||
issues: write | ||
pull-requests: write | ||
steps: | ||
- name: Set issue url and creator login | ||
if: ${{ github.event.issue }} | ||
run: | | ||
echo "ISSUE=${{ github.event.issue.html_url }}" >> $GITHUB_ENV | ||
echo "CREATOR=${{ github.event.issue.user.login }}" >> $GITHUB_ENV | ||
- name: Set pull_request url and creator login | ||
if: ${{ github.event.pull_request }} | ||
# yamllint disable rule:line-length | ||
run: | | ||
echo "PR=${{ github.event.pull_request.html_url }}" >> $GITHUB_ENV | ||
echo "CREATOR=${{ github.event.pull_request.user.login }}" >> $GITHUB_ENV | ||
# yamllint enable rule:line-length | ||
- name: Assign issue to creator (issue) | ||
if: ${{ github.event.issue }} | ||
run: gh issue edit ${{ env.ISSUE }} --add-assignee ${{ env.CREATOR }} | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
- name: Assign issue to creator (PR) | ||
if: ${{ github.event.pull_request }} | ||
run: gh pr edit ${{ env.PR }} --add-assignee ${{ env.CREATOR }} | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
|
||
required-labels: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: mheap/github-action-required-labels@v4 | ||
with: | ||
mode: minimum | ||
count: 1 | ||
labels: "T:enhancement, T:documentation, T:code-hygiene, T:bug, T:adr, T:sdk, T:testing, T:question, T:dependencies, T:spec-and-docs, T:da-integration, T:dev-usability-and-ux" # yamllint disable-line rule:line-length |
This file was deleted.
Oops, something went wrong.
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.
I think this is fine to leave in