Skip to content

Add agent-build devx as co-owner of /.gitlab/build/bazel/ - #55607

Open
aiuto wants to merge 1 commit into
mainfrom
aiuto/own_config
Open

Add agent-build devx as co-owner of /.gitlab/build/bazel/#55607
aiuto wants to merge 1 commit into
mainfrom
aiuto/own_config

Conversation

@aiuto

@aiuto aiuto commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

What does this PR do?

Add agent-build devx as co-owner of /.gitlab/build/bazel/

Motivation

A lot of the specification there is about infrastructure provisioning. The responsibility should be shared between the teams.

@aiuto aiuto added changelog/no-changelog No changelog entry needed qa/no-code-change No code change in Agent code requiring validation labels Aug 28, 2026
@aiuto
aiuto requested review from chouetz and quentinus95 August 28, 2026 17:46
@dd-octo-sts dd-octo-sts Bot added the internal Identify a non-fork PR label Aug 28, 2026
@github-actions github-actions Bot added the short review PR is simple enough to be reviewed quickly label Aug 28, 2026

@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

Patch is incorrect: the new CODEOWNERS entry neither uses valid team syntax nor adds the co-owner stated in the PR description.

Comment thread .github/CODEOWNERS
/.gitlab/.pre/* @DataDog/agent-devx
/.gitlab/.post/* @DataDog/agent-devx
/.gitlab/build/bazel/* @DataDog/agent-build
/.gitlab/build/bazel/* @DataDog/agent-build DataDog/agent-delivery

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.

This owner is missing the required @, and it names agent-delivery even though the stated change is to add agent-devx. As written, GitHub will not recognize the intended co-owner. This should presumably be @DataDog/agent-devx.

@dd-octo-sts

dd-octo-sts Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Files inventory check summary

File checks results against ancestor 241e0cc4:

Results for datadog-agent_7.84.0~devel.git.562.8cedf74.pipeline.134002455-1_amd64.deb:

No change detected

Results for datadog-iot-agent_7.84.0~devel.git.562.8cedf74.pipeline.134002455-1_amd64.deb:

No change detected

@chouetz chouetz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think you should approve my suggestion 😄
Thanks!

Comment thread .github/CODEOWNERS
/.gitlab/.pre/* @DataDog/agent-devx
/.gitlab/.post/* @DataDog/agent-devx
/.gitlab/build/bazel/* @DataDog/agent-build
/.gitlab/build/bazel/* @DataDog/agent-build DataDog/agent-delivery

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
/.gitlab/build/bazel/* @DataDog/agent-build DataDog/agent-delivery
/.gitlab/build/bazel/* @DataDog/agent-build @DataDog/agent-devx

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

Labels

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.

2 participants