Skip to content

Emit typed bootstrap skip outcomes - #671

Open
riyazsh wants to merge 2 commits into
mainfrom
codex/paxos-bootstrap-failure-signals
Open

Emit typed bootstrap skip outcomes#671
riyazsh wants to merge 2 commits into
mainfrom
codex/paxos-bootstrap-failure-signals

Conversation

@riyazsh

@riyazsh riyazsh commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add typed SkipResource metadata and optional NDJSON details for actionable skip outcomes
  • emit destination_metric_missing for metrics_metadata, metric_percentiles, and metric_tag_configurations with metric_name and operation details
  • emit integration_pipeline_bootstrap_required for logs_pipelines bootstrap gaps with pipeline_name and ddsource details

Context

Paxos DDR needs sync-cli to distinguish terminal/no-op skips from skips that require managed-sync bootstrap. PR #669 covers metric tag configuration behavior, but metrics_metadata and related metric resources still serialized missing-destination metrics as failure_class=unknown, leaving managed-sync without a clean signal.

This keeps the existing HTTP failure taxonomy intact and only adds failure_class/details when a resource model deliberately raises a typed SkipResource.

Testing

  • .tox/py312/bin/pytest -q tests/unit/test_failure_class.py tests/unit/test_sync_report.py tests/unit/test_metrics_metadata.py tests/unit/test_metric_percentiles.py tests/unit/test_metric_tag_configurations.py tests/unit/test_logs_pipelines.py
  • .tox/py312/bin/black --line-length 120 --check datadog_sync/utils/resource_utils.py datadog_sync/utils/sync_report.py datadog_sync/utils/resources_handler.py datadog_sync/model/metrics_metadata.py datadog_sync/model/metric_percentiles.py datadog_sync/model/metric_tag_configurations.py datadog_sync/model/logs_pipelines.py tests/unit/test_failure_class.py tests/unit/test_metrics_metadata.py tests/unit/test_metric_percentiles.py tests/unit/test_metric_tag_configurations.py tests/unit/test_logs_pipelines.py
  • git diff --check

@riyazsh
riyazsh requested a review from a team as a code owner August 25, 2026 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant