Skip to content

Fix powerpack dependency ID remapping - #682

Open
riyazsh wants to merge 1 commit into
mainfrom
riyazsh/powerpacks-remap-group-widget-refs
Open

Fix powerpack dependency ID remapping#682
riyazsh wants to merge 1 commit into
mainfrom
riyazsh/powerpacks-remap-group-widget-refs

Conversation

@riyazsh

@riyazsh riyazsh commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remap monitor and SLO IDs under the actual powerpack group_widget payload shape
  • keep existing top-level widget paths for compatibility
  • add unit coverage for powerpack ID remapping and top-level identity/author stripping

Why

Powerpacks wrap dashboard widgets under attributes.group_widget, but sync-cli only had dashboard-style widgets.* dependency paths. That means monitor/SLO references inside powerpacks could remain as source-org IDs in the destination copy.

Testing

  • tox -e py -- --black datadog_sync/model/powerpacks.py tests/unit/test_powerpacks.py tests/unit/test_import_force_missing_deps.py::test_extract_source_ids_overrides_complete

@riyazsh
riyazsh requested a review from a team as a code owner August 28, 2026 20:58
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