Skip to content

Conversation

@rswanson
Copy link
Member

Adds the genesis file for parmigian host and makes it available in the standard signet container so we don't have testnet 1-off containers to manage

@rswanson rswanson requested a review from a team as a code owner December 11, 2025 16:25
@rswanson rswanson self-assigned this Dec 11, 2025
@prestwich
Copy link
Member

so a few things

  • we make the genesis available in the signet-genesis package. can we reuse that instead of maintaining 2 of them?
  • the json is parmigiana.genesis.json but you specify host in the PR body. is it parmigiana or parmigiana host?

Copy link
Member Author

its host, I can update the name accordingly

Copy link
Member Author

rswanson commented Dec 11, 2025

this is for usage by the running node as part of startup (reth requirement not our code) its for the --chain=/network_configs/parmigiana-host.genesis.json flag on the start command

@prestwich
Copy link
Member

this is for usage by the running node as part of startup (reth requirement not our code) its for the --chain=/network_configs/parmigiana-host.genesis.json flag on the start command

yes I understand the purpose. Given we already have a maintained version of this file, is there a way we can re-use that maintained version instead of maintaining a second version?

Copy link
Member Author

that is the genesis for the rollup is it not?

@prestwich
Copy link
Member

Copy link
Member Author

oh i see, how would I make that file available on the filesystem? wouldn't I need to do something a bit hacky like checkout that repo separately in this ci and pull it into the dockerfile?

Copy link
Member Author

It feels much cleaner to keep the container deps in the same repo as the dockerfile

Copy link
Member Author

I could move the dockerfile to node-components but then i'd have the inverse problem with building the binary for the container

@prestwich
Copy link
Member

i'm going to approve this PR, but can you make a ticket to solution this more? we can likely modify node init to bypass that CLI flag and use a genesis from the package

@prestwich
Copy link
Member

oh, dockerfile is still called "pecorino"

Copy link
Member Author

no its not I deleted that file and the corresponding workflow

@rswanson rswanson merged commit 8c749ef into main Dec 11, 2025
5 checks passed
@rswanson rswanson deleted the swanny/parmigiana-container branch December 11, 2025 16:42
@prestwich
Copy link
Member

oh yep dope

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants