Skip to content

refactor: scheduler/observer telemetry#3157

Open
ruslandoga wants to merge 9 commits intoLogflare:mainfrom
ruslandoga:rd/scheduler-observer-telemetry
Open

refactor: scheduler/observer telemetry#3157
ruslandoga wants to merge 9 commits intoLogflare:mainfrom
ruslandoga:rd/scheduler-observer-telemetry

Conversation

@ruslandoga
Copy link
Contributor

ANL-1268

Removed log-based scheduler/observer metrics and converted to more poll-based telemetry.

schedulers_online: :erlang.system_info(:schedulers_online),
schedulers_available: :erlang.system_info(:schedulers_online),
otp_release: :erlang.system_info(:otp_release),
version: :erlang.system_info(:version),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

version and otp_release are probably not measurements, but metadata.

Copy link
Contributor

Choose a reason for hiding this comment

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

can remove

prev_sample = :scheduler.get_sample_all()
Process.sleep(duration)
next_sample = :scheduler.get_sample_all()
:erlang.system_flag(:scheduler_wall_time, false)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This implementation is different from poller.ex, instead of keeping prev sample around it samples over one second each time. I think it might produce a more accurate result.

@ruslandoga ruslandoga marked this pull request as ready for review February 12, 2026 18:08
@ruslandoga
Copy link
Contributor Author

@Ziinc 👋

This PR should be ready for your review now!

Copy link
Contributor

@Ziinc Ziinc left a comment

Choose a reason for hiding this comment

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

some compilation errors

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.

2 participants