Skip to content

Conversation

@clangenb
Copy link
Contributor

@clangenb clangenb commented Jun 3, 2020

the enclave does now perform a state update on every shard upon block import with the keys given by the newly introduced STF::storage_hashes_to_update_on_block

As of now the enclave creates a new shard if it does not exist already.

Every shard located in the shards directory is updated.

@clangenb clangenb force-pushed the cl-on-block-storage-updates-for-every-shard branch from b2396ab to e8b85d4 Compare June 3, 2020 09:07
@clangenb clangenb requested a review from brenzi June 3, 2020 09:11
Err(_) => error!("Error executing relevant extrinsics"),
};

if update_states(signed_block.block.header, node_url).is_err() {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@brenzi As the runtime does perform state updates at the end of block execution I suppose it does make sense that we also update the state after we executed the relevant extrinsics?

@clangenb clangenb removed the request for review from brenzi June 3, 2020 09:39
@clangenb clangenb merged commit 8e36c63 into master Jun 3, 2020
@clangenb clangenb mentioned this pull request Jun 3, 2020
brenzi pushed a commit to encointer/encointer-worker that referenced this pull request Sep 8, 2020
* [enclave] extracted verify worker response.

* [STF/enclave] whip update storages upon block import

* [enclave] perform state updates on all shards upon block import

* [enclave] Create shard if it does not exist. Currently, the workers follow an auto-join policy for new shards.

* clippy

* [enclave] do not return an error if could not execute relevant extrinsic in stf. Only log the error.
@clangenb clangenb deleted the cl-on-block-storage-updates-for-every-shard branch May 31, 2021 14:46
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.

2 participants