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

feat(indexer-service): Add graph-node version endpoint from status api #841

Merged
merged 2 commits into from
Jan 10, 2024

Conversation

abarmat
Copy link
Collaborator

@abarmat abarmat commented Jan 10, 2024

The graph-node is exposing the version information using the following struct, however, it's not exposed through the indexer-service status endpoint.

This PR adds that information so that the gateway can have access to it.

struct Version {
    version: String,
    commit: String,
}

@abarmat abarmat requested a review from fordN January 10, 2024 14:14
Copy link
Member

@Theodus Theodus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally this would be surfaced via /version. But it's more important that this gets to indexers ASAP so that we can start enforcing a graph-node feature set.

Copy link
Contributor

@fordN fordN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Theodus Theodus merged commit 7039d55 into main Jan 10, 2024
9 checks passed
@Theodus Theodus deleted the ab/version-in-status branch January 10, 2024 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🚗 Merged
Development

Successfully merging this pull request may close these issues.

3 participants