Skip to content

Conversation

@bukata-sa
Copy link
Collaborator

@bukata-sa bukata-sa commented Dec 29, 2025

Initializing beholder in ccv repo just for standalone executor/verifier
Inside the node beholder client will be available, don't override it by default

@bukata-sa bukata-sa force-pushed the ccip-8881/fix-beholder-init branch from 42c1c37 to 51b26c4 Compare December 29, 2025 15:20
@bukata-sa bukata-sa marked this pull request as ready for review December 29, 2025 17:54
@bukata-sa bukata-sa requested review from a team and skudasov as code owners December 29, 2025 17:54
@bukata-sa bukata-sa changed the title CCIP 8881: fix beholder init CCIP 8881 fix beholder init Dec 29, 2025
Comment on lines -41 to -42
// Initialize pyroscope for continuous profiling
if _, err := pyroscope.Start(pyroscope.Config{
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not against removing it entirely, but if we foresee us potentially using this in the future, maybe we can only start it based off of a flag or something?

@bukata-sa bukata-sa added this pull request to the merge queue Dec 30, 2025
Merged via the queue into main with commit 93cf92e Dec 30, 2025
14 checks passed
@bukata-sa bukata-sa deleted the ccip-8881/fix-beholder-init branch December 30, 2025 10:05
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