elastic / elastic/integrations

[docs-patrol] Update CI docs for schedule changes and skip-CI file matching

Open
#18,934 1 comment 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 changes need corresponding documentation updates.

## Changes Requiring Documentation Updates

### 1. Scheduled CI behavior changed, but `docs/ci_pipelines.md` still describes nightly + weekly runs

**Commit(s):**
- https://github.com/elastic/integrations/commit/bebb005b5848848aaef614b653d22c5e7af48af1

**What changed:**
- `catalog-info.yaml` changed daily schedules to weekdays only (`cronline: "30 0 * 1-5"`, `"30 1 * 1-5"`, `"30 3 * 1-5"`, `"30 4 * 1-5"`) at `catalog-info.yaml` lines 186, 194, 200, 206.
- Weekly schedule entries were commented out in `catalog-info.yaml` lines 259-265 (`# NOTE: Stop running weekly for now`).

**Documentation impact:**
- `docs/ci_pipelines.md` still states:
- “pipeline running every night” for schedule-daily (`docs/ci_pipelines.md` line 9)
- “pipeline running once per week” for schedule-weekly (`docs/ci_pipelines.md` line 10)
- “Every night ... daily job” (`docs/ci_pipelines.md` line 149)
- active weekly section (“Every week it is configured to run...”) (`docs/ci_pipelines.md` lines 196-202)
- These statements are now inaccurate and should be updated to reflect weekday-only daily schedules and paused weekly scheduling.

### 2. PR CI package-selection docs are stale after expanded skip patterns

**Commit(s):**
- https://github.com/elastic/integrations/commit/989678548f4a95e82eb7968fe8d9f7004a7cd223

**What changed:**
- Expanded non-package skip patterns in `.buildkite/pull-requests.json` `skip_ci_on_only_changed` (lines 16-35), including `docs/`, `README.md`, `CONTRIBUTING.md`, `.mergify.yml`, `catalog-info.yaml`, etc.
- Matching pattern list in `.buildkite/scripts/common.sh` `non_package_patterns` was expanded (lines 765-788).

**Documentation impact:**
- `docs/ci_pipelines.md` still says that if files outside `packages` are changed, all packages are tested, with only narrow exceptions (`.github/CODEOWNERS` or `.docs/`) (`docs/ci_pipelines.md` line 70).
- This no longer matches current behavior and should be updated with the broader exception set and current path matching logic.

## Suggested Actions

- [ ] Update `docs/ci_pipelines.md` top-level pipeline descriptions and Daily/Weekly sections to reflect weekday-only daily schedules and currently disabled weekly schedules.
- [ ] Update `docs/ci_pipelines.md` Pull Request package-selection rules to match current skip patterns in `.buildkite/pull-requests.json` and `.buildkite/scripts/common.sh`.
- [ ] Replace stale examples (`.docs/`) with current paths (`docs/`) and list representative exclusions now treated as non-package changes.

> [!NOTE]
>
> 🔒 Integrity filter blocked 11 items
>
> The following items were blocked because they don't meet the GitHub integrity level.
>
> - [#6571](https://github.com/elastic/integrations/issues/6571) `search_issues`: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
> - [#16894](https://github.com/elastic/integrations/issues/16894) `search_issues`: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
> - [#18591](https://github.com/elastic/integrations/pull/18591) `search_pull_requests`: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
> - [#17979](https://github.com/elastic/integrations/pull/17979) `search_pull_requests`: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
> - [#17506](https://github.com/elastic/integrations/pull/17506) `search_pull_requests`: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
> - [#18662](https://github.com/elastic/integrations/pull/18662) `search_pull_requests`: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
> - [#18744](https://github.com/elastic/integrations/pull/18744) `search_pull_requests`: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
> - [#18671](https://github.com/elastic/integrations/pull/18671) `search_pull_requests`: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
> - [#16732](https://github.com/elastic/integrations/pull/16732) `search_pull_requests`: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
> - [#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".
>
> 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/25666044931)

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.
> - [x] expires on May 18, 2026, 11:09 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.