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

Document tips around clock skew #47

Open
codefromthecrypt opened this issue May 26, 2016 · 7 comments
Open

Document tips around clock skew #47

codefromthecrypt opened this issue May 26, 2016 · 7 comments

Comments

@codefromthecrypt
Copy link
Member

Q: how do you sync the times on the various services? Do the clients connect to the zipkin server and work out the difference between their clock and the servers?
Or are you using some other way of stitching the requests together

A: clock sync is not attacked by Zipkin, there are existing solutions for it (see NTP). But, especially in a virtualized / cloud environment, there will always be clock drift; Zipkin does try to address that when displaying data with various heuristics (like, if this service called that service, then this span must come before that span, and not after, even if the timestamps say differently)

@DanielYWoo
Copy link

Even with NTP, it could be 300 ms clock difference between services, but an RPC could only take 30 ms, so the timestamp can only be used to calculate duration, we should never compare timestamps across services. Am I right?

@dineshkapoor27
Copy link

Should we have this feature where there is one time which every service follows? Or should the time stamp be one the basis of the entry time of the log in a central log repository (maybe ELK)?

@codefromthecrypt
Copy link
Member Author

codefromthecrypt commented Mar 6, 2018 via email

@jcchavezs
Copy link

There is definitively clock-skew logic in UI (isn't @tacigar). Maybe we need to add a rationale document around it next to that code? cc @adriancole

@jcchavezs
Copy link

Shall this issue be moved to zipkin/zipkin-support? cc @jorgheymans

@jorgheymans
Copy link

Sure, that or the new discussions feature ?

@codefromthecrypt codefromthecrypt transferred this issue from openzipkin/openzipkin.github.io Dec 17, 2020
@codefromthecrypt
Copy link
Member Author

can do both if you want.. discussions feature in this repo!

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

No branches or pull requests

5 participants