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

HH-199910 additional information about invocation for otlp spans #664

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

Sparkymann
Copy link
Contributor

@Sparkymann Sparkymann commented Nov 28, 2023

https://jira.hh.ru/browse/HH-199910

версия - патч
Описание:
В телеметрийные спаны добавлены новые атрибуты: code.function и code.namespace.
code.function - наименование вызываемого метода
code.namespace - полный путь к классу, метод которого вызывается

if key not in excluded_span_attributes:
return set_attribute(self, key, value)

SpanImpl.set_attribute = patched_set_attribute
Copy link
Contributor Author

Choose a reason for hiding this comment

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

span api не предоставляет возможность удаления атрибутов, которые уже засеттили https://github.com/open-telemetry/opentelemetry-python/blob/1625b35ab55782f1781dd189f5742efab5fdbf8a/opentelemetry-api/src/opentelemetry/trace/span.py#L57

решил, что лучше пропатчить метод и вообще не проставлять сомнительные атрибуты, чем потом их удалять через приватные поля:

    tornado_handler = 'tornado.handler'
    attributes = span._attributes
    if tornado_handler in attributes:
        del attributes[tornado_handler]

патчинг судя по этому не возбраняется

Copy link
Contributor

Choose a reason for hiding this comment

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

мне не нравится манкипатчинг
и там я надеюсь доберусь и переделаю через асинио дебаг как-нибудь

но тут как будто нет вариков нормальных, не форкать же

@Sparkymann Sparkymann marked this pull request as ready for review November 28, 2023 14:20
@Sparkymann Sparkymann requested a review from a team as a code owner November 28, 2023 14:20
Copy link

hh-sonar bot commented Nov 29, 2023

SonarQube Quality Gate

Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@hhrelease hhrelease merged commit 5ec0411 into master Nov 29, 2023
2 checks passed
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

Successfully merging this pull request may close these issues.

3 participants