-
Notifications
You must be signed in to change notification settings - Fork 690
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
On-demand & Coretime parachain upgrade fees #2372
Closed
Closed
Changes from all commits
Commits
Show all changes
109 commits
Select commit
Hold shift + click to select a range
111a536
initial commit
Szegoo 36ff907
required fee for upgrading
Szegoo 628b3f0
add basic docs
Szegoo 0439ef9
start making tests work
Szegoo f12aaf2
tests passing
Szegoo adeb298
first test
Szegoo 58b1219
refund solution - incomplete
Szegoo cdf22f6
progress
Szegoo ab087f6
incomplete implementation
Szegoo cef5196
small fixes
Szegoo d7eba24
new logic almost complete
Szegoo 3758a15
cleanup
Szegoo 30a051a
complete first test & grammar
Szegoo bf1b5e3
new test case
Szegoo eed4c4d
better approach
Szegoo f852348
refactor
Szegoo 04d22e0
cleanup & small fixes
Szegoo bdba7cb
new test & fixes
Szegoo 51f2d16
fix & new tests
Szegoo 06b02d5
fix?
Szegoo 8604146
should compile
Szegoo 8cbf899
nitpicks
Szegoo 74d680f
make this a non-breaking change
Szegoo 10d681a
add migration
Szegoo c54492b
Merge branch 'master' into para-upgrade-fees
Szegoo daa0dcc
add test
Szegoo dd54e5f
fix try-runtime
Szegoo 6c90742
implement the better solution
Szegoo e82525d
safe fund handling
Szegoo cfd3092
improve docs
Szegoo 14b23df
refactoring | incomplete
Szegoo 3153d6b
cleanup
Szegoo 36f8390
cleanup & new test
Szegoo f006db2
renaming
Szegoo 35f65ee
add benchmark & docs
Szegoo 064464b
update schedule_code_upgrade benchmark
Szegoo d7f33ad
fmt
Szegoo 26e6779
fixes
Szegoo 8a0789a
force_set_parachain_billing_account_works
Szegoo 04339d9
Update polkadot/runtime/parachains/src/paras/mod.rs
Szegoo c380dfa
resolve comments
Szegoo 7d0709a
fix benchmark
Szegoo b6562a8
merge BillingInfo into ParaInfo
Szegoo 316b1e3
add migration
Szegoo 97974f8
Revert "add migration"
Szegoo a178142
require billing_account to be explicitly set
Szegoo 44a6cae
add test
Szegoo f9d3936
Merge branch 'master' into para-upgrade-fees
Szegoo e88b3f4
add events
Szegoo 1ac7da7
add docs
Szegoo 189c5da
add migration
Szegoo eb2af3d
better wording
Szegoo 700cd1e
add PreCodeUpgradeChecker type
Szegoo 282588e
implement PreCodeUpgrade | WIP
Szegoo 1da6ac8
execute pre_code_upgrade in inclusion
Szegoo 713ad0b
use proper return type | WIP
Szegoo 4082b7a
fixes
Szegoo 66a8e62
better docs & add missing weight
Szegoo 865063d
more clarity
Szegoo f1dd961
doc fixes
Szegoo c0f31fb
Update polkadot/runtime/common/src/paras_registrar/mod.rs
Szegoo f35bf62
Update polkadot/runtime/common/src/paras_registrar/migration.rs
Szegoo 2082dbf
don't short circuit
Szegoo d6082ab
set billing account to none upon registration
Szegoo 2f78321
update tests to explicitly set the billing account & small fix in pre…
Szegoo a04914b
benchmarks
Szegoo bca6464
improve readability
Szegoo 3373b18
use IsSystem
Szegoo 8fce70a
Merge branch 'master' into para-upgrade-fees
Szegoo d6f861a
improve readability
Szegoo 34ee5ce
less nested
Szegoo 090bccb
small fix
Szegoo 61d9b26
required_para_deposit helper function
Szegoo 88c9ef3
add pr doc
Szegoo 6119222
missing line
Szegoo 69aaebc
Merge branch 'master' into para-upgrade-fees
Szegoo 37b6617
fmt
Szegoo 6ac4eaa
Merge branch 'master' into para-upgrade-fees
Szegoo 40f303a
remove unnecessary generic
Szegoo 7015621
fix
Szegoo 0da3ee4
make clippy happy
Szegoo ba9f448
final fixes
Szegoo 3319410
..
Szegoo 55d7129
fix benchmark
Szegoo 490f8b9
fix zombienet tests
Szegoo da03660
.. fix
Szegoo 12cfb44
fix
Szegoo eb79964
use versi
Szegoo c79c4b0
remove upgrade fees from westend
Szegoo 99a95ec
fix
Szegoo ea0961f
🙈
Szegoo c6532e5
Merge branch 'master' into para-upgrade-fees
Szegoo 49b0142
fix benchmark
Szegoo 9aac6f7
Merge branch 'master' into para-upgrade-fees
Szegoo 527e7b3
Update polkadot/runtime/common/src/paras_registrar/mod.rs
Szegoo 5ad659b
Update polkadot/runtime/common/src/paras_registrar/migration.rs
Szegoo 4e842e6
Update polkadot/runtime/common/src/paras_registrar/migration.rs
Szegoo c16d4a9
Update polkadot/runtime/common/src/paras_registrar/mod.rs
Szegoo d382ee9
Update polkadot/runtime/common/src/paras_registrar/mod.rs
Szegoo 19ef131
Update polkadot/runtime/parachains/src/paras/mod.rs
Szegoo 47a4d81
Update polkadot/runtime/parachains/src/paras/mod.rs
Szegoo 91f8877
Update polkadot/runtime/common/src/paras_registrar/mod.rs
Szegoo f03377d
Update polkadot/runtime/common/src/paras_registrar/mod.rs
Szegoo 15d9a27
renaming & remove SovereignAccountOf
Szegoo d9c3335
Merge branch 'master' into para-upgrade-fees
Szegoo 6197aa1
bring back old migration
Szegoo 2e25232
signal_code_upgrade_failure
Szegoo cc3e0df
fix warning
Szegoo 78be2ae
make charging costs transactional
Szegoo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same :P
While I'm now asking myself why we have two of these tests.