Skip to content

Add the option to build a declaration report and exit immediately#5989

Merged
nccatoni merged 3 commits intomainfrom
nccatoni/declartion-dump
Jan 12, 2026
Merged

Add the option to build a declaration report and exit immediately#5989
nccatoni merged 3 commits intomainfrom
nccatoni/declartion-dump

Conversation

@nccatoni
Copy link
Copy Markdown
Collaborator

@nccatoni nccatoni commented Jan 9, 2026

Motivation

When making modification to the manifest processing or making bulk edits to the manifests/decorators, it is very hard to test the outcome. This adds the option to dump the declarations and exit immediately making it possible to make assertion on the end to end activation/deactivation process without having to actually run the setup and test functions.

Changes

Workflow

  1. ⚠️ Create your PR as draft ⚠️
  2. Work on you PR until the CI passes
  3. Mark it as ready for review
    • Test logic is modified? -> Get a review from RFC owner.
    • Framework is modified, or non obvious usage of it -> get a review from R&P team

🚀 Once your PR is reviewed and the CI green, you can merge it!

🛟 #apm-shared-testing 🛟

Reviewer checklist

  • Anything but tests/ or manifests/ is modified ? I have the approval from R&P team
  • A docker base image is modified?
    • the relevant build-XXX-image label is present
  • A scenario is added, removed or renamed?

@nccatoni nccatoni requested a review from cbeauchesne January 9, 2026 15:13
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 9, 2026

CODEOWNERS have been resolved as:

conftest.py                                                             @DataDog/system-tests-core

@nccatoni nccatoni marked this pull request as ready for review January 12, 2026 08:14
@nccatoni nccatoni requested a review from a team as a code owner January 12, 2026 08:14
@nccatoni nccatoni enabled auto-merge (squash) January 12, 2026 08:14
@nccatoni nccatoni merged commit 4b4f40f into main Jan 12, 2026
2669 of 2671 checks passed
@nccatoni nccatoni deleted the nccatoni/declartion-dump branch January 12, 2026 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants