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

Support Backend acts as pure Zipkin collector #2424

Merged
merged 10 commits into from
Mar 30, 2019
Merged

Conversation

wu-sheng
Copy link
Member

@peng-yongsheng @adriancole

This PR brings a new feature in SkyWalking. In old zipkin-skywalking adaptor, I can't make it 100% production ready(even 90% is hard). But in these months, I work around SkyWalking in service mesh, a pure Zipkin native format collector takes my attention.

In SkyWalking/Istio/Envoy solution, we could work in piece very well. And we miss tracing. SkyWalking's agents could work(that is for sure), and at the same time, Zipkin ecosystem could work too, especially, could be used in the production env and sampled based, because it would affect the metric, mesh observability provides 100% traffic/topology metrics.

So, I bring this PR here. Hope you will like this. Used to talk about this in #2044 (6.3.0 feature), I just speed this up.

@wu-sheng wu-sheng added backend OAP backend related. feature New feature labels Mar 29, 2019
@wu-sheng wu-sheng added this to the 6.1.0 milestone Mar 29, 2019
## Zipkin receiver
Zipkin receiver could work in two different mode.
1. Tracing mode(default). Tracing mode is that, skywalking OAP acts like zipkin collector, which provide persistence and query,

Choose a reason for hiding this comment

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

maybe mention the features of the query? I assume this is skywalking queries, not zipkin's https://zipkin.io/zipkin-api/

Copy link
Member Author

Choose a reason for hiding this comment

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

Please review the new description again.

  1. Tracing mode(default). Tracing mode is that, skywalking OAP acts like zipkin collector, fully supports Zipkin v1/v2 formats through HTTP service, also provide persistence and query in skywalking UI. But it wouldn't analysis metric from them. In most case, I suggest you could use this feature, when metrics come from service mesh. Notice, in this mode, Zipkin receiver requires zipkin-elasticsearch storage implementation active. Read this to know
    how to active.

Choose a reason for hiding this comment

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

I understand now. thanks!

@coveralls
Copy link

coveralls commented Mar 29, 2019

Coverage Status

Coverage decreased (-0.02%) to 13.897% when pulling c039284 on zipkin-trace into 1fd5c57 on master.

@peng-yongsheng peng-yongsheng merged commit 601b472 into master Mar 30, 2019
@wu-sheng wu-sheng deleted the zipkin-trace branch March 30, 2019 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend OAP backend related. feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants