elastic / elastic/integrations

[docs-patrol] Update CI pipeline docs for nested package paths

Open
#18,760 2 comments 0 reactions 0 assignees View on GitHub
Team:Ecosystem
Dominant language
Handlebars
Stars
333
Forks
647
Avg merge
3d 4h
Merged PRs (30d)
209

Description

Recent code changes in the repository have introduced documentation drift. The following change needs a corresponding documentation update.

## Changes Requiring Documentation Updates

### 1. CI package-selection behavior now supports nested package directories, but docs still describe flat paths

**Commit(s):**
- https://github.com/elastic/integrations/commit/55c9cf43ab3903fb1d94b89daaa4c8213e50cd07
- Related rollout: https://github.com/elastic/integrations/commit/f8e677d6cb76452294b73803ca5997ed8c76d7b4

**What changed:**
- `.buildkite/scripts/common.sh` now resolves package directories via `mage ... listPackages` (`list_all_directories`, lines 883-885 in current file) and checks changed files against full `package_path` (`grep -E "^\$\{package_path}/"`, line 801).
- `.buildkite/scripts/trigger_integrations_in_parallel.sh` iterates `package_path` values from `list_all_directories` and passes full paths through test selection/execution (lines 53-57).
- `dev/citools/packages.go` `ListPackages` walks directories recursively and returns valid package paths, enabling nested package directories (lines 19-45).

**Documentation impact:**
- `docs/ci_pipelines.md` still states changed packages are `packages/*` (line 69), which implies flat one-level package directories.
- This should be updated to describe path-based/nested package selection (for example `packages/**` / full package paths), so contributors understand how CI decides impacted packages.

## Suggested Actions

- [ ] Update `docs/ci_pipelines.md` package-selection text to reflect nested package directory support (path-based matching instead of only `packages/*`).
- [ ] Add a concrete example with a nested package path to clarify CI behavior.

> [!NOTE]
>
> 🔒 Integrity filter blocked 3 items
>
> The following items were blocked because they don't meet the GitHub integrity level.
>
> - [#18165](https://github.com/elastic/integrations/issues/18165) `search_issues`: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
> - [#18576](https://github.com/elastic/integrations/issues/18576) `search_issues`: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
> - [#18186](https://github.com/elastic/integrations/issues/18186) `search_issues`: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
>
> To allow these resources, lower `min-integrity` in your GitHub frontmatter:
>
> ```yaml
> tools:
> github:
> min-integrity: approved # merged | approved | unapproved | none
> ```
>
>

---
[What is this?](https://ela.st/github-ai-tools) | [From workflow: Docs Patrol](https://github.com/elastic/integrations/actions/runs/25211136000)

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.
> - [x] expires on May 8, 2026, 10:37 AM UTC

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.