OTel: add trace_flags to JSON log records so unsampled trace ids are recognizable

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#7,729 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

@ShaanGS y travaille déjà.

Depuis le 4/11/2026.

  • #7734 par @ShaanGS — ouverte

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
82/100
Type d'issue
Fonctionnalité
Clarté
Clairement spécifiée
Activité
Active
Stack technique
python
Domaine
observability

Piste de recherche

Commence par lire otel.trace_fields() et les tests test_trace_fields_inside_span et test_json_mode_adds_trace_ids ; l’issue mentionne également docs/api-reference/observability.md. Vérifie comment le contexte du span expose les indicateurs de trace, puis exécute les tests concernés. Le travail est terminé lorsque les enregistrements de logs JSON incluent l’indicateur W3C en minuscules sur deux chiffres, qu’un test ALWAYS_OFF attend "00" et que la documentation Logs explique cette valeur.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

enhancement

Follow-up to #7668 (fixes #6903).

Problem

#7668 adds trace_id and span_id to JSON log records written inside an OpenTelemetry span. It does not add trace_flags.

A span that is not sampled still has a valid span context. So its log records get ids, but the tracing backend never stores that trace. A reader then searches for a trace that does not exist, and the log line gives no sign of why.

This case is normal in Reflex. The event span uses a parent-based sampler by default, so a client traceparent with the sampled flag off gives an unsampled span. A ratio sampler (OTEL_TRACES_SAMPLER=traceidratio) does the same for most spans.

Repro

from opentelemetry.sdk.trace import TracerProvider
from opentelemetry.sdk.trace.sampling import ALWAYS_OFF
from reflex_base import otel

otel.enable(tracer_provider=TracerProvider(sampler=ALWAYS_OFF))
with otel._tracer.start_as_current_span("x") as span:
    print(span.is_recording())  # False
    print(otel.trace_fields())  # {'trace_id': '...', 'span_id': '...'}, no sampled flag

Proposed change

Add trace_flags to otel.trace_fields(), as the OpenTelemetry spec "Trace Context in Non-OTLP Log Formats" recommends. Use the W3C format: two lowercase hex digits, "01" when sampled, "00" when not.

"trace_flags": f"{span_context.trace_flags:02x}",
  • Extend test_trace_fields_inside_span and test_json_mode_adds_trace_ids to check the field.
  • Add a test with an ALWAYS_OFF sampler that expects "trace_flags": "00".
  • Add the field to the "Logs" section of docs/api-reference/observability.md, with one sentence on what "00" means.
Langage dominant
Python
Étoiles
29k
Forks
1.8k
Merge moyen
2 j 22 h
PR mergées (30 j)
143

Préparer son environnement

Ouvrir dans Codespaces

Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de reflex-dev/reflex

Toutes les issues de reflex-dev/reflex

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.