Add kurtosis devnet for celo-rebase-13#394
Open
Kourin1996 wants to merge 107 commits intocelo-rebase-14from
Open
Add kurtosis devnet for celo-rebase-13#394Kourin1996 wants to merge 107 commits intocelo-rebase-14from
Kourin1996 wants to merge 107 commits intocelo-rebase-14from
Conversation
Adding funding.json file required to apply for retropgf
* Add script to update `celo-org/op-geth` dependencies Signed-off-by: Maximilian Langenfeld <15726643+ezdac@users.noreply.github.com> * Add github action for update-geth script Signed-off-by: Maximilian Langenfeld <15726643+ezdac@users.noreply.github.com> * Use inline geth base-ref Signed-off-by: Maximilian Langenfeld <15726643+ezdac@users.noreply.github.com> * Discard automatic fields Signed-off-by: Maximilian Langenfeld <15726643+ezdac@users.noreply.github.com> * token perms Signed-off-by: Maximilian Langenfeld <15726643+ezdac@users.noreply.github.com> * Use read-only GCP SA Signed-off-by: Maximilian Langenfeld <15726643+ezdac@users.noreply.github.com> * Better parameter handling in update-geth action Signed-off-by: Maximilian Langenfeld <15726643+ezdac@users.noreply.github.com> * Remove `update-celo-geth` cmd from justfile --------- Signed-off-by: Maximilian Langenfeld <15726643+ezdac@users.noreply.github.com> Co-authored-by: alvarof2 <fernandez.f.alvaro@gmail.com>
The one from Optimism seems to be registered as private in the orb registry.
This will be removed upstream in ethereum-optimism#15397
Setting `open-pull-requests-limit: 0` prevent version updates but should keep the security scans and updates according to https://docs.github.com/en/code-security/dependabot/dependabot-security-updates/configuring-dependabot-security-updates#overriding-the-default-behavior-with-a-configuration-file > If you only require security updates and want to exclude version > updates, you can set open-pull-requests-limit to 0 in order to prevent > version updates for a given package-ecosystem.
* mark it as executable * allow running on both MacOS and linux (GNU vs BSD tool differences) * terminate on errors * less strict go.mod matching to allow applying it directly to upstream
This cheatcode is used to store the Celo L2 predeploy addresses in a JSON file when run with forge. Inside TestEndToEndApply/initial_chain , the same code is called but we don't care for the generated JSON file. So just returning `nil` makes the test work without any disadvantages.
Remove common/interfaces/IExchange.sol (duplicate) We keep the one in the `mento` directory. Remove common/libraries/ReentrancyGuard.sol (duplicate) Directly use the original from OpenZeppelin.
so that BridgedETH and other bridged ERC20 tokens can be used as fee currencies. I won't included testing until the FeeCurrencyDirectory is ready.
Until we fix the errors caused by enabling it.
* isthmus: Update L1Block contract bytecode --------- Co-authored-by: Gastón Ponti <gaston.ponti@clabs.co> Co-authored-by: Gaston Ponti <pontigaston@gmail.com>
Changed shebang from #!/usr/bin/env fish to #!/bin/sh Replaced $argv with "$@" for command line arguments Replaced [ -z "$argv" ] with [ -z "$1" ] to check for arguments Replaced status filename with basename "$0" to get script name Changed return to exit 1 for script termination Used standard variable assignment and export syntax Used "$@" in the for loop to properly handle arguments with spaces Used $(...) for command substitution (POSIX compliant)
…' into Kourin1996/celo-rebase-13-devnet
…celo-rebase-13-devnet
palango
reviewed
Jul 9, 2025
|
|
||
| # Celo Isthmus devnet | ||
| celo-isthmus-devnet: | ||
| ./setup-local-repos.sh Kourin1996/celo-rebase-13-contracts-devnet celo-rebase-13-no-HF-overrides |
There was a problem hiding this comment.
What changes do those branches include? Should we merge those first?
Author
There was a problem hiding this comment.
Kourin1996/celo-rebase-13-contracts-devnet is a branch for op-deployer and foundry scripts. This branch is forked from a different branch celo-contracts/v3.0.0--1.
celo-rebase-13-no-HF-overrides is a branch of op-geth forked from `celo-rebase-13. It doesn't include overriding holocene and isthmus activation times.
palango
reviewed
Jul 9, 2025
kurtosis-devnet/justfile
Outdated
| ./setup-local-repos.sh Kourin1996/celo-rebase-13-contracts-devnet celo-rebase-13-no-HF-overrides | ||
| {{just_executable()}} devnet celo-isthmus.yaml | ||
|
|
||
| celo-isthmus-minimal-devnet: |
There was a problem hiding this comment.
Not sure if we want to merge this into our development branch.
palango
reviewed
Jul 9, 2025
| PragueTime: config.IsthmusTime(l1StartTime), | ||
| InteropTime: config.InteropTime(l1StartTime), | ||
| Cel2Time: config.RegolithTime(l1StartTime), | ||
| GingerbreadBlock: big.NewInt(0), |
Author
There was a problem hiding this comment.
This is necessary for avoiding genesis hash mismatch between op-geth and op-node.
Kourin1996
changed the title
|
Hi Celo Team! I'm Sishan from Espresso. We're currently working on a devnet for the Celo-Espresso integration, but I ran into an issue when running this 2025-07-28T18:58:04.904434Z ERROR foundry_compilers_artifacts_solc::sources: error="/home/ec2-user/optimism/kurtosis-devnet/optimism-contracts/packages/contracts-bedrock/lib/safe-contracts/contracts/GnosisSafe.sol": No such file or directory (os error 2)
Error: failed to resolve file: "/home/ec2-user/optimism/kurtosis-devnet/optimism-contracts/packages/contracts-bedrock/lib/safe-contracts/contracts/GnosisSafe.sol": No such file or directory (os error 2); check configured remappings
--> /home/ec2-user/optimism/kurtosis-devnet/optimism-contracts/packages/contracts-bedrock/scripts/deploy/DeployAuthSystem.s.sol
safe-contracts/GnosisSafe.sol
error: Recipe `build-no-tests` failed on line 33 with exit code 1
exit status 1
error: Recipe `devnet` failed with exit code 1
error: Recipe `celo-isthmus-devnet` failed on line 107 with exit code 1It seems like the safe-contracts dependency (specifically GnosisSafe.sol) is missing or not properly linked. Do you know what might be causing this, or if I’m missing a setup step? |
Author
|
Hello, @dailinsubjam! Thank you for trying our devnet. I could reproduce with newly cloned repository. I missed this problem cause it didn't occur in existing repository. We're working on hotfix and I'll let you know once it's ready. Sorry for the inconvenience. |
Author
|
Hi, @dailinsubjam. Now it's resolved. You can simply delete |
|
Thanks @Kourin1996 ! It works for me now. |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
14 participants
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.
This PR adds configuration for kurtosis devnet on
celo-rebase-13.This PR depends on another PR #396 (op-deployer & contract scripts), which targets a different branch, celo-contracts/v3.0.0--1
To run kurtosis devnet, run the following commands:
Address #394