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

Add option to only relay free headers to substrate-relay #2884

Merged
merged 68 commits into from
Apr 25, 2024

Conversation

svyatonik
Copy link
Contributor

@svyatonik svyatonik commented Mar 18, 2024

related to #2871

Adds support of #2873 to our standalone GRANDPA and finality relays. The main thing that is left here is to actually test it and for that I need to get updated Rococo<>Westend runtimes first

UPD: obviously I'll also need to change tx submitter to support that new free_execution_expected flag

…of_ex + check it from the transaction extension
…ge GRANDPA transactions are obsolete and, if not, it may apply priority boost to
…coming from CheckAndBoostBridgeGrandpaTransactions
@svyatonik svyatonik changed the title Sv relay free headers Add option to only relay free headers to substrate-relay Mar 18, 2024
@svyatonik
Copy link
Contributor Author

svyatonik commented Mar 19, 2024

I've updated zn test "one" here: https://github.com/paritytech/polkadot-sdk/tree/sv-try-new-bridge-fees

Two remaining issues there:

  • by some reason the parachain relayer account loses its funds, even though all his transactions are free. Investigating should be fixed by d1045df;
  • not sure why, but parachain heads are updated at every 20th block, even though the free interval is set to 10 🤦 fixed intervals won't work because GRANDPA finalizes range of headers

@bkontur bkontur self-requested a review March 25, 2024 13:34
@svyatonik
Copy link
Contributor Author

Superseded by paritytech/polkadot-sdk#4157. BUT. Chain-specific changes from this PR must be merged afteer polkadot-sdk is upgraded. So I'm leaving this PR open

github-merge-queue bot pushed a commit to paritytech/polkadot-sdk that referenced this pull request Apr 25, 2024
…4157)

Original PR:
paritytech/parity-bridges-common#2884. Since
chain-specific code lives in the `parity-bridges-common` repo, some
parts of original PR will require another PR

---------

Co-authored-by: Adrian Catangiu <[email protected]>
@svyatonik svyatonik requested a review from a team as a code owner April 25, 2024 08:01
@svyatonik svyatonik changed the base branch from sv-refund-non-mandatory-headers to master April 25, 2024 08:02
@svyatonik svyatonik removed the request for review from a team April 25, 2024 09:26
@svyatonik svyatonik added the A-release A new relayer release label Apr 25, 2024
@svyatonik svyatonik merged commit 2573d65 into master Apr 25, 2024
12 checks passed
@svyatonik svyatonik deleted the sv-relay-free-headers branch April 25, 2024 10:10
svyatonik added a commit to paritytech/polkadot-sdk that referenced this pull request May 8, 2024
…4157)

Original PR:
paritytech/parity-bridges-common#2884. Since
chain-specific code lives in the `parity-bridges-common` repo, some
parts of original PR will require another PR

---------

Co-authored-by: Adrian Catangiu <[email protected]>
TarekkMA pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this pull request Aug 2, 2024
…aritytech#4157)

Original PR:
paritytech/parity-bridges-common#2884. Since
chain-specific code lives in the `parity-bridges-common` repo, some
parts of original PR will require another PR

---------

Co-authored-by: Adrian Catangiu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-release A new relayer release P-Relay
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants