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.

Sign up for GitHub

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

Jump to bottom

Road to v1: service unification, indexing dashboard and other improvements #92

Merged
merged 34 commits into from
Feb 5, 2024
Merged

Road to v1: service unification, indexing dashboard and other improvements #92

merged 34 commits into from
Feb 5, 2024

Conversation

neysofu
Copy link
Member

@neysofu neysofu commented Jan 29, 2024 •
edited
Loading

  • Switch from warp to axum.
  • Greatly simplify the code architecture and split the application into multiple crates.
  • Merge cross-checker service and API server.
  • Add support for graph-node version monitoring, subgraph version.
  • Indexers are now identified by address, not name, resulting in simpler logic in quite a few places.
  • Much improved error handling and monitoring.
  • Indexer dashboard endpoints and Diesel queries.

Follow-ups (WIP) will include a GH release with a published GraphQL API schema file and guide on how to setup and maintain Grafana dashboards; will then re-prioritize feature work.

@neysofu
Adjust log levels
6cbf1f5 
Signed-off-by: Filippo Costa <[email protected]>
@neysofu
Adjust log levels
ad362d5 
Signed-off-by: Filippo Costa <[email protected]>
@neysofu
Add support for graph-node version
791ed74
@neysofu
Use containerized store tests
b1c9b21
@neysofu
Add previously removed test
a39d757
@neysofu
Rename NetworkSubgraphClient
85daa56
@neysofu
Rename network subgraph directory, remove dead code
cd3a1ac
@neysofu
Add network subgraph client test
aa9622c
@neysofu
Fix query variable
9780a4b
@neysofu
Much more effective network subgraph queries
62313bd
@neysofu
Add indexer and network info on Grafana
8836c58
@neysofu
Removed dead file
a7329e6
@neysofu
More clearly separate indexer name and ID
560deac
@neysofu
Remove unused struct IndexerRow
3af1ea0
@neysofu
Add IndexerId trait, indexer_network_subgraph_entry
9ce2cd3
@neysofu
Simplify API of graphql_api module
f33ebe8
@neysofu
Update to Rust 1.75
3f58d9d
@neysofu
Delete frontend code
68dbb87
@neysofu
Remove useless .clone()
aa3c518
@neysofu
Fix all Clippy warnings
18f2374
@neysofu
Update async-graphql to v7.0.1
7b5ada3
@neysofu
Switch api-server from Warp to Axum
1193ccf
@neysofu
Remove unused .gitkeep
5eeaa2c
@neysofu
Remove prelude modue; explicit imports
b9e5980
@neysofu
cargo +nightly fmt --all
58ef385 
Signed-off-by: Filippo Costa <[email protected]>
@neysofu
Make indexer addresses non-optional
a01f631
@neysofu
wip refactor
928212a 
Signed-off-by: Filippo Costa <[email protected]>
@neysofu
get it to compile
aae0fa6
@neysofu
cargo +nightly fmt --all
7bf84df 
Signed-off-by: Filippo Costa <[email protected]>
@neysofu
Reduce name cloning
caecc9f
@neysofu neysofu requested a review from LNSD January 29, 2024 16:30
@neysofu
Copy link
Member Author

neysofu commented Jan 29, 2024

Partial fix: #91
Closes #76.
Closes #48.
Closes #34.

@fordN fordN self-assigned this Jan 29, 2024
LNSD
LNSD approved these changes Jan 30, 2024
View reviewed changes
Copy link
Contributor

@LNSD LNSD left a comment

Choose a reason for hiding this comment

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

LGTM ✅

@neysofu neysofu merged commit 186b62f into main Feb 5, 2024
5 checks passed
@neysofu neysofu deleted the filippo/improvements branch February 5, 2024 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LNSD LNSD LNSD approved these changes

Assignees

@fordN fordN

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@neysofu @LNSD @fordN