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

Companion for Generate storage info for pallet im_online #9654 #3744

Merged
3 commits merged into from
Sep 20, 2021
Merged

Companion for Generate storage info for pallet im_online #9654 #3744

3 commits merged into from
Sep 20, 2021

Conversation

georgesdib
Copy link
Contributor

@georgesdib georgesdib commented Aug 30, 2021

Companion to paritytech/substrate#9654

Adding MaxKeys, MaxPeerInHeartbeats and MaxPeerIdEncodingSize to pallet im_online

@shawntabrizi @KiChjang @thiolliere

Copy link
Contributor

@gui1117 gui1117 left a comment

Choose a reason for hiding this comment

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

looks good to me

runtime/kusama/src/lib.rs Outdated Show resolved Hide resolved
@KiChjang KiChjang added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes 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 19, 2021
@KiChjang
Copy link
Contributor

KiChjang commented Sep 19, 2021

@georgesdib Do you mind cherry-picking your commits off to another branch instead of working on top of master? I think our CI bot is unable to build this PR due to different master commit histories of your repo vs ours.

@georgesdib georgesdib closed this Sep 19, 2021
`MaxPeerDataEncodingSize` to paller `im_online` after substrate changes
@georgesdib
Copy link
Contributor Author

@KiChjang I have rebased and committed from scratch, hope this solves the issue

@georgesdib georgesdib reopened this Sep 19, 2021
@ghost
Copy link

ghost commented Sep 20, 2021

Waiting for commit status.

@ghost
Copy link

ghost commented Sep 20, 2021

Merge aborted: Checks failed for 75d1299

@gui1117
Copy link
Contributor

gui1117 commented Sep 20, 2021

bot merge

@ghost
Copy link

ghost commented Sep 20, 2021

Trying merge.

@ghost ghost merged commit 3e92a34 into paritytech:master Sep 20, 2021
ordian added a commit that referenced this pull request Sep 20, 2021
* master:
  Bump `parity-util-mem` to 0.10.2 (#3897)
  Revert "remove connected disconnected state only (#3868)" (#3896)
  Companion for Generate storage info for pallet im_online #9654 (#3744)
  Gather memory usage statistics through `parity-util-mem` (#3893)
  ci: cache seed in fuzzer job (#3885)
  Add benchmarking for parachain runtime configuration pallet (#3862)
  Run fmt (#3887)
  remove connected disconnected state only (#3868)
ordian added a commit that referenced this pull request Sep 21, 2021
* master:
  weights: pallet-staking & pallet-bags-list (#3872)
  Re-enable metadata compare CI check (#3895)
  Bump `parity-util-mem` to 0.10.2 (#3897)
  Revert "remove connected disconnected state only (#3868)" (#3896)
  Companion for Generate storage info for pallet im_online #9654 (#3744)
  Gather memory usage statistics through `parity-util-mem` (#3893)
  ci: cache seed in fuzzer job (#3885)
  Add benchmarking for parachain runtime configuration pallet (#3862)
  Run fmt (#3887)
  remove connected disconnected state only (#3868)
  Dispute distribution improvements (#3853)
  Companion for Store voters in unsorted bags to get good stake iteration properties (#3413)
  Add info logging to dispute activation (#3870)
  update Rococo.json (#3871)
  Companion for #9764 (Force Unreserve) (#3840)
This pull request was closed.
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. B0-silent Changes should not be mentioned in any release notes 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.

3 participants