-
Notifications
You must be signed in to change notification settings - Fork 15
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
Thoughts on replacing mocks with Telemetry events #50
Comments
Small PoC: #47 (comment) |
flame_on and eflambe are separate project. flame_on used to use eflambe, but no longer does. Can you explain what the benefit to stopping and starting via telemetry would be? Why not just use |
I realize that. I thought they could both benefit from this idea so I opened this issue in both.
It allows removing mocking from production :) |
👋
Originally posted in DockYard/flame_on#48
Similar to the issue above, what if
eflambe
starts and stops tracing ontelemetry
events?Since
telemetry:execute
andtelemetry:span
run in the caller process, we have access to the current pid, and we can use the new trace module with very minimal effort for flamegraphs. Flamecharts (which is what FlameOn is doing, and I guesseflambe
as well?) would require extra effort. Withtrace
we might also be able to cross process boundary and start tracing the processes that the current process "calls".The API could be just a telemetry event for which to start tracing, and an optional event for when to stop tracing.
Context:
The text was updated successfully, but these errors were encountered: