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

Remove without_storage_info for the poa pallet - V2 #633

Merged
merged 8 commits into from Jul 28, 2022

Conversation

rajesh-nodle
Copy link
Contributor

Part of #173

HighLights

  • Storage Validators is moved to bounded vector.
  • Upperlimit for Oracles is Configured via MaxValidators.
  • Introduced new events ValidatorsUpdated(u32) & ValidatorsMaxOverflow(u32, u32)
  • UnitTest Extension::Done
  • Storage Migration (try_runtime)

TODO

Ready for final Intake Review

Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
@codecov
Copy link

codecov bot commented Jul 12, 2022

Codecov Report

Merging #633 (e39c9f1) into master (a331f30) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #633      +/-   ##
==========================================
- Coverage   89.74%   89.73%   -0.01%     
==========================================
  Files          33       33              
  Lines        6238     6232       -6     
==========================================
- Hits         5598     5592       -6     
  Misses        640      640              
Impacted Files Coverage Δ
runtimes/eden/src/pallets_consensus.rs 0.00% <ø> (ø)
pallets/poa/src/lib.rs 35.29% <10.00%> (-9.16%) ⬇️
pallets/staking/src/lib.rs 85.74% <50.00%> (+0.05%) ⬆️
pallets/poa/src/tests.rs 75.75% <75.00%> (-11.75%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a331f30...e39c9f1. Read the comment docs.

R.Rajeshkumar added 2 commits July 14, 2022 07:11
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
@@ -34,7 +34,10 @@ pub mod weights;

use frame_support::pallet;
pub(crate) mod hooks;
mod migrations;

// TODO:: Take it part of PR621

Choose a reason for hiding this comment

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

Is this TODO still outstanding?

Choose a reason for hiding this comment

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

I suppose it needs to be pulled in as part of #621?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup, has to be taken care part of #621

Copy link
Contributor

Choose a reason for hiding this comment

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

Is it possible that we don't make any changes to this file as part of this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, right now not possible, mainly due to the requirements of moving validatorsets from pallet members to invulnerables storage.

@secretfader
Copy link

A few questions, but otherwise LGTM. If this PR should be merged in a certain order (relating to other work that is currently in-flight), that would be helpful to know for future reference.

R.Rajeshkumar and others added 4 commits July 22, 2022 13:44
pallets/poa/src/migrations.rs Show resolved Hide resolved
pallets/poa/src/migrations.rs Show resolved Hide resolved
pallets/poa/src/tests.rs Show resolved Hide resolved
pallets/staking/src/mock.rs Outdated Show resolved Hide resolved
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
@rajesh-nodle rajesh-nodle merged commit 8a1b78c into master Jul 28, 2022
@rajesh-nodle rajesh-nodle deleted the shamb0/v2_bounded_poa branch July 28, 2022 04:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants