Skip to content

Conversation

@kpinter-iohk
Copy link
Contributor

@kpinter-iohk kpinter-iohk commented Oct 17, 2025

Description

This PR implements the following using Dolos:

  • SidechainRpcDataSourceImpl
  • McHashDataSourceImpl
  • AuthoritySelectionDataSourceImpl

It also fixes some bugs in the miniBF client.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages.
  • The size limit of 400 LOC isn't needlessly exceeded
  • The PR refers to a JIRA ticket (if one exists)
  • New tests are added if needed and existing tests are updated.
  • New code is documented and existing documentation is updated.
  • Relevant logging and metrics added
  • Any changes are noted in the changelog.md for affected crate
  • Self-reviewed the diff

@kpinter-iohk kpinter-iohk force-pushed the ETCM-12351-dolos-auth-selection-data-source branch 2 times, most recently from 5086ea3 to 119950d Compare October 20, 2025 14:45
@kpinter-iohk kpinter-iohk changed the base branch from ETCM-12351-dolos-client to master October 20, 2025 14:57
@kpinter-iohk kpinter-iohk force-pushed the ETCM-12351-dolos-auth-selection-data-source branch from 119950d to 175175c Compare October 30, 2025 11:58
@kpinter-iohk kpinter-iohk marked this pull request as ready for review October 30, 2025 12:54
@kpinter-iohk kpinter-iohk force-pushed the ETCM-12351-dolos-auth-selection-data-source branch from ccf95d0 to 17835d2 Compare November 3, 2025 08:45
@kpinter-iohk
Copy link
Contributor Author

Merged with #1075


fn format_asset_id(asset_id: &AssetId) -> String {
let AssetId { policy_id, asset_name } = asset_id;
format!("{}{}", &policy_id.to_hex_string()[2..], &asset_name.to_hex_string()[2..])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it really concat without a separator?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this has been tested against a running Dolos instance too. (docs)

_reference_timestamp: sp_timestamp::Timestamp,
) -> Result<Option<MainchainBlock>> {
Err("not implemented".into())
Ok(Some(from_block_content(self.client.blocks_latest().await?)?))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will fail when syncing to existing network, wouldn't it?

@kpinter-iohk kpinter-iohk deleted the ETCM-12351-dolos-auth-selection-data-source branch November 6, 2025 09:02
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