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.

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

Proof of concept index metrics tracker. #153

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

tpendragon
Copy link
Contributor

@tpendragon tpendragon commented Oct 21, 2024

This is nowhere near complete, but it's a nice proof of concept for time to poll as defined in #115

The :monotonic_time stuff is pulled from Telemetry's recommendations: https://hexdocs.pm/telemetry/readme.html

Copy link

github-actions bot commented Oct 21, 2024

Container Scanning Status: ✅ Success


ghcr.io/pulibrary/dpul-collections:pr-153 (debian 12.6)
=======================================================
Total: 0 (HIGH: 0, CRITICAL: 0)

@hackartisan
Copy link
Member

This is looking good. Do you think we shouldn't use a span, then?

@hackartisan
Copy link
Member

@tpendragon
Copy link
Contributor Author

Like using https://hexdocs.pm/opentelemetry_api/OpenTelemetry.Tracer.html#start_span/2

I looked at that, but I couldn't figure out how to wrap something around a process that happens over a long period of time. Like I guess I could spawn a task that spins and waits...but I think just sending an event for the duration is just as good.

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