Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Introduce new Runtime API endpoint for fetching the validation data #3728

Merged
merged 9 commits into from
Oct 22, 2021

Conversation

slumber
Copy link
Contributor

@slumber slumber commented Aug 26, 2021

The corresponding implementers guide section will be updated along with the candidate validation subsystem as a part of #3655
Resolves #3700

cumulus companion: paritytech/cumulus#693

`persisted_validation_data_with_code_hash` that will be used
by the candidate validation subsystem in order to decrease amount
of runtime API requests.
Copy link
Contributor

@pepyakin pepyakin left a comment

Choose a reason for hiding this comment

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

One of the things we would still need to do is to update the implementer's guide.

runtime/parachains/src/paras.rs Outdated Show resolved Hide resolved
primitives/src/v1/mod.rs Outdated Show resolved Hide resolved
runtime/parachains/src/runtime_api_impl/v1.rs Show resolved Hide resolved
@slumber
Copy link
Contributor Author

slumber commented Aug 27, 2021

One of the things we would still need to do is to update the implementer's guide.

Yea, it will also be necessary to update the subsystem diagram, but first I want to finalize the PR, e.g. there're naming issues, as you mentioned.

@slumber slumber added A0-please_review Pull request needs code review. B7-runtimenoteworthy C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. labels Sep 27, 2021
@bkchr
Copy link
Member

bkchr commented Oct 1, 2021

We should probably also bump the version of the runtime api? And add some fallback for when it is not available?

@pepyakin
Copy link
Contributor

pepyakin commented Oct 1, 2021

The code that depends on this PR, #3655, is actually a low-priority one. We can as well delay the dependent PR until after this gets on the relay-chain. This way we will save us some effort by not introducing a fallback.

Do you think it is sensible?

@bkchr
Copy link
Member

bkchr commented Oct 1, 2021

Okay, sounds fine.

@pepyakin
Copy link
Contributor

bot merge

@paritytech-processbot paritytech-processbot bot merged commit fc32642 into master Oct 22, 2021
@paritytech-processbot paritytech-processbot bot deleted the slumber-runtime-api-upgrade branch October 22, 2021 12:49
ordian added a commit that referenced this pull request Oct 27, 2021
* master: (73 commits)
  Fix XCM Teleport Benchmark (#4146)
  Allow Queries and Subscriptions (#4150)
  Fix weights on hard-coded XCM fragments (#4144)
  Bump spec versions (#4142)
  Bump libc from 0.2.104 to 0.2.105 (#4141)
  Enable bags-list pallet in polkadot (#4080)
  Move artifacts states into memory in PVF validation host (#3907)
  Introduce new Runtime API endpoint for fetching the validation data (#3728)
  export xcm_pallet config (#4116)
  Apply cargo fmt (#4122)
  remove logging from the check-dependent-* job (#4120)
  Remove stale migrations (#4107)
  Bump structopt from 0.3.23 to 0.3.25 (#4098)
  remove hardcoded pipeline scripts tag (#4109)
  Bump libc from 0.2.103 to 0.2.104 (#4099)
  some spelling fixes (#4088)
  polkadot: remove call filters on registrar pallets (#4093)
  Fix typos in docs (#4092)
  Rename Statemint where appropriate (#4087)
  bump substrate (#4091)
  ...
emostov pushed a commit that referenced this pull request Nov 1, 2021
…3728)

* Introduce new Runtime API endpoint

`persisted_validation_data_with_code_hash` that will be used
by the candidate validation subsystem in order to decrease amount
of runtime API requests.

* Node-side part of new runtime API request

* Define code hash getter via macro

* Rename new endpoint to `assumed_validation_data`

* Docs for runtime API impl of new endpoint

* AssumedValidationData specialized request function

* fmt
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New Parachains Runtime API tailored for backing.
3 participants