Feat/hyperbridge onboarding - #143
Merged
Merged
Conversation
…e the right value
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Two corrections found while onboarding Orbinum onto Hyperbridge's Paseo testnet.
The Paseo relayer config was missing
slot_duration. Tesseract'sHostConfig(tesseract/consensus/grandpa/src/lib.rs:85) declares it without a default, so the relayer aborted withmissing field slot_durationbefore connecting to anything. The local config had it; the production template did not, which is why local validation never caught this. Added to both sections, along with theconsensus_update_frequencytheir docs specify.The fee model was documented backwards. The comments claimed a zero relayer fee would let anyone "give away work to a third party", and that opening
DispatchOriginrequired deciding a price first. Hyperbridge's fee docs and their own dispatcher say otherwise:With a zero fee nothing is transferred and nobody is owed. The real effect is that Hyperbridge's permissionless relayers won't deliver, they prioritise by fee. Orbinum self-relays, which their docs call the intended integration, so zero is the correct value, not a placeholder.
Also corrected:
POLICY = falseon theFeeHandlerwas described as disabling relayer fees. It governs the inbound side, whether whoever submits an incoming message pays its execution weight. Unrelated to the outbound relayer fee inFeeMetadata.Impact
Comments and one config file. No code changes —
git diffon the Rust files is entirely//lines.Onboarding status
Done on-chain with these fixes, verified live on testnet:
Hyperbridge → Orbinum verifies now. The other direction waits on polytope-labs/hyperbridge#1200, which only they can action.