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

upgrade substrate dep to > v14 metadata #85

Merged
merged 9 commits into from
Dec 6, 2021

Conversation

echevrier
Copy link
Contributor

@echevrier echevrier commented Nov 8, 2021

Fix #83

This branch must not be merged to master!!! until integritee-network/worker#484 is done

@echevrier echevrier force-pushed the 83/upgrade_substrate_dep_v14 branch from 3eab5b7 to 1fd5a43 Compare November 9, 2021 09:39
@echevrier echevrier requested review from haerdib and clangenb November 9, 2021 11:33
Copy link
Collaborator

@haerdib haerdib left a comment

Choose a reason for hiding this comment

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

LGTM, as long as we don't merge yet :)

@@ -52,15 +51,15 @@ impl<T: frame_system::Config> pallet_teerex::WeightInfo for WeightInfo<T> {
// Storage: Teerex EnclaveIndex (r:1 w:0)
// Storage: Teerex WorkerForShard (r:0 w:1)
// Storage: Teerex LatestIpfsHash (r:0 w:1)
fn confirm_call() -> Weight {
fn confirm_processed_parentchainblock() -> Weight {
Copy link
Collaborator

Choose a reason for hiding this comment

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

That one is already outdated again. But I'll have to do an upgrade to pallet_teerex anyway to finish integritee-network/worker#457, so leave it for now. We just need to make sure we don't downgrade once we're ready to merge this one.

@echevrier echevrier marked this pull request as draft November 9, 2021 12:02
Copy link
Collaborator

@clangenb clangenb 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, but I guess, we will have to wait a bit more until merging....

@echevrier echevrier force-pushed the 83/upgrade_substrate_dep_v14 branch 2 times, most recently from ae2833b to 86da514 Compare November 24, 2021 11:57
@brenzi brenzi merged commit a97ddbc into master Dec 6, 2021
@echevrier echevrier deleted the 83/upgrade_substrate_dep_v14 branch December 6, 2021 13:50
echevrier added a commit that referenced this pull request Dec 8, 2021
* upgrade to substrate to 7406442bea0194ffcafc4e8d48d895d4d8d11346
* for v14 metadata
* upgrade to subwasm_version 0.14.1
* Update weights function name for pallet teerex according to PR#14
* Add weight for the teeracle

Co-authored-by: echevrier <[email protected]>
Co-authored-by: Bigna Härdi <[email protected]>
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.

fix teeracle weights upgrade substrate dep to > v14 metadata
4 participants