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

Update Solana exporter to add more metrics #2

Merged
merged 51 commits into from
Jan 10, 2025
Merged

Update Solana exporter to add more metrics #2

merged 51 commits into from
Jan 10, 2025

Conversation

naviat
Copy link

@naviat naviat commented Jan 8, 2025

Update Solana exporter to add more metrics

johnstonematt and others added 30 commits October 24, 2024 12:09
@johnstonematt
standardised metric naming
0cc0f1d
@johnstonematt
added custom descriptor
fa33b1d
@johnstonematt
dried tests
a3261dd
@johnstonematt
replaced klog -> zap in rpc
1bbaa29
@johnstonematt
replaced klog -> zap in exporter
14eab97
@johnstonematt
added iso format for log ts
888d065
@johnstonematt
added configurable log levels
c9fabde
@johnstonematt
Merge pull request #58 from asymmetric-research/custom-desc
a7cbb1e 
Custom desc + test drying
@johnstonematt
added light mode
a2f7e82
@johnstonematt
made collector + watcher use config directly
e46d20d
@johnstonematt
counted block size by vote vs non-vote tx
6fa9afb
@johnstonematt
removed identity label
6406fb9
@johnstonematt
Merge pull request #57 from asymmetric-research/metric-naming
46cd96d 
Standardised metric naming
@johnstonematt
Merge pull request #60 from asymmetric-research/zap-logging
c5d167d 
Zap logging!
@johnstonematt
Merge pull request #61 from asymmetric-research/light-mode
cd3068f 
Light mode!
@johnstonematt
Merge pull request #62 from asymmetric-research/vote-count
a0d34e7 
Measure block size by vote tx and normal
@johnstonematt
Merge pull request #64 from asymmetric-research/remove-identity
cbdf506 
Remove identity label
@johnstonematt
removed solana_active_validators metric
c9f2bd5
@johnstonematt
added mock rpc server
43a02d7
@johnstonematt
made logger init automatically
36a11a9
@johnstonematt
added mock server + rpc tests
dda909a
@johnstonematt
big mock server test refactor
fa71eeb
@johnstonematt
added config tests
1582782
@johnstonematt
refactored WatchSlots_Dynamic test
d06edc0
@johnstonematt
added TestCountVoteTransactions
cb8422c
@johnstonematt
solved sorting issue in getVoteAccounts test
7d194b7
@johnstonematt
ran go mod tidy
03568cc
@johnstonematt
moved collectionTest to desc_test
8b8648c
@johnstonematt
updated README
86c912d
@johnstonematt
standardised on solana_node name for node-specific metrics
b846676
johnstonematt and others added 21 commits October 29, 2024 21:48
@johnstonematt
made light mode incompatible with other config options
b0844ad
@johnstonematt
updated logs
9542fc1
@johnstonematt @SEJeff
Update README.md - fix typo
a49f702 
Co-authored-by: Jeff Schroeder <[email protected]>
@johnstonematt
Merge pull request #68 from asymmetric-research/readme
9480812 
Readme update
@johnstonematt
Merge pull request #67 from asymmetric-research/remove-active-validators
898e0f7 
Removed solana_active_validators metric
@johnstonematt
standardised on counter names ending in _total
ef6941e
@johnstonematt
made active stake measure in SOL
9864422
@johnstonematt
reverted name back to solana_node_transactions_total
0babcea
@johnstonematt
renamed solana_exporter -> solana-exporter
95657c9
@johnstonematt
Merge pull request #65 from asymmetric-research/mock-server
7e33c00 
Big mock-server testing refactor...
@johnstonematt
minor readme update + jeffs nits
d2c2ff2
@johnstonematt
Merge pull request #69 from asymmetric-research/running-updates
49486bc 
WIP: Running updates
@johnstonematt
Merge pull request #70 from asymmetric-research/exporter-rename
42d5669 
renamed solana_exporter -> solana-exporter
@johnstonematt
fixed fee bug
f7cf3c2
@johnstonematt
removed usage code (for later)
0d3fa3d
@johnstonematt
added solana_cluster_slots_by_epoch_total
af6fa19
@johnstonematt
Merge pull request #72 from asymmetric-research/bug-fix
bb45ab3 
Running fixes/updates
@johnstonematt
registered solana_cluster_slots_by_epoch_total
0d59821
@johnstonematt
Merge pull request #73 from asymmetric-research/bug-fix
cd52340 
registered solana_cluster_slots_by_epoch_total
@johnstonematt
added example prometheus setup
cbf2954
@johnstonematt
Merge pull request #74 from asymmetric-research/example-prometheus
c689d36 
added example prometheus setup
@guardrails GuardRails
Copy link

guardrails bot commented Jan 8, 2025

⚠️ We detected 1 security issue in this pull request:

Insecure Network Communication (1)
Severity Details Docs
High Title: Plaintext HTTP Server

solana-exporter/cmd/solana-exporter/main.go

Line 39 in c689d36

logger.Fatal(http.ListenAndServe(config.ListenAddress, nil))
📚

More info on how to fix Insecure Network Communication in Go.

👉 Go to the dashboard for detailed results.

📥 Happy? Share your feedback with us.

@GranderStark GranderStark merged commit 9b31756 into chainstack:master Jan 10, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@naviat @GranderStark @johnstonematt