-
Notifications
You must be signed in to change notification settings - Fork 115
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
Luxor/vested wg spending #5072
Luxor/vested wg spending #5072
Conversation
To be rebased onto #5062 |
… metadatat" This reverts commit 4041f80.
7390606
to
0a147e4
Compare
I think there is a simpler way to do this, without needing to have an account on each working group. |
|
I think this now needs to be rebased on luxor since I merged the reduce council budget PR |
tests/network-tests/src/fixtures/workingGroups/SpendBudgetFixture.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some additional cleanup for build to works after refactor (removing the moduleId in working groups)
0cf5f79
to
ac4b0b1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Benchmark is failing
Error: Input("Benchmark working_group::vested_spend_from_budget failed: AmountLow")
2024-03-29 22:54:25 Starting benchmark: working_group::vested_spend_from_budget
There was a problem generating the weights for working_group, check the error above
Error: Process completed with exit code 1.
None | ||
); | ||
|
||
let amount_vesting = VestingBalanceOf::<T>::from(10_000u32); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Benchmark is failing with error hereAmountLow
because amount is less than vesting pallet configuration of <T as vesting::Config>::MinVestedTransfer::get()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So perhaps use that constant to make the amount_vesting
at least that amount.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
from earlier
None | ||
); | ||
|
||
let amount_vesting = VestingBalanceOf::<T>::from(10_000u32); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So perhaps use that constant to make the amount_vesting
at least that amount.
|
||
let amount_vesting = VestingBalanceOf::<T>::from(10_000u32); | ||
let budget = BalanceOf::<T>::from(100_000u32); | ||
let block_no = 100u32; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
block_no
is not used
strange that I'm seeing errors like I think you forgot to commit changes to: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, next PR to luxor should commit changes to types/src/*.rs missed in this PR
Running integration tests locally on my machine works, we need to investigate why they are failing on github runners.
Addresses: #4948
based on top of #5062
Note: I have used the extrinsic
vested_transfer
inside the extrinsicspend_from_budget
all extrinsic in substrate are nowtransactional
by default, this means that if thevested_transfer
fails thenspend_from_budget
is equivalent to a no-op.Also I have created one internal account for each WG where tokens are minted before they are transferred with vesting.