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.

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

[Access] Add API that return node information such as version, commit, spork Id and protocol version. #1332

Conversation

Guitarheroua
Copy link
Contributor

onflow/flow-go#2134

Description

I have added a new request/response for the node version information API query in GRPC protobuf. Additionally, I have extended the OpenAPI model for REST with a new endpoint for node version information.

@Guitarheroua Guitarheroua changed the title Guitarheroua/2134 Add API that return node information such as version, commit, spork Id and protocol version. [Access] Add API that return node information such as version, commit, spork Id and protocol version. Apr 18, 2023
@Guitarheroua Guitarheroua marked this pull request as ready for review April 18, 2023 11:17
@peterargue
Copy link
Contributor

@Guitarheroua can you address those conflicts? they're just from merging your other PR

Guitarheroua and others added 3 commits April 25, 2023 01:58
…protocol' of github.com:Guitarheroua/flow into guitarheroua/2134-add-api-to-return-current-version-of-protocol
@bthaile bthaile merged commit ff3b4e5 into onflow:master Apr 26, 2023
bors bot added a commit to onflow/flow-go that referenced this pull request Apr 28, 2023
4235: [Access] Add API that return node information such as version, commit, spork Id and protocol version. r=peterargue a=Guitarheroua

#2134

### Context

This pull request adds a new API query `GetNodeVersionInfo` for RPC/REST that returns essential node information, including version number, commit hash, spork ID, and protocol version.

As part of this PR:

* The protobuf/openapi structures have been extended (see onflow/flow#1332)
* Tests for the new API have been added for RPC/REST.

NOTE:
The Cadence version is still under discussion. See #2134 (comment)

Co-authored-by: Andriy Slisarchuk <andriyslisarchuk@gmail.com>
Co-authored-by: Andrii Slisarchuk <andriyslisarchuk@gmail.com>
Co-authored-by: Andrii Slisarchuk <Guitarheroua@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants