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

chainHead: Always report discarded items for storage operations #6760

Merged
merged 6 commits into from
Dec 5, 2024

Conversation

lexnv
Copy link
Contributor

@lexnv lexnv commented Dec 4, 2024

This PR ensures that substrate always reports discarded items as zero.
This is needed to align with the rpc-v2 spec

Closes: #6683

cc @paritytech/subxt-team

lexnv added 2 commits December 4, 2024 15:48
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
@lexnv lexnv added the T3-RPC_API This PR/Issue is related to RPC APIs. label Dec 4, 2024
@lexnv lexnv self-assigned this Dec 4, 2024
@lexnv lexnv added the I5-enhancement An additional feature request. label Dec 4, 2024
@lexnv
Copy link
Contributor Author

lexnv commented Dec 4, 2024

/cmd prdoc --audience node_dev --bump patch

actions-user and others added 2 commits December 4, 2024 15:55
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
@niklasad1 niklasad1 added the A4-needs-backport Pull request must be backported to all maintained releases. label Dec 4, 2024
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
@paritytech-workflow-stopper
Copy link

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/12166203776
Failed job name: test-linux-stable-no-try-runtime

@lexnv lexnv added this pull request to the merge queue Dec 5, 2024
Merged via the queue into master with commit df1375e Dec 5, 2024
199 of 200 checks passed
@lexnv lexnv deleted the lexnv/chainhead-resp branch December 5, 2024 13:47
sylvaincormier pushed a commit to sylvaincormier/polkadot-sdk that referenced this pull request Dec 5, 2024
…tytech#6760)

This PR ensures that substrate always reports discarded items as zero.
This is needed to align with the rpc-v2 spec

Closes: paritytech#6683


cc @paritytech/subxt-team

---------

Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Co-authored-by: GitHub Action <action@github.com>
lexnv added a commit that referenced this pull request Dec 11, 2024
This PR ensures that substrate always reports discarded items as zero.
This is needed to align with the rpc-v2 spec

Closes: #6683


cc @paritytech/subxt-team

---------

Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Co-authored-by: GitHub Action <action@github.com>
EgorPopelyaev pushed a commit that referenced this pull request Dec 11, 2024
…rage operations (#6842)

Backport PR for:
- #6760

cc @paritytech/subxt-team

Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Co-authored-by: GitHub Action <action@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A4-needs-backport Pull request must be backported to all maintained releases. I5-enhancement An additional feature request. T3-RPC_API This PR/Issue is related to RPC APIs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

JSON-RPC: chainHead_v1_storage started event is missing discardedItems property
4 participants