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
[Merged by Bors] - API phase 9: finish implementing grpc services #2071
Conversation
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.
see comments and fix lint
06c9db3
to
da4c6eb
Compare
97aa29f
to
22ec8b0
Compare
Pass layersPerEpoch into the MeshService
@lrettig - looks like you are getting travis status via bors now again.... |
bors try |
bors try- |
bors try |
tryBuild succeeded: |
bors try |
tryBuild failed: |
bors try |
tryBuild failed: |
bors try |
tryBuild succeeded: |
bors merge |
## Motivation See #1764 Also Fixes #2116 Requires #2092, merge that first ## Changes - Adds stubs for all remaining services and endpoints - Upgrades to latest API build, renames a few API data items to be consistent with improved naming in the API code - Allows MeshService to read and return layer root hash: makes TransactionProcessor GetLayerStateRoot method public - Adds a config param (internal, only for testing) to force the syncer to always enable listening to gossip - Event Reporter no longer reports any details about Status updates, it just "pings" the listener which is responsible for re-fetching all status items they care about - Adds account, reward, receipt channels to Reporter; allows Reporter to block, or to increase channel buffer sizes, on initialization - Allows Reporter report methods to block (useful for tests) - Implements all endpoints that can be implemented (some, e.g. those involving transaction receipt, cannot yet be implemented) - Adds a method `GetSmesherID` to ATX to allow the nodeID that created it to be read, as part of finding ATXs by smesher for the API ## Test Plan - Adds tests for all new endpoints and services
Pull request successfully merged into develop. Build succeeded: |
Motivation
See #1764
Also Fixes #2116
Requires #2092, merge that first
Changes
GetSmesherID
to ATX to allow the nodeID that created it to be read, as part of finding ATXs by smesher for the APITest Plan