Skip to content

[Security Solution] Add security-engineering-productivity ownership for Security Solution Scout-related code #245361

Merged
MadameSheema merged 5 commits intoelastic:mainfrom
MadameSheema:scout-ownership
Dec 5, 2025
Merged

[Security Solution] Add security-engineering-productivity ownership for Security Solution Scout-related code #245361
MadameSheema merged 5 commits intoelastic:mainfrom
MadameSheema:scout-ownership

Conversation

@MadameSheema
Copy link
Copy Markdown
Contributor

Summary

Adds @elastic/security-engineering-productivity as a code owner for Security Solution Scout-related code to improve onboarding and ensure best practices during the initial adoption phase.

Changes

  • Added @elastic/security-engineering-productivity as a code owner for x-pack/solutions/security/packages/kbn-scout-security (alongside existing @elastic/appex-qa ownership)
  • Added code ownership for /x-pack/solutions/security/plugins/security_solution/test/scout directory

Motivation

These ownership updates will:

  • Improve onboarding by ensuring the Security Engineering Productivity team is notified of Scout-related changes
  • Help maintain best practices through consistent review and guidance during the initial adoption period
  • Provide teams with support and mentorship as they learn the new tool and testing patterns

Note on Test Ownership

The ownership of the Scout test directory is intended to be temporary. Once all teams have a solid understanding of best practices and are comfortable with the new tool, we plan to transfer ownership back to the respective feature teams. This temporary ownership structure ensures consistent quality and knowledge transfer during the transition period.

@MadameSheema MadameSheema self-assigned this Dec 5, 2025
@MadameSheema MadameSheema added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v9.3.0 labels Dec 5, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

@MadameSheema MadameSheema enabled auto-merge (squash) December 5, 2025 11:04
@MadameSheema MadameSheema disabled auto-merge December 5, 2025 11:13
@MadameSheema MadameSheema requested a review from a team as a code owner December 5, 2025 11:19
Copy link
Copy Markdown
Member

@pheyos pheyos left a comment

Choose a reason for hiding this comment

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

LGTM

@MadameSheema MadameSheema enabled auto-merge (squash) December 5, 2025 13:13
@MadameSheema MadameSheema merged commit 30711f1 into elastic:main Dec 5, 2025
12 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

cc @MadameSheema

wildemat pushed a commit to wildemat/kibana that referenced this pull request Dec 5, 2025
…or Security Solution Scout-related code (elastic#245361)

## Summary

Adds `@elastic/security-engineering-productivity` as a code owner for
Security Solution Scout-related code to improve onboarding and ensure
best practices during the initial adoption phase.

## Changes

- Added `@elastic/security-engineering-productivity` as a code owner for
`x-pack/solutions/security/packages/kbn-scout-security` (alongside
existing `@elastic/appex-qa` ownership)
- Added code ownership for
`/x-pack/solutions/security/plugins/security_solution/test/scout`
directory

## Motivation

These ownership updates will:
- Improve onboarding by ensuring the Security Engineering Productivity
team is notified of Scout-related changes
- Help maintain best practices through consistent review and guidance
during the initial adoption period
- Provide teams with support and mentorship as they learn the new tool
and testing patterns

## Note on Test Ownership

The ownership of the Scout test directory is intended to be
**temporary**. Once all teams have a solid understanding of best
practices and are comfortable with the new tool, we plan to transfer
ownership back to the respective feature teams. This temporary ownership
structure ensures consistent quality and knowledge transfer during the
transition period.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
JordanSh pushed a commit to JordanSh/kibana that referenced this pull request Dec 9, 2025
…or Security Solution Scout-related code (elastic#245361)

## Summary

Adds `@elastic/security-engineering-productivity` as a code owner for
Security Solution Scout-related code to improve onboarding and ensure
best practices during the initial adoption phase.

## Changes

- Added `@elastic/security-engineering-productivity` as a code owner for
`x-pack/solutions/security/packages/kbn-scout-security` (alongside
existing `@elastic/appex-qa` ownership)
- Added code ownership for
`/x-pack/solutions/security/plugins/security_solution/test/scout`
directory

## Motivation

These ownership updates will:
- Improve onboarding by ensuring the Security Engineering Productivity
team is notified of Scout-related changes
- Help maintain best practices through consistent review and guidance
during the initial adoption period
- Provide teams with support and mentorship as they learn the new tool
and testing patterns

## Note on Test Ownership

The ownership of the Scout test directory is intended to be
**temporary**. Once all teams have a solid understanding of best
practices and are comfortable with the new tool, we plan to transfer
ownership back to the respective feature teams. This temporary ownership
structure ensures consistent quality and knowledge transfer during the
transition period.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants