mirror of
https://github.com/dashpay/dash.git
synced 2024-12-26 12:32:48 +01:00
07b5a451f0
<!-- *** Please remove the following help text before submitting: *** Provide a general summary of your changes in the Title above Pull requests without a rationale and clear improvement may be closed immediately. Please provide clear motivation for your patch and explain how it improves Dash Core user experience or Dash Core developer experience significantly: * Any test improvements or new tests that improve coverage are always welcome. * All other changes should have accompanying unit tests (see `src/test/`) or functional tests (see `test/`). Contributors should note which tests cover modified code. If no tests exist for a region of modified code, new tests should accompany the change. * Bug fixes are most welcome when they come with steps to reproduce or an explanation of the potential issue as well as reasoning for the way the bug was fixed. * Features are welcome, but might be rejected due to design or scope issues. If a feature is based on a lot of dependencies, contributors should first consider building the system outside of Dash Core, if possible. --> ## Issue being fixed or feature implemented <!--- Why is this change required? What problem does it solve? --> <!--- If it fixes an open issue, please link to the issue here. --> ## What was done? <!--- Describe your changes in detail --> Added the fields `numValidMembers` and `healthRatio` in `quorum listextended` RPC reply, as we need a quick way to see the health of all quorums with a single command. `healthRatio` range is `[0.0 - 1.0]` Note: The decision to include both fields was taken because we need cover the case where a quorum was created with `minSize` members but all of them were valid. ## How Has This Been Tested? <!--- Please describe in detail how you tested your changes. --> <!--- Include details of your testing environment, and the tests you ran to --> <!--- see how your change affects other areas of the code, etc. --> ## Breaking Changes <!--- Please describe any breaking changes your code introduces --> ## Checklist: <!--- Go over all the following points, and put an `x` in all the boxes that apply. --> - [x] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have added or updated relevant unit/integration/functional/e2e tests - [x] I have made corresponding changes to the documentation **For repository code-owners and collaborators only** - [x] I have assigned this pull request to a milestone
12 lines
650 B
Markdown
12 lines
650 B
Markdown
New RPCs
|
|
--------
|
|
|
|
- `quorum listextended` is the cousin of `quorum list` with more enriched reply. By using `height` parameter RPC will list active quorums at specified height (or at tip if `height` is not specified.)
|
|
This RPC returns the following data per quorum grouped per llmqTypes:
|
|
- For each `quorumHash`:
|
|
- `creationHeight`: Block height where its DKG started
|
|
- `quorumIndex`: Returned only for rotated llmqTypes
|
|
- `minedBlockHash`: Hash of the block containing the mined final commitment
|
|
- `numValidMembers`: The total of valid members.
|
|
- `healthRatio`: The ratio of healthy members to quorum size. Range [0.0 - 1.0].
|