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

[refactor] #3901: Add method to define parameter in executor migrate entrypoint #4513

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dima74
Copy link
Contributor

@dima74 dima74 commented Apr 25, 2024

Description

  • Added method add_parameter which can be called from executor (in migrate entrypoint) to define new executor parameter
  • Added test which demonstrates how executor can use it, as suggested here
  • Note that this PR was mostly implemented before discussion in dynamic parameter value #4352, so potentially this should be implemented differently. E.g. if it will be decided to store executor parameters in Domain::metadata, then there will be no need for add_parameter method since executor parameter will be defined using SetKeyValue ISI
  • Note that I didn't removed NewParameter ISI since it is currently used to define chain-wide parameters (for wasm engine and sumeragi), and potentially chain-wide parameters should be defined/stored differently as discussed in dynamic parameter value #4352 (comment)

Linked issue

Closes #3901
Related: #4352
Related: #4310

Benefits

Checklist

  • I've read CONTRIBUTING.md
  • I've used the standard signed-off commit format (or will squash just before merging)
  • All applicable CI checks pass (or I promised to make them pass later)
  • (optional) I've written unit tests for the code changes
  • I replied to all comments after code review, marking all implemented changes with thumbs up

…or `migrate` entrypoint

Signed-off-by: Dmitry Murzin <diralik@yandex.ru>
@github-actions github-actions bot added Refactor Improvement to overall code quality iroha2-dev The re-implementation of a BFT hyperledger in RUST labels Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
iroha2-dev The re-implementation of a BFT hyperledger in RUST Refactor Improvement to overall code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[suggestion] Move NewParameter into Executor
1 participant