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

Falcon Example #133

Open
konfortes opened this issue Nov 8, 2022 · 3 comments
Open

Falcon Example #133

konfortes opened this issue Nov 8, 2022 · 3 comments

Comments

@konfortes
Copy link

Hey guys, any particular reason the falcon example uses manual instrumentation in the /hello endpoint?
It should be automatically instrumented. Am I missing something?

@pellared
Copy link
Contributor

pellared commented Nov 8, 2022

Do you mean this:

with tracer.start_as_current_span('internal_span', kind=SpanKind.INTERNAL) as span:
?

If so, then this is just an example of creating a manual span on top of automatic instrumentation.

@konfortes
Copy link
Author

Yes.
I find it confusing since this is a repo with examples of instrumenting Python frameworks. manual instrumentation has its place but probably somewhere else.
Having the internal span go along all the span we get from the auto instrumentation also attributes to my confusion.

P.S
I came across this example after I wasn't able to automatically instrument my Falcon app and it made me wonder whether I should manually instrument Falcon apps

@konfortes
Copy link
Author

I found the issue.
My poc wasn't working because I used the code from the example with falcon 3.1 and opentelemetry-instrumentation-falcon 0.33b0.
opentelemetry-instrumentation-falcon is only able to instrument when using the App class (instead of the API class which is deprecated in version 3 of falcon).

Opened a PR to update the example
I also removed the manual tracing since it is confusing and not compatible with the other examples.

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