-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Add Loop-compatible Head Reporter for Solana (#17580) #17627
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
base: release/2.23.1-ccip
Are you sure you want to change the base?
Conversation
* Add Solana Head Reporter * Add explanation to //golint:nosec * Address PR comments Renames: - NewEVMTelemetry -> NewLegacyEVMTelemetry - NewPrometheusReporter -> NewLegacyEVMPrometheusReporter - evmTelemetryReporter -> legacyEVMTelemetryReporter - NewSolanaTelemetryReporter -> NewTelemetryReporter - solanaTelemetryReporter -> loopTelemetryReporter Now, NewTelemetryReporter handles all relayers in the same way (including EVM). The legacy EVM Telemetry Reporter is left operational so we still receive metrics about finalized heads. This can be removed once we extend the relayer interface to be able to fetch finalized heads from there. * Remove unused error output from GetIDToRelayerMap()
I see you updated files related to
|
|
Note: fuzz test failures are fixed by these 14 commits, but doesn't seem worth pulling all of that into release branch to get a non-required test to pass and making partial changes manually instead will cause future merge conflicts.
|
cherry-pick of #17580 , so it's ready in case we need a hot fix for 2.23.1-ccip release before 2.24.0 is ready