Skip to content

Fix ownershp of cmd/. - #55913

Open
aiuto wants to merge 2 commits into
mainfrom
aiuto/rem
Open

Fix ownershp of cmd/.#55913
aiuto wants to merge 2 commits into
mainfrom
aiuto/rem

Conversation

@aiuto

@aiuto aiuto commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

What does this PR do?

Changes ownership of /cmd/ from fleet-remediation to agent-build.

Motivation

It clearly should not be owned by fleet-remediation, and they have complained about the spam.

@aiuto aiuto added changelog/no-changelog No changelog entry needed qa/no-code-change No code change in Agent code requiring validation labels Sep 5, 2026
@dd-octo-sts dd-octo-sts Bot added the internal Identify a non-fork PR label Sep 5, 2026
@github-actions github-actions Bot added the short review PR is simple enough to be reviewed quickly label Sep 5, 2026
@aiuto

aiuto commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator Author

@chouquette IIRC, the complaint about ownership came up in your iot change.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AI review by Codex (OpenAI) - workflow run

The patch is correct. It changes the default owner of /cmd/ to @DataDog/agent-build while preserving subsequent, more specific ownership rules. No actionable issues found.

@datadog-prod-us1-6

This comment has been minimized.

@dd-octo-sts

dd-octo-sts Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Files inventory check summary

File checks results against ancestor 02d2f08e:

Results for datadog-agent_7.85.0~devel.git.28.e096238.pipeline.135684726-1_amd64.deb:

No change detected

Results for datadog-iot-agent_7.85.0~devel.git.28.e096238.pipeline.135684726-1_amd64.deb:

No change detected

@aiuto aiuto added the ask-review Ask in slack required teams to review this PR label Sep 5, 2026
@aiuto
aiuto requested review from a team and matt-dz September 5, 2026 17:52
@aiuto aiuto changed the title correct ownershp of /cmd/ Fix ownershp of cmd/. Sep 5, 2026
@aiuto

aiuto commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator Author

The broken test is broken at head on main. Don't worry about it.

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

Labels

ask-review Ask in slack required teams to review this PR changelog/no-changelog No changelog entry needed internal Identify a non-fork PR qa/no-code-change No code change in Agent code requiring validation short review PR is simple enough to be reviewed quickly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant