Skip to content
This repository has been archived by the owner on Jun 24, 2024. It is now read-only.

feat: getting access to DA's consensus state #101

Merged
merged 10 commits into from
Mar 22, 2024

Conversation

Farhad-Shabani
Copy link
Member

@Farhad-Shabani Farhad-Shabani commented Mar 15, 2024

Closes: #7

Description

  • Ended up introducing the sov-consensus-state-tracker module, a custom kernel that enables the extraction of specific ConsensusState relevant fields from the DA's header.
  • This module is generic across different DAs and offers extensibility through the implementation of the HasConsensusState trait for each targeted DA (Of course at the cost of some complexity).
  • When integrating with the rollup's starter, each implementation can be individually enabled using feature flags.
  • In addition, when integrating IBC with the starter, the sov-consensus-state-tracker should be utilized to import all objects relevant to a particular DA, so as not to have to deal with the Rust orphan rule in the starter.

PR author checklist

  • Linked to GitHub issue.
  • Updated code comments and documentation (e.g., docs/).

@Farhad-Shabani Farhad-Shabani marked this pull request as ready for review March 15, 2024 04:39
@Farhad-Shabani Farhad-Shabani merged commit 5d860e9 into main Mar 22, 2024
10 checks passed
@Farhad-Shabani Farhad-Shabani deleted the farhad/try-has-consensus-poc branch March 22, 2024 16:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Implement ValidationContext::host_consensus_state() for IbcExecutionContext
2 participants