Skip to content

live_validate files "Cannot mix time-series aggregate" as other, so --fail-on-bug misses it #460

Description

@shmsr

Summary

verifier.live_validate classifies Elasticsearch's Cannot mix time-series aggregate … and regular aggregate … rejection as other. --fail-on-bug only exits non-zero on real_bug, so a translator bug of this class stays green.

Found while running the live-cluster Datadog/Grafana pass in #454. The mixed-STATS emitter itself is a separate Grafana concern (_normalize_mixed_ts_stats_exprs in promql.py); this issue is the gate that fails to count the rejection.

Where

parity-rig/verifier/live_validate.py

  • classify_error (other is the fallback after _DATA_GAP and _REAL_BUG)
  • _REAL_BUG does not match this verification exception
  • main returns non-zero only when --fail-on-bug and summary["real_bugs"]

The module docstring defines other as "anything else (5xx, transport, timeouts)". A query Elasticsearch rejects because the emitted STATS mixes aggregate kinds is a translator bug, the same class as the parsing/type errors already in _REAL_BUG.

Expected

That error text classifies as real_bug, is printed with the other real bugs, and fails a --fail-on-bug run.

Parent

Sub-issue of https://github.com/elastic/obs-integration-team/issues/1203.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingprogram:trustVerification, reports, parity, audits

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions