Skip to content

Fix powerpack dependency ID remapping - #682

Merged
riyazsh merged 1 commit into
mainfrom
riyazsh/powerpacks-remap-group-widget-refs
Aug 31, 2026
Merged

Fix powerpack dependency ID remapping#682
riyazsh merged 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
@riyazsh
riyazsh merged commit 5f39ee9 into main Aug 31, 2026
12 of 13 checks passed
@riyazsh
riyazsh deleted the riyazsh/powerpacks-remap-group-widget-refs branch August 31, 2026 14:48
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.

2 participants