Roa feature #4
Merged
Roa feature #4
Conversation
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
Implmented sysio.roas native action 'reducepolicy' and all pieces
necessary for that.
Replaced userres and got roa usage tracking added
Replaced userres in chain_plugin to reference sysio.roa's reslimit table.
Adjusted the flow of resource usage tracking:
RAM isn't charged during the action play through instead incrementing our new total_ram_usage
In finalize we do our check to determine who to charge contract or caller where we use
total_ram_usage to charge the appropriate person.
Cleaner overall flow and maintains Leap resource management architecture.
Also added a check, to not allow nodeowners to remove the sysio or sysio.* policies that are
established on registration as these are system requirements. They can expand them but not
lower them.
Modified newaccount so only sysio can call it.
sysio is charged for resources used for account creation.
all accounts are created with no resources.
Removed delegatebw and buyram from account creation.
Fixed finalize() fallback logic
Fixed an issue in finalize() logic when it checks contracts resources.
Logic was incorrect for measuring accounts with unlimited CPU / NET.
Fixed fallback edge case
There was an edge case where contract == user and so it would SYS_ASSERT
instead since that is to be expected sometimes. We attempt to charge the contract
and if it doesn't have enough we reject without trying the user.
modified the "free resources" check to accomodate bios boot.
toggles after block 200.
Fixed chain plugin get_account
Previous edits didn't fully adjust to account for our sysio.roa resource management changes
It now gets and populates properly. Resolved unlimited CPU / NET on clio get account
dtaghavi
force-pushed
the
roa-feature
branch
from
799b5b5 to
fb20499
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.