Skip to content

fix: charge dedicated-synchronizer traffic like the global synchronizer - #65

Merged
sadiq1971 merged 7 commits into
mainfrom
fix/dedicated-sync-traffic-like-global
Oct 4, 2026
Merged

sadiq1971 merged 7 commits into
mainfrom
fix/dedicated-sync-traffic-like-global

Conversation

@sadiq1971

@sadiq1971 sadiq1971 commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Closes ChainSafe/canton-extending-mainnet#139

Summary: the sync operator forces a zero base rate, takes the per-recipient read cost from the global synchronizer's on-ledger amulet config through Scan, and always enables free confirmation responses, so none of the three is configurable. The sync operator integration test checks the dedicated synchronizer ends up with the global synchronizer's settings.

…er [ci]

Signed-off-by: sadiq1971 <sadiqurr8@gmail.com>
@sadiq1971 sadiq1971 self-assigned this Sep 30, 2026
@sadiq1971
sadiq1971 marked this pull request as ready for review September 30, 2026 15:59
Signed-off-by: sadiq1971 <sadiqurr8@gmail.com>
Signed-off-by: sadiq1971 <sadiqurr8@gmail.com>
…fig [ci]

Signed-off-by: sadiq1971 <sadiqurr8@gmail.com>
Signed-off-by: sadiq1971 <sadiqurr8@gmail.com>

@moritzkiefer-da moritzkiefer-da left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

thanks

Signed-off-by: sadiq1971 <sadiqurr8@gmail.com>
Signed-off-by: sadiq1971 <sadiqurr8@gmail.com>
@sadiq1971
sadiq1971 merged commit f25dceb into main Oct 4, 2026
182 of 187 checks passed
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.

[P2-E5.10] Charge traffic on the dedicated synchronizer the way the global one does

2 participants