python-django

python-django copied to clipboard

Django 4.1 all tests fail with 'OpenTracingMiddleware' object has no attribute '_is_coroutine'

1

```py [ 37s] ERROR: test_middleware_untraced_no_trace_all (test_site.test_middleware.TestDjangoOpenTracingMiddleware) [ 37s] ---------------------------------------------------------------------- [ 37s] Traceback (most recent call last): [ 37s] File "/usr/lib/python3.8/site-packages/django/test/utils.py", line 460, in inner [ 37s] return func(*args, **kwargs) [...

jayvdb

if i want to get rpc in django, should i must use @tracing.trace() def some_view_func(request): new_request = some_http_request current_span = tracing.get_span(request) text_carrier = {} opentracing_tracer.inject(span, opentracing.Format.TEXT_MAP, text_carrier) for k, v...

waitqaq

Error: object has no attribute 'META'

3

Hi! I got the following error `'GetExampleView' object has no attribute 'META'` when i implement `@tracing.trace()` inside a class that inherits from any kind of APiView related model. ``` class...

julidr

Using the `another_decorator()`, allows to crate a new span for custom functions or methods that are not views.

crosvera

Django-REST integration?

5

Would this be able to pick up DRF interactions, like Serializer -> ORM -> DB ?

ror6ax

which path or databse the tracing data storage ? in a log file or in database ? thanks~

minicloudsky

Fixes https://github.com/opentracing-contrib/python-django/issues/64

jayvdb

Without `tests/test_site`, the inclusion of `tests/manage.py` doesnt make any sense.

jayvdb

https://github.com/opentracing-contrib/python-django/blob/160871ddfb00329de6a9296bb59990eb9e8da09f/django_opentracing/tracing.py#L59 As seen in the tracing.py line 59 when handling a traced view using decorator while tracing_all is activated, the view is called without args nor kwargs which will break...

piratos

The current implementation of the `DjangoTracing` class forces the use of `view_func.__name__` as the operation name for the span. I have a use case with class based views where the...

AgDude