elastic / elastic/integrations
[docs-patrol] Update CI pipeline docs for nested package paths
- 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
Assessment
This issue has not been assessed yet.