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

Could not jaeger containerd-shim-wasm #787

Open
giteewif opened this issue Jan 1, 2025 · 6 comments
Open

Could not jaeger containerd-shim-wasm #787

giteewif opened this issue Jan 1, 2025 · 6 comments

Comments

@giteewif
Copy link

giteewif commented Jan 1, 2025

I follow the readme.md ,and setup-otel.sh in scripts, and ctr run wasi-demo-app, but could only get the containerd tracing, there is no containerd-shim-wasm tracing info?

@Mossaka
Copy link
Member

Mossaka commented Jan 2, 2025

Have you followed this document about enabling OpenTelmetry in the shim? https://github.com/containerd/runwasi/blob/main/docs/opentelemetry.md

@giteewif
Copy link
Author

giteewif commented Jan 3, 2025

Have you followed this document about enabling OpenTelmetry in the shim? https://github.com/containerd/runwasi/blob/main/docs/opentelemetry.md

Yes I have followed the document, i could get a part of the tracing info, but not the shim_main. It seems like that the tracing info consists of that after the create_task_service. There is no tracing func like shim_main or start_shim.

The picture show what i see in jaeger, it contains some tracing info in the shim
Image

I see that there is a func "shim_main_with_otel" in the opentTelmetry.md, but in the latest code, I could not find it. And shim_main also setup the otel.
Image

@Mossaka
Copy link
Member

Mossaka commented Jan 9, 2025

yea we renamed shim_main_with_otel to shim_main(). I will update the doc to reflect this change, and thanks for pointing it out!

Yes I have followed the document, i could get a part of the tracing info, but not the shim_main.

I will look into this issue but we should be able to see shim_main and start_shim

@giteewif
Copy link
Author

giteewif commented Jan 9, 2025

yea we renamed shim_main_with_otel to shim_main(). I will update the doc to reflect this change, and thanks for pointing it out!

Yes I have followed the document, i could get a part of the tracing info, but not the shim_main.

I will look into this issue but we should be able to see shim_main and start_shim

Thanks, and there is also the validate or exec tracing info in executor not see.

@Mossaka
Copy link
Member

Mossaka commented Jan 9, 2025

There is a bug in the main branch. I was able to trace to a particular commit f38990d4da42e4de6f3c363998dafec253c33a4f

Before that commit I was able to see the full tracing info. e.x.

Image

@giteewif
Copy link
Author

giteewif commented Jan 9, 2025

Yes,I do use the latest in the main branch.

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

2 participants