[AHM] Relax the requirement for RC-Client to receive +1 session reports#8702
Merged
kianenigma merged 10 commits intomasterfrom Jun 4, 2025
Merged
[AHM] Relax the requirement for RC-Client to receive +1 session reports#8702kianenigma merged 10 commits intomasterfrom
kianenigma merged 10 commits intomasterfrom
Conversation
Contributor
Author
|
/cmd prdoc --bump patch |
kianenigma
changed the title
kianenigma
changed the title
Contributor
|
Command "prdoc --bump patch --audience runtime-dev" has failed ❌! See logs here |
…kill-last-ending-session-index
Ank4n
reviewed
Jun 3, 2025
Ank4n
approved these changes
Jun 3, 2025
…ytech/polkadot-sdk into kiz-kill-last-ending-session-index
|
All GitHub workflows were cancelled due to failure one of the required jobs. |
Contributor
Author
|
/cmd fmt |
auto-merge was automatically disabled
June 4, 2025 23:04
Pull Request is not mergeable
8 tasks
pgherveou
pushed a commit
that referenced
this pull request
Jun 11, 2025
…ts (#8702) Something that we learned in westend: if `ah-client` decides to go to `Buffered` mode ever again, it might skip some sessions. Atm, we were enforcing sessions to be strictly incremented by one once received by RC client. This could cause us to drop some validator reward points. As a simple fix, this PR relaxes this check such that if the last repot was `x`: 1. x+1 is our expected behavior 2. x+2 and more is still accepted, but we emit warning events 3. x and below is still dropped. This can only happen due to: unforeseen bug in RC, or XCM failing to send messages in order. --------- Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
ordian
added a commit
that referenced
this pull request
Jun 12, 2025
* master: (62 commits) release/build-macos-binaries: add missing FEATURES argument (#8816) Add XCM Precompile to `pallet-xcm` (#8693) [Release|CI/CD] Exclude test runtimes from the runtimes build (#8820) Add freebsd sysinfo for telemetry (#7985) release-reusable-rc-build: add optional `features` input that can be considered for nodes building (#8755) [Staking] Cleanups and some improvements (#8701) Fix typos in 3 files in Implementers Guide (#8799) Update `RemoteExporter` docs to reflect removal of `forward_id_for` (#8795) Snowbridge: enforce fee when registering Polkadot native asset (#8725) Bump the ci_dependencies group across 1 directory with 7 updates (#8788) Docker hub 'master' image short sha (#8790) [Release|CI/CD] Combine branch-off and RC automation flows (#8754) Move Transaction depth limit checks (#8750) Add genesis presets for remaining runtimes in polkadot-parachain-bin (#8426) Do not make pallet-identity benchmarks signature-dependent (#8179) Introduction of Approval Slashes [Disabling Strategy Stage 4] (#6827) [AHM] Prepare For Westend Cleanup (#8715) Actually use RP offset in YAP parachain (#8745) [AHM] Relax the requirement for RC-Client to receive +1 session reports (#8702) Don't read storage items in logging (#8749) ...
alvicsam
pushed a commit
that referenced
this pull request
Oct 17, 2025
…ts (#8702) Something that we learned in westend: if `ah-client` decides to go to `Buffered` mode ever again, it might skip some sessions. Atm, we were enforcing sessions to be strictly incremented by one once received by RC client. This could cause us to drop some validator reward points. As a simple fix, this PR relaxes this check such that if the last repot was `x`: 1. x+1 is our expected behavior 2. x+2 and more is still accepted, but we emit warning events 3. x and below is still dropped. This can only happen due to: unforeseen bug in RC, or XCM failing to send messages in order. --------- Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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
3 participants
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.
Something that we learned in westend: if
ah-clientdecides to go toBufferedmode ever again, it might skip some sessions. Atm, we were enforcing sessions to be strictly incremented by one once received by RC client. This could cause us to drop some validator reward points.As a simple fix, this PR relaxes this check such that if the last repot was
x: