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

Standardize on internal tag/attribute naming as @jaeger@something #6522

Open
yurishkuro opened this issue Jan 11, 2025 · 2 comments · May be fixed by #6524
Open

Standardize on internal tag/attribute naming as @jaeger@something #6522

yurishkuro opened this issue Jan 11, 2025 · 2 comments · May be fixed by #6524
Labels
good first issue Good for beginners help wanted Features that maintainers are willing to accept but do not have cycles to implement

Comments

@yurishkuro
Copy link
Member

Currently we use inconsistent naming for internal attributes

I propose we use a shorter but more distinct naming scheme that does not match the normal dot-separated format of OTEL attributes. The above three keys would become

  • @jaeger@warnings
  • @jaeger@format ("span" is implied since it's on the span)
  • @jaeger@hash ("span" is implied since it's on the span)
@yurishkuro yurishkuro added good first issue Good for beginners help wanted Features that maintainers are willing to accept but do not have cycles to implement labels Jan 11, 2025
@ADI-ROXX ADI-ROXX linked a pull request Jan 11, 2025 that will close this issue
3 tasks
@ADI-ROXX
Copy link

ADI-ROXX commented Jan 11, 2025

jaeger.internal.warnings in internal/jptrace/warning.go
internal.span.format in cmd/collector/app/span_processor.go

@yurishkuro Apart from the above naming conventions and span.hash, which will be pushed later on, is there any other naming convention that needs to be changed to "@jaeger@something" format?

@yurishkuro
Copy link
Member Author

I don't recall if we have any other internal tags.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for beginners help wanted Features that maintainers are willing to accept but do not have cycles to implement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants