Add details of Jaeger binary architecture

jkowall commented 23 hours ago

Which problem is this PR solving?

Part of #777

Description of the changes

Added new diagram Provided details of all components, ref:

How was this change tested?

local make develop

Checklist

[X] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[X] I have signed all commits

netlify[bot] commented 23 hours ago

Deploy Preview for romantic-neumann-1959d7 ready!

Name	Link
Latest commit	c79b24dc982206011e3070cc46357c0bd3f653ff
Latest deploy log	https://app.netlify.com/sites/romantic-neumann-1959d7/deploys/673cffb876843e0008326700
Deploy Preview	https://deploy-preview-783--romantic-neumann-1959d7.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

jkowall commented 19 hours ago

It doesn't fully address the ticket, as I would like the diagram to be more detailed w.r.t. the core extensions

Can you provide what you mean by "core extensions" that you'd like to see?

yurishkuro commented 18 hours ago

Can you provide what you mean by "core extensions" that you'd like to see?

jaeger_storage, jaeger_query and jaeger_storage_exporter are what I consider core components, since without them there is no Jaeger-v2.

jkowall commented 17 hours ago

All of these should be addressed with the latest push (two).

yurishkuro commented 17 hours ago

linter complains of missleppings

/home/runner/work/documentation/documentation/content/docs/next-release-v2/architecture.md:79:6 - Unknown word (Recievers) fix: (Receivers)
/home/runner/work/documentation/documentation/content/docs/next-release-v2/architecture.md:80:103 - Unknown word (Recieve) fix: (Receive)
/home/runner/work/documentation/documentation/content/docs/next-release-v2/architecture.md:82:115 - Unknown word (Recieve) fix: (Receive)
/home/runner/work/documentation/documentation/content/docs/next-release-v2/architecture.md:84:113 - Unknown word (Recieve) fix: (Receive)
/home/runner/work/documentation/documentation/content/docs/next-release-v2/architecture.md:86:115 - Unknown word (Recieve) fix: (Receive)
/home/runner/work/documentation/documentation/content/docs/next-release-v2/architecture.md:98:126 - Unknown word (OLTP)
/home/runner/work/documentation/documentation/content/docs/next-release-v2/architecture.md:100:1[18](https://github.com/jaegertracing/documentation/actions/runs/11921650283/job/33226240441?pr=783#step:4:19) - Unknown word (OLTP)

jkowall commented 17 hours ago

Should be fixed now, sorry for the missed spelling correction.

jaegertracing / documentation