Skip to content

[beyondtrust_isi] Update changelog, version in manifest and Sync CODEOWNERS#19157

Merged
ShourieG merged 2 commits into
elastic:feature/beyondtrust__isi-0.1.0from
janvi-elastic:fix_changelog_beyondtrust_isi
May 22, 2026
Merged

[beyondtrust_isi] Update changelog, version in manifest and Sync CODEOWNERS#19157
ShourieG merged 2 commits into
elastic:feature/beyondtrust__isi-0.1.0from
janvi-elastic:fix_changelog_beyondtrust_isi

Conversation

@janvi-elastic
Copy link
Copy Markdown
Contributor

Proposed commit message

Update changelog, version in manifest for BeyondTrust ISI and sync CODEOWNERS.

@janvi-elastic janvi-elastic requested a review from a team as a code owner May 22, 2026 08:49
@github-actions
Copy link
Copy Markdown
Contributor

TL;DR

Buildkite failed in Check go sources because packages/elastic_agent/manifest.yml (at the failing commit) sets owner.github: elastic/elastic-agent, but .github/CODEOWNERS does not include @elastic/elastic-agent anywhere. Align the manifest owner with an existing CODEOWNERS handle (or add that handle to CODEOWNERS).

Remediation

  • Update packages/elastic_agent/manifest.yml owner to a CODEOWNERS-listed team (for example elastic/elastic-agent-control-plane, which is already listed for /packages/elastic_agent).
  • Re-run .buildkite/scripts/check_sources.sh (or CI) to confirm github.com/elastic/integrations/dev/codeowners.Check passes.
Investigation details

Root Cause

dev/codeowners.Check validates that every package manifest owner exists in .github/CODEOWNERS.

At failing commit a5a507964d7115da27251bb6aa8fb3f5f3b06e54:

  • packages/elastic_agent/manifest.yml:14github: elastic/elastic-agent
  • .github/CODEOWNERS:226/packages/elastic_agent @elastic/elastic-agent-data-plane @elastic/elastic-agent-control-plane``

So the manifest owner does not match any CODEOWNERS entry.

Evidence

Error: error validating packages in directory 'packages': error checking manifest 'packages/elastic_agent': owner "elastic/elastic-agent" defined in "packages/elastic_agent/manifest.yml" is not in ".github/CODEOWNERS"

Verification

  • Not run locally in this workflow; diagnosis is based on Buildkite logs plus repository content at the failing commit SHA.

Follow-up

If this was intentionally moved to a new owner, add @elastic/elastic-agent to the /packages/elastic_agent CODEOWNERS line instead.

Note

🔒 Integrity filter blocked 3 items

The following items were blocked because they don't meet the GitHub integrity level.

To allow these resources, lower min-integrity in your GitHub frontmatter:

tools:
  github:
    min-integrity: approved  # merged | approved | unapproved | none

What is this? | From workflow: PR Buildkite Detective

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

@ShourieG
Copy link
Copy Markdown
Contributor

@janvi-elastic, why are we downgrading the version ?

@ShourieG ShourieG merged commit 2ec38a3 into elastic:feature/beyondtrust__isi-0.1.0 May 22, 2026
8 checks passed
@andrewkroh andrewkroh added the Crest Contributions from Crest developement team. label May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Crest Contributions from Crest developement team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants