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

Create V2 Execution API to support Forma migration #1993

Closed
ethanoroshiba opened this issue Feb 20, 2025 · 0 comments · Fixed by #1962
Closed

Create V2 Execution API to support Forma migration #1993

ethanoroshiba opened this issue Feb 20, 2025 · 0 comments · Fixed by #1962
Assignees
Labels
enhancement New feature or request proto pertaining to the Astria Protobuf spec

Comments

@ethanoroshiba
Copy link
Contributor

We are significantly changing the way Geth and Conductor communicate, hence we need to break the outlined changes into v2 to support restart of Conductor via "sessions", as determined by the rollup:

  • Break protos into 1-1-1
  • Introduce ExecutionSession and associated RPC to replace GetCommitmentState and GetGenesisInfo:
message ExecutionSession {
  string session_id = 1;
  ExecutionSessionParameters execution_config = 2;
  CommitmentState commitment_state = 3;
}
  • Replace GenesisInfo with ExecutionSessionParameters:
message ExecutionSessionParameters {
  astria.primitive.v1.RollupId rollup_id = 1;
  uint64 rollup_start_block_number = 2;
  uint64 rollup_end_block_number = 3;
  string sequencer_chain_id = 4;
  uint64 sequencer_start_block_height = 5;
  string celestia_chain_id = 6;
  uint64 celestia_block_variance = 7;
}
@ethanoroshiba ethanoroshiba added enhancement New feature or request proto pertaining to the Astria Protobuf spec labels Feb 20, 2025
@ethanoroshiba ethanoroshiba self-assigned this Feb 20, 2025
@ethanoroshiba ethanoroshiba linked a pull request Feb 26, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request proto pertaining to the Astria Protobuf spec
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant