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

Simple integration of Jaeger / Opentracing #32

Merged
merged 12 commits into from
Dec 3, 2020
Merged

Simple integration of Jaeger / Opentracing #32

merged 12 commits into from
Dec 3, 2020

Conversation

frouioui
Copy link
Owner

@frouioui frouioui commented Dec 3, 2020

This PR proposes a simple a integration of Jaeger in to tagenal. A new namespace observability was created and hosts the operator, the agent, and all the other required resources to run Jaeger.

Jaeger is run using the AllInOne configuration, as the purposes is solely for demo / experimentation.

Tracing is implemented throughout the whole application. From Traefik to Vitess.

Is traced:

  • Requests passing through Traefik
  • Frontend service
  • The implemented HTTP endpoints of both users and articles APIs
  • VTGate
  • VTTablets

The propagation from the APIs to Vitess was achieved by adding a DialOption (grpc.UnaryClientInterceptor) into the GRPCDialOptions of the vitessdriver's configuration.

A new service was created in the vitess keyspace, which aim to ease of development. We can now call vitess-zone1-vtgate-srv instead of calling the pseudo-random-generated service name by Vitess operator.

@frouioui frouioui added enhancement New feature or request observability Everything related to the observability labels Dec 3, 2020
@frouioui frouioui self-assigned this Dec 3, 2020
@frouioui
Copy link
Owner Author

frouioui commented Dec 3, 2020

Here is an example of a trace going from the web client, to Traefik, then to the Frontend, then to the Articles API, and finally reaching Vitess cluster (more specifically vtgate, and then the multiple vttablets, since we have 2 shards).
image

@frouioui frouioui merged commit f03d59e into master Dec 3, 2020
@frouioui frouioui deleted the jaeger branch December 20, 2020 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request observability Everything related to the observability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant