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

OTLP/JSON: Clarify behavior when traceid/spanid fields are not present or not valid #3040

Closed
tigrannajaryan opened this issue Dec 15, 2022 · 0 comments · Fixed by open-telemetry/opentelemetry-proto#442
Assignees
Labels
spec:protocol Related to the specification/protocol directory

Comments

@tigrannajaryan
Copy link
Member

The discussion here revealed that we do not have clear enough definition of what happens the traceid/spanid fields are not present or not valid in the JSON payload.

This needs to be clarified before we can declare OTLP/JSON Stable.

@tigrannajaryan tigrannajaryan added the spec:protocol Related to the specification/protocol directory label Dec 15, 2022
tigrannajaryan added a commit to tigrannajaryan/opentelemetry-proto that referenced this issue Jan 3, 2023
…ields

Resolves open-telemetry/opentelemetry-specification#3040

This is not a breaking change:

- For Span it now defines more precisely the receiver behavior that was
  previously defined vaguely (e.g. it was unclear what "empty" means for
  bytes field).

- For LogRecord it now defines the receiver behavior that was previously
  unspecified. This ensures that the wording are consistent with what we
  have for the Span.
tigrannajaryan added a commit to tigrannajaryan/opentelemetry-proto that referenced this issue Jan 20, 2023
…ields

Resolves open-telemetry/opentelemetry-specification#3040

This is not a breaking change:

- For Span it now defines more precisely the receiver behavior that was
  previously defined vaguely (e.g. it was unclear what "empty" means for
  bytes field).

- For LogRecord it now defines the receiver behavior that was previously
  unspecified. This ensures that the wording are consistent with what we
  have for the Span.
tigrannajaryan added a commit to tigrannajaryan/opentelemetry-proto that referenced this issue Jan 20, 2023
…ields

Resolves open-telemetry/opentelemetry-specification#3040

This is not a breaking change:

- For Span it now defines more precisely the receiver behavior that was
  previously defined vaguely (e.g. it was unclear what "empty" means for
  bytes field).

- For LogRecord it now defines the receiver behavior that was previously
  unspecified. This ensures that the wording are consistent with what we
  have for the Span.
tigrannajaryan added a commit to tigrannajaryan/opentelemetry-proto that referenced this issue Jan 20, 2023
…ields

Resolves open-telemetry/opentelemetry-specification#3040

This is not a breaking change:

- For Span it now defines more precisely the receiver behavior that was
  previously defined vaguely (e.g. it was unclear what "empty" means for
  bytes field).

- For LogRecord it now defines the receiver behavior that was previously
  unspecified. This ensures that the wording are consistent with what we
  have for the Span.
tigrannajaryan added a commit to open-telemetry/opentelemetry-proto that referenced this issue Jan 27, 2023
…ields (#442)

Resolves open-telemetry/opentelemetry-specification#3040

This is not a breaking change:

- For Span it now defines more precisely the receiver behavior that was
  previously defined vaguely (e.g. it was unclear what "empty" means for
  bytes field).

- For LogRecord it now defines the receiver behavior that was previously
  unspecified. This ensures that the wording are consistent with what we
  have for the Span.
VinozzZ pushed a commit to VinozzZ/opentelemetry-proto that referenced this issue Jun 21, 2024
…ields (open-telemetry#442)

Resolves open-telemetry/opentelemetry-specification#3040

This is not a breaking change:

- For Span it now defines more precisely the receiver behavior that was
  previously defined vaguely (e.g. it was unclear what "empty" means for
  bytes field).

- For LogRecord it now defines the receiver behavior that was previously
  unspecified. This ensures that the wording are consistent with what we
  have for the Span.
VinozzZ pushed a commit to VinozzZ/opentelemetry-proto that referenced this issue Jun 21, 2024
…ields (open-telemetry#442)

Resolves open-telemetry/opentelemetry-specification#3040

This is not a breaking change:

- For Span it now defines more precisely the receiver behavior that was
  previously defined vaguely (e.g. it was unclear what "empty" means for
  bytes field).

- For LogRecord it now defines the receiver behavior that was previously
  unspecified. This ensures that the wording are consistent with what we
  have for the Span.
VinozzZ pushed a commit to VinozzZ/opentelemetry-proto that referenced this issue Jun 21, 2024
…ields (open-telemetry#442)

Resolves open-telemetry/opentelemetry-specification#3040

This is not a breaking change:

- For Span it now defines more precisely the receiver behavior that was
  previously defined vaguely (e.g. it was unclear what "empty" means for
  bytes field).

- For LogRecord it now defines the receiver behavior that was previously
  unspecified. This ensures that the wording are consistent with what we
  have for the Span.
VinozzZ pushed a commit to VinozzZ/opentelemetry-proto that referenced this issue Jun 21, 2024
…ields (open-telemetry#442)

Resolves open-telemetry/opentelemetry-specification#3040

This is not a breaking change:

- For Span it now defines more precisely the receiver behavior that was
  previously defined vaguely (e.g. it was unclear what "empty" means for
  bytes field).

- For LogRecord it now defines the receiver behavior that was previously
  unspecified. This ensures that the wording are consistent with what we
  have for the Span.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec:protocol Related to the specification/protocol directory
Projects
None yet
2 participants