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.

Sign up for GitHub

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

Jump to bottom

Different fuel limit for Validator and other WASM executable #3354

Closed
Arjentix opened this issue Apr 4, 2023 · 1 comment
Closed

Different fuel limit for Validator and other WASM executable #3354

Arjentix opened this issue Apr 4, 2023 · 1 comment
Assignees
@Arjentix @timofeevmd
Labels
api-changes Changes in the API for client libraries Enhancement New feature or request iroha2-dev The re-implementation of a BFT hyperledger in RUST QA-confirmed This bug is reproduced and needs a fix

Comments

@Arjentix
Copy link
Contributor

Arjentix commented Apr 4, 2023

We should introduce different config parameters for fuel limit. Current fuel limit should be applied only for smartcontracts and triggers. Newly introduced fuel limit should be applied for Runtime Permission Validator.
@Arjentix Arjentix added Enhancement New feature or request iroha2-dev The re-implementation of a BFT hyperledger in RUST api-changes Changes in the API for client libraries labels Apr 4, 2023
@Arjentix Arjentix mentioned this issue Apr 4, 2023
Merged
5 tasks
@Arjentix
Copy link
Contributor Author

Arjentix commented Apr 4, 2023

Originally suggested by @mversic here

@Arjentix Arjentix self-assigned this Feb 19, 2024
Arjentix added a commit to Arjentix/iroha that referenced this issue Feb 19, 2024
@Arjentix
[feature] hyperledger#3354: Different fuel limit for Executor
c5d541c 
Signed-off-by: Daniil Polyakov <arjentix@gmail.com>
@Arjentix Arjentix mentioned this issue Feb 19, 2024
Merged
5 tasks
Arjentix added a commit to Arjentix/iroha that referenced this issue Feb 19, 2024
@Arjentix
[feature] hyperledger#3354: Different fuel limit for Executor
8aca356 
Signed-off-by: Daniil Polyakov <arjentix@gmail.com>
Arjentix added a commit to Arjentix/iroha that referenced this issue Feb 20, 2024
@Arjentix
[feature] hyperledger#3354: Different fuel limit for Executor
e776ed6 
Signed-off-by: Daniil Polyakov <arjentix@gmail.com>
Arjentix added a commit that referenced this issue Feb 20, 2024
@Arjentix
[feature] #3354: Different fuel limit for Executor
64e9e31 
Signed-off-by: Daniil Polyakov <arjentix@gmail.com>
@timofeevmd timofeevmd self-assigned this Feb 22, 2024
@timofeevmd timofeevmd added the QA-confirmed This bug is reproduced and needs a fix label Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Arjentix Arjentix

@timofeevmd timofeevmd

Labels
api-changes Changes in the API for client libraries Enhancement New feature or request iroha2-dev The re-implementation of a BFT hyperledger in RUST QA-confirmed This bug is reproduced and needs a fix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Arjentix @timofeevmd